Anomie has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/339451 )

Change subject: Fix HHVM test script
......................................................................

Fix HHVM test script

The HHVM config used by Jenkins sets hhvm.dynamic_extension_path to
something other than the default '.', which results in the script
testing the system luasandbox.so instead of the one that was just
compiled. Avoid that issue by specifying the full path to the
just-complied luasandbox.so.

Change-Id: I958c0a105db52bf6cd6ffb42bb05c69d707b9dcd
---
M hhvm-test.sh
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/php/luasandbox 
refs/changes/51/339451/1

diff --git a/hhvm-test.sh b/hhvm-test.sh
index 1446426..682aa31 100755
--- a/hhvm-test.sh
+++ b/hhvm-test.sh
@@ -1,4 +1,4 @@
 #!/bin/sh
 # First run `phpize`, and then ./hhvm-test.sh run-tests.php
 export TEST_PHP_EXECUTABLE=$0
-hhvm -d "hhvm.dynamic_extensions[luasandbox]=luasandbox.so" "$@"
+hhvm -d "hhvm.dynamic_extensions[luasandbox]=$(pwd)/luasandbox.so" "$@"

-- 
To view, visit https://gerrit.wikimedia.org/r/339451
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I958c0a105db52bf6cd6ffb42bb05c69d707b9dcd
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/php/luasandbox
Gerrit-Branch: master
Gerrit-Owner: Anomie <bjor...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to