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

Generate Windows-compatible plugin artifact #16

Merged
merged 1 commit into from
Dec 29, 2017
Merged

Conversation

gfecher
Copy link

@gfecher gfecher commented Dec 28, 2017

Generate Windows-compatible plugin artifact
Fixed build on Windows

@gfecher
Copy link
Author

gfecher commented Dec 28, 2017

Hi,

I managed to make it build under Windows and the build now also creates a Windows-compatible plugin artifact (using launch4j).
I have replaced the prepend-stub.sh with a groovy implementation to make it platform-agnostic.
Unfortunately it seems that the working directory for the plugin is different between the Windows and Linux protoc implementation, so I needed to hack the path to the yaml file for Windows.

Let me know what you think.

I'd appreciate it if you could cut a new release from this.

Cheers,
Feso (pronounced Fesho)

Copy link
Owner

@Xorlev Xorlev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really great work! Thanks for cleaning this up.

@Xorlev
Copy link
Owner

Xorlev commented Dec 29, 2017

I've since moved on from FullContact (though I retain full copyright and license). I'll be republishing this library to bintray today.

@Xorlev Xorlev merged commit 6a13ccf into Xorlev:master Dec 29, 2017
@gfecher
Copy link
Author

gfecher commented Dec 29, 2017

Thanks a lot!

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