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

Created Rudimentary EXE Compiler #44

Merged
merged 1 commit into from
Oct 14, 2017
Merged

Conversation

tunnelsnake
Copy link
Contributor

Happy Hacktoberfest!

@panagiks
Copy link
Owner

panagiks commented Oct 2, 2017

Happy Hcktoberfest! Thank you for you contribution and sorry for missing your previous comment on #33 :s I'll test this PR in the following couple days (as soon as I spin up a windows vm :p )

@tunnelsnake
Copy link
Contributor Author

Yeah, and as far as I can see, it's really hard to compile a windows application from a linux platform (which I feel your pain, but I have a dual boot system). Maybe there is a way around this, though. What if we were to compile a single executable as a release, and then integrate the addition of something like serialized plugins to customize the payload? That way you wouldn't have to recompile every time. P.s. great project, good code, look forward to many contributions

@panagiks
Copy link
Owner

panagiks commented Oct 3, 2017

Hmmm that sounds like a very good idea :) Command addition through plug-ins is already implemented on v0.4.0_rc so once that is merged into master it should be more or less straight forward.

Even if this doesn't work out well though it's not such a blow to require the user to compile on Windows, I just noted it to excuse the fact that it'll be a couple days before I can test and merge :p

Nevertheless I'll have to make my self a windows env to test RSPET on (and to build the release executables) and I've been putting it off for a while so ... it's time to do it.

Looking forward to future contributions :)

@panagiks panagiks merged commit de4356e into panagiks:master Oct 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants