-
-
Notifications
You must be signed in to change notification settings - Fork 7k
[windows] Do not unzip jnidispatch.dll in tmp folder #6052
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
Conversation
Normally if JNA doesn't find the jnidispatch.dll it tries to extract it from the jar into a temp folder and load from there. This is being restriscted on Microsoft Apps for the app-store and this commit prevents this behaviour.
Previously the ant call "chmod" was used, but this is a no-op on Windows, so it would works only if the IDE is cross-built from a unix host. This patch allows to build a distribution of the IDE also from a Windows host.
355eac8
to
f09b24c
Compare
✅ Build completed.
Please test this code using one of the following:
⬇️ http://downloads.arduino.cc/javaide/pull_requests/arduino-PR-6052-BUILD-659-linux32.tar.xz
⬇️ http://downloads.arduino.cc/javaide/pull_requests/arduino-PR-6052-BUILD-659-linux64.tar.xz
⬇️ http://downloads.arduino.cc/javaide/pull_requests/arduino-PR-6052-BUILD-659-windows.zip
⬇️ http://downloads.arduino.cc/javaide/pull_requests/arduino-PR-6052-BUILD-659-macosx.zip
⬇️ http://downloads.arduino.cc/javaide/pull_requests/arduino-PR-6052-BUILD-659-linuxarm.tar.xz
i️ The linuxarm
build is still experimental and may not be always available.
Normally if jnidispatch.dll is not installed system-wide, JNA tries to extract it from his own jar into the temp folder and tries to load from there.
This is being restriscted on Microsoft Apps and this patch prevents this behaviour.