-
Notifications
You must be signed in to change notification settings - Fork 3
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
Alan v2 interpreter license #53
Comments
Thanks for you interest in Alan! I will add a license file in the v2 branch to clarify that it is actually the same license as the later versions. If you are interested in joining the very small Alan team I would be happy to have you maintain the v2 branch from within the AlanIF organisation. That way we could keep a unified branch and perhaps also have v3 benefit from some of your cleanups. Just a thought. |
I'd be happy to do maintenance under the AlanIF organization. However, when I said I rewrote large chunks, that's probably an understatement at this point. A lot of the core logic is still the same (I mean, it has to be, obviously), but it's really a very different beast at this point. I'll list some of the major things I did, so you can get a sense of the state of my version of the interpreter, and how you'd feel about it being at least a semi-official part of Alan.
There are likely other relatively large things I've forgotten (my Git history is too scattered to easily scan for the major changes). I'll also quickly explain the 64-bit problems that I was original addressing:
My original goal was to ensure the interpreter works fine in Gargoyle, and as such, I was only really concerned with supporting systems on which Gargoyle runs (which is to say, really, anything with a C++14 compiler). Once I started doing a lot of the renovations, I wasn't concerned with older/more obscure target systems. So that may not really be in line with what you're looking for, but on the other hand, I didn't go out of my way to make it unportable. Anyway, if you'd like, I can at least toss up a repository somewhere with these changes, if you don't mind it being public, or if you'd prefer, I can privately send you a copy of the latest version I've got, so you can peruse it. I'm happy to work with you to ensure it has everything you need, but I completely understand if this is just a bridge too far, after all the changes I've made. |
While Alan v2 is obsolete, Gargoyle still includes an interpreter for older games that were created using it, thus my interest here.
The Alan v2 interpreter (in the
v2_x-updates
branch) doesn't work on modern 64-bit systems, as it assumes pointers are 32 bits. Gargoyle made some changes a while back to fix issues with pointers on the stack, but there are still problems: theclock.alan
regression test, for example, segfaults under Gargoyle due to another pointer issue.I was working on making it 64-bit clean, planning on getting some minimal fixes into the v2 branch here, and then into Gargoyle. But I got kind of carried away and rewrote large chunks of the interpreter, some necessary for the 64-bit fixes, which were a bit difficult to get done, some not necessary for the fixes.
Anyway, since I know you're not maintaining v2 given that v3's been out forever, I plan on maintaining this version myself, but I see that the v2 source contains no license (neither in the Git branch nor the
interpreter287src.tgz
archive), at least as far as I can tell.Can you clarify what license the v2 source code is under, please? I don't want to distribute something I have no right to.
The text was updated successfully, but these errors were encountered: