Hi,
I would like to ask on your opinion on
https://bugzilla.redhat.com/show_bug.cgi?id=912960#c25
Mamoru have in spec in %check section:
ruby -Ilib:test ./test/run-test.rb || echo "Please investigate this"
I'm saying that it should not be waived this way as it waive all failure.
I'm saying that if there is no failure, it should be just:
ruby -Ilib:test ./test/run-test.rb
and if there happened some problem which need upstream attention and
could not be fixed immediatelly, it should be replaced with something like:
FILE=$(mktemp)
ruby -Ilib:test ./test/run-test.rb | tee $FILE || :
# test foo is failing, reported as http://foo/issue/1
cat $FILE | grep "4 tests, 4 assertions, 1 failures, 0 errors, 0
pendings, 0 omissions, 0 notifications
Mamoru disagree. For me it is blocker.
I would like to ask for your opinions. Which way is correct and should
it be blocker for review?
Thanks in advance.
Mirek
--
packaging mailing list
packaging@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/packaging
