Skip to content

Conversation

@sogaiu
Copy link
Collaborator

@sogaiu sogaiu commented Jan 11, 2025

This PR is an attempt to address #244.

Please see that issue for some background.


If this PR is merged before #242, one of the files in that PR should probably have its filename changed.

The file in question is this one. It is currently named ->.janet, but likely a better name might end up being -_62.janet.

@bakpakin
Copy link
Member

Yup, this looks fine to me. Since this is a Makefile based project I don't really use windows for this but that could be revisited.

@bakpakin bakpakin merged commit d896d89 into janet-lang:master Jan 11, 2025
1 check passed
@sogaiu
Copy link
Collaborator Author

sogaiu commented Jan 12, 2025

Re: operation on Windows

As a data point, I tried make via a msys2 mingw64 shell [1] (the built janet.exe reports this as :cygwin for some reason).

The result was that mongoose wouldn't build (presumably this is for circlet).

If I can figure out how to set up a janet.exe that detects things as :mingw, I might try that too.


[1] I think this was set up via scoop's msys2 package.

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.

2 participants