On 03/24/2014 10:42 AM, Mike Latimer wrote:
The network/100-apply-verify-host.t test compares the results of various network commands (route, iptables, ebtables, etc...) against known, valid results. Depending on the versions of these tools, minor whitespace differences in command output can result in the failure of the test. This patch resolves the problem by removing trailing spaces and replacing three or more concurrent spaces with just two. These changes are only made to the temporary files involved with the tests.
--- scripts/networks/networkApplyTest.sh | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/scripts/networks/networkApplyTest.sh b/scripts/networks/networkApplyTest.sh index 3e93c0a..924130f 100644 --- a/scripts/networks/networkApplyTest.sh +++ b/scripts/networks/networkApplyTest.sh @@ -142,6 +142,11 @@ checkExpectedOutput() { break fi
+ # Remove trailing whitespace, replace multiple spaces with just two + for file in ${tmpfile} ${tmpfile2}; do + sed -i -e 's/ *$//' -e 's/ */ /g' $file
'sed -i' is not portable (it is a GNU-ism) - but it's not the first time we've assumed GNU tools in this testsuite so I can overlook it. Your code does not quite match your commit comment. The first -e says to remove trailing spaces; the second -e says to replace all runs of 1 or more space with 2 spaces. You MEANT to use 's/ */ /g' for the second expression. ACK with that fix, and pushed. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org