Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

Commit

Permalink
use native libs bundled with jna when running tests
Browse files Browse the repository at this point in the history
Summary:
This fixes test_ng test for appveyor.
The default logic of jna is to load native libs installed in the system
(http://java-native-access.github.io/jna/4.4.0/javadoc/overview-summary.html).
It may be the case that the system has jna native lib incompatible with what we use
(discussion: #1364) - and nailgun will not be able to start.

Test Plan:
Tested on appveyor: https://ci.appveyor.com/project/ilya-klyuchnikov/buck-07nvh/build/6
Tested on travis: https://travis-ci.org/ilya-klyuchnikov/nailgun/builds/238824664

Reviewed By: Coneko

fbshipit-source-id: 5e7c8f8
  • Loading branch information
ilya-klyuchnikov authored and facebook-github-bot committed Jun 5, 2017
1 parent 34afa35 commit 1a039d5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion third-party/nailgun/pynailgun/test_ng.py
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ def preexec_fn():
creationflags = DETACHED_PROCESS

self.ng_server_process = subprocess.Popen(
['java', '-jar', self.getNailgunUberJar(), self.transport_address],
['java', '-Djna.nosys=true', '-jar', self.getNailgunUberJar(), self.transport_address],
close_fds=True,
preexec_fn=preexec_fn,
creationflags=creationflags,
Expand Down

0 comments on commit 1a039d5

Please sign in to comment.