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

Add filesystem API and improve console output (add small text) #2162

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

empathicqubit
Copy link
Contributor

No description provided.

@suborb
Copy link
Member

suborb commented Jan 16, 2023

I've no idea why the build is wonky :(

@empathicqubit
Copy link
Contributor Author

It's because there's a symlink to the Windows output folder

@empathicqubit
Copy link
Contributor Author

This works on TI 83 Plus now (the file API. Screen stuff is a little sloppy), but I would like it to work on some others if possible before merging.

What are we doing about tests? I have written 5 or so dumb ones. I'm not sure how that would work considering how platform specific it is.

@suborb
Copy link
Member

suborb commented Jan 17, 2023

I've not come up with a good way of automatically testing machine specific stuff - and going down the route of taking screenshots of an emulator and comparing it to reference pictures is neither sustainable or fun.

What we usually end up doing is creating an example that sits in examples/[machine] that exercises the functionality - they can be hooked into the build so at least "it compiles" and then manually run when/if things change.

@empathicqubit
Copy link
Contributor Author

@suborb Is there a good way to figure out what causes the program size to be huge? Like some kind of dependency graph or something? My "simple" test program is close to 8K.

@suborb
Copy link
Member

suborb commented Jan 29, 2023

@suborb Is there a good way to figure out what causes the program size to be huge? Like some kind of dependency graph or something? My "simple" test program is close to 8K.

Sorry, I totally missed this. Adding -Cl-v is quite useful to figure out which routines are being pulled in.

@zx70
Copy link
Member

zx70 commented Oct 20, 2023

Appmake is finally able to manage the ti83p binary format decently.

@zx70
Copy link
Member

zx70 commented May 20, 2024

I'd really like to get the code merged but I'm lost.
Is it necessary to force the library cleanup on any target ?

=======
< ti-improvements
$(MAKE) -C target/ticalc clean
$(MAKE) -C target/ticalc

@empathicqubit, plase confirm the changes you are proposing in the Makefiles are still valid, and/or please refresh the two files to resolve the conflict.

libsrc/Makefile
libsrc/target/ticalc/Makefile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants