Skip to content

janet support #2079

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 77 commits into from
Jan 22, 2023
Merged

janet support #2079

merged 77 commits into from
Jan 22, 2023

Conversation

AlecTroemel
Copy link
Contributor

@AlecTroemel AlecTroemel commented Jan 11, 2023 •
edited
Loading

possible replacement for #2069, builds off the work of @dressupgeekout. All functions bindings are complete, things left to do:

  • make tic80 a module so the user can choose between (import tic80) or (use tic80). This is to avoid function name collisions of print and map
  • fix bug where random characters show up at end of errors
  • get all builds passing

dressupgeekout and others added 30 commits September 1, 2022 00:40
@dressupgeekout
WIP WIP WIP Adding Janet bindings
2f253ac
@dressupgeekout
WIP slowly but surely filling in the gaps
118037d
@dressupgeekout
WIP Add Janet 'vendor' submodule, begin touching up CMake
41a96ec
@dressupgeekout
OK I think this is how we're gonna init the janet env
12081d7
@dressupgeekout
Actually kinda builds
c7462ee
@dressupgeekout
I think I'm getting close
1a6f3b1
@dressupgeekout
needed to add ourselves in the tic80_config.h
ef9d33b
@dressupgeekout
more nuanced error handling and stuff
e9b29d3
@dressupgeekout
Don't die with the other "intcallback" callbacks
6b2b048
@dressupgeekout
filling in more gaps
e5cf36e
@dressupgeekout
Much better way of determining whether a symbol exists
6374963 
They REALLY should mention janet_resolve() in Janet's docs
@dressupgeekout
more error reporting
db8a63d
@dressupgeekout
whitespace and other fun
4813d7b
@dressupgeekout
Sync vendor/janet to the latest version in 'master'
9763b97 
Turns out there actually was a bug in Janet's compiler, which was
responsible for making Janet panic in a scenario where it shouldn't have
@dressupgeekout
add some more functions
be76200
@dressupgeekout
use TIC-80's integer typedefs
086798e
@dressupgeekout
add fget() and fset() bindings for Janet
0b0b044
@AlecTroemel
add implimentation for janet_spr
1d7b3c5
@AlecTroemel
add janet_map... no remap though (wren doesnt have it either)
77fde67
@AlecTroemel
add peek and poke
7794a37
@AlecTroemel
refactor to reuse colorkey code, imp ttri
b1781bc
@AlecTroemel
use opt for janet_music
c6c94cf
@AlecTroemel
use opt* in janet_sync
e0a1204
@AlecTroemel
imp janet_sfx, handle case where note is str
d3ab669
@AlecTroemel
cleanup janet_font
564f841
@AlecTroemel
finish up janet_mouse
ce72cf9
@AlecTroemel
handle arity error
de7a30d
@AlecTroemel
add janet demos, update keywords
acc9b1e
@AlecTroemel
use put instead of place
a117be1
@AlecTroemel
print out stacktrace
541cb00
@AlecTroemel
put the janet core api "map" fn under the name iter/map
c281011 
avoids name collision with the tic80 map function. Could have just as easily
done the same to that one, but chances are we'll be using the tic80 map more
often.
@nesbox
Copy link
Owner

nesbox commented Jan 12, 2023

Great, new language support is a good reason to release a new TIC-80 version.
Thank you 👍

@AlecTroemel
Copy link
Contributor Author

To the people watching this PR, I think I'm going to need some assistance with the failing builds.

any help or points in the right direction would greatly appreciated, as I'm pretty stumped..

@AlecTroemel AlecTroemel mentioned this pull request Jan 21, 2023
Closed
@AlecTroemel AlecTroemel marked this pull request as ready for review January 22, 2023 19:13
@AlecTroemel
Copy link
Contributor Author

With the help of the great people over in the janet Gitter chat, all platforms builds are now passing! I've marked this PR as ready for review

@nesbox nesbox merged commit 9a3d502 into nesbox:main Jan 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nesbox nesbox nesbox approved these changes

Assignees

@AlecTroemel AlecTroemel

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AlecTroemel @nesbox @dressupgeekout