Fixed embed.exe loader #51

Merged
merged 1 commit into from Apr 10, 2013

Projects

None yet

2 participants

@csoren
csoren commented Apr 3, 2013

embed.exe for Windows should not include the full classpath.jar as it's not being loaded by the embed-loader and only takes up (a lot of) space. Instead it should include the native portions only and the user should use ProGuard to cherrypick the parts of classpath.jar he needs.

This patch restores the original functionality to embed and embed-loader as originally submitted by me.

@dicej
Member
dicej commented Apr 3, 2013

Thanks, Carsten. I don't currently have a Windows machine available to test this with. I want to make sure it doesn't cause regressions with the Android and OpenJDK classpath builds, so I'll need to wait until next week when I can test it properly before I merge it.

@dicej dicej merged commit 43f41f6 into ReadyTalk:master Apr 10, 2013

1 check passed

Details default The Travis build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment