Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TAS Playback (0.1) #91

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

rcombs
Copy link

@rcombs rcombs commented Jul 29, 2023

Application Submission

This app plays back tool-assisted speedrun files on video game consoles (currently only the Nintendo 64)

Extra Requirements

This reads files from the microSD card and sends output to a game console connected via the GPIO pins. This usually requires modifying a controller extension cable.

Author Checklist (Fill this out)

  • I've read the contribution guidelines and my PR follows them
  • I own the code I'm submitting or have code owner's permission to submit it
  • I validated my manifest to be valid

Reviewer Checklist (Don't fill this out)

  • Bundle is valid
  • There are no obvious issues with the source code
  • I've ran this application and verified its functionality

@rcombs
Copy link
Author

rcombs commented Jul 29, 2023

It looks like the CI build is failing because the Flipper code in the repo #includes code from outside the Flipper-specific subdirectory, and the CI copies just the Flipper subdirectory into the tmp location where the build is run. Is this a CI bug, or is this configuration not supported? It's somewhat important for multi-platform applications like this one.

@hedger
Copy link
Member

hedger commented Jul 29, 2023

Yes, everything your app uses should be in its folder. You can use fap_private_libs for building 3rd-party code with specific defines or requirements.

Your app is cool, looking forward to compatibility fixes!

@rcombs
Copy link
Author

rcombs commented Jul 30, 2023

This isn't about third-party code; it's about my own app's platform-generic code, which is pulled in by the flipper variant. Just cramming it in flipper/lib wouldn't help. I guess I could put the .fam in a separate repo, and pull in the real code as a submodule? Or I could put the .fam in the root of the repo, but it'd be pretty weird to have a single flipper-specific file in there while everything else lives in a subdirectory.

@hedger
Copy link
Member

hedger commented Jul 30, 2023

I looked into details of your project - and I think it can be slightly changed by moving common code into a submodule and adding that submodule to Flipper's application dir.

Unfortunately, current catalog's implementation requires all code to be within root app's directory - the one with application.fam in it. Bundler script creates a snapshot of source code in that directory and stores it on the backend. Includes from outside of that directory are not resolvable after that, leading to app build errors.

@skotopes
Copy link
Member

@hedger i think we can switch to git-archive-all that can perform submodules packing

@hedger hedger added app (new) fixes needed Unresolved issues with the app requiring maintainer's attention labels Nov 9, 2023
@hedger
Copy link
Member

hedger commented Dec 22, 2023

Please un-draft when fixed.

@hedger hedger marked this pull request as draft December 22, 2023 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app (new) fixes needed Unresolved issues with the app requiring maintainer's attention
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants