commit 47c345d0274f34ec28d318603f6f39b6cb081106 Author: Jacob Appelbaum <jacob@appelbaum.net> Date: Fri Dec 7 11:49:23 2012 +0630 This fixes a broken commit and fixes scripts/before_i_commit.sh Normally, I would not put two fixes into one commit - however, git master was broken. The fix in txscapy.py fixes it such that I can run tests at all. The fix in scripts/before_i_commit.sh allows me to actually run the thing. It now depends on bash and as a result, it works properly. If we actually want this this to run on an older unix machine, such a user can fix it to work with a /bin/sh that isn't based on bash or later implementations. The previous version of scripts/before_i_commit.sh erased all .yamloo files and as a result, erased a lot of import data. If we run tests, we should not ever erase the data automatically. --- ooni/utils/txscapy.py | 2 +- scripts/before_i_commit.sh | 21 +++++++++++++++++---- 2 files changed, 18 insertions(+), 5 deletions(-) diff --git a/ooni/utils/txscapy.py b/ooni/utils/txscapy.py index fc021f1..62bde94 100644 --- a/ooni/utils/txscapy.py +++ b/ooni/utils/txscapy.py @@ -22,6 +22,7 @@ try: from scapy.arch import pcapdnet config.pcap_dnet = True + from scapy.all import Gen, SetGen, MTU except ImportError, e: log.err("pypcap or dnet not installed. " @@ -86,7 +87,6 @@ class ScapyFactory(abstract.FileDescriptor): https://github.com/enki/muXTCP/blob/master/scapyLink.py """ def __init__(self, interface, super_socket=None, timeout=5): - from scapy.all import Gen, SetGen, MTU abstract.FileDescriptor.__init__(self, reactor) if interface == 'auto': diff --git a/scripts/before_i_commit.sh b/scripts/before_i_commit.sh index 69975ff..918b137 100755 --- a/scripts/before_i_commit.sh +++ b/scripts/before_i_commit.sh @@ -1,4 +1,4 @@ -#!/bin/sh +#!/bin/bash # This script should be run before you commit to verify that the basic tests # are working as they should # Once you have run it you can inspect the log file via @@ -10,10 +10,24 @@ # rm *.yamloo; rm before_i_commit.log # -rm before_i_commit.log +if [ -f before_i_commit.log ]; +then + # this is technically the date it was moved, not the date it was created + mv before_i_commit.log before_i_commit-`date +%s`.log; + touch before_i_commit.log; +else + touch before_i_commit.log; +fi find . -type f -name "*.py[co]" -delete +if [ -f env/bin/activate ]; +then + source env/bin/activate; +else + echo "Assuming that your virtual environment is pre-configured..."; +fi + ./bin/ooniprobe -i decks/before_i_commit.testdeck echo "Below you should not see anything" @@ -25,5 +39,4 @@ echo "Read through the log file and fix it." echo "If you are having some problems fixing some things that have to do with" echo "the core of OONI, let's first discuss it on IRC, or open a ticket" read -cat *yamloo | less -rm -f *yamloo +#cat *.yamloo | less