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

Comments Missing #1

Open
Cekis opened this issue Oct 31, 2019 · 10 comments
Open

Comments Missing #1

Cekis opened this issue Oct 31, 2019 · 10 comments

Comments

@Cekis
Copy link

Cekis commented Oct 31, 2019

Would love it if you could take the original files, grab the comments, and insert them into their respective position in the new Java files. Obviously, we've progressed beyond a point that they're really needed, but they are useful from time to time.

By the way, thank you for creating this originally! For those projects that did migrate to Java from their .script equivalent versions it created a very solid .script to Java conversion.

@Rohansi
Copy link
Owner

Rohansi commented Nov 21, 2019

Sorry for the late reply! By new Java files do you mean the output from this program or the ones being used by the private server? It'd be relatively easy to add to the output here but I think adding them to the private server's copy might be better if done as a git merge after reconverting the scripts.

@Cekis
Copy link
Author

Cekis commented Nov 21, 2019

Ah, that's actually a great idea. What I originally meant was adding them to the "server's copy". Which, by the way, isn't an active server. We took your original output from 2014-ish and intended to open a server with it, but didn't. Instead, I pushed to "open" the work we did and the result has been "SWG-Source". We shared all of our bug fixes, additions, etc. with anybody that wants to add them to their server (private or public). It's now a progressing repo: https://github.com/SWG-Source/.

@Rohansi
Copy link
Owner

Rohansi commented Nov 21, 2019

Very cool! I've never actually got to try the game before so I might have to give it a go later.

@Cekis
Copy link
Author

Cekis commented Nov 21, 2019

I figured you probably hadn't tried it, but what I wanted to say was that without your conversion, it would have been much more difficult for people to work on it. Was thinking you may want to see how far it has come via commit history and such and just how long your "output" has been in use. But playing the game is a good option too, lol! At any rate, I plan on using your tool to do what you suggested. Hopefully things have not changed that much and it can still happen.

@Rohansi
Copy link
Owner

Rohansi commented Nov 21, 2019

Yeah, it's nice seeing what came of this. I didn't really know the person that asked me if I could complete this and never kept in touch with him after so it was silence until I dug around and found that repo. Keep up the good work!

@Rohansi
Copy link
Owner

Rohansi commented Nov 24, 2019

@Cekis I meant to ask, did anyone end up making fixes to this code? I might start on this today or next weekend, but it'd be good to bring changes back into this repo first!

@Cekis
Copy link
Author

Cekis commented Nov 26, 2019

Sorry was on vacation. I don't believe anybody made changes? It's possible as we didn't do the full switch from script to java until later (2016 ish), but I doubt there were any changes.

@Rohansi
Copy link
Owner

Rohansi commented Nov 26, 2019

I guess we'll find out when it's time to merge then!

@Cekis
Copy link
Author

Cekis commented Nov 26, 2019 via email

@rohvani
Copy link

rohvani commented Jan 5, 2020

Just some background: the people behind this request was the original SWG Reborn team (myself/seefo, aconite, light, etc). This tool was developed prior to @apathyboy's python-based transpiler being released alongside the rest of swgreveniens work (in the swgmasters repos).

@Rohansi Just wanted to clarify, since we never moved forward with your tool, due to issues with many of the scripts not being properly converted (even though they all compiled), so these issues would need to be fixed before @Cekis could use this.

@apathyboy's transpiler may be found here: https://bitbucket.org/swgmasters/swg-src/src/master/utils/mocha/

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

No branches or pull requests

3 participants