Michal Skrivanek has posted comments on this change. Change subject: net_tests: Move network unit tests into package ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/26874/1/tests/run_tests_local.sh.in File tests/run_tests_local.sh.in: Line 2: if [ -z "$PYTHON_EXE" ]; then Line 3: PYTHON_EXE="@PYTHON@" Line 4: fi Line 5: Line 6: # The following line is taken from http://stackoverflow.com/a/246128/206009 > attribution is great, but please say what it does - it was not clear to me. if you get rid of that @PYTHON@ above you don't have to process the file anymore….it can be a simple .sh Line 7: DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )" Line 8: Line 9: if [ ! -f "$DIR/jsonrpc-tests.server.crt" ] || [ ! -f "$DIR/jsonrpc-tests.server.csr" ] || [ ! -f "$DIR/jsonrpc-tests.server.key" ]; then Line 10: $DIR/makecert.sh -- To view, visit http://gerrit.ovirt.org/26874 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6960ce365d67ab4bb0a5475d7957e6117bef7e60 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
