On 27 June 2012 08:51, Thomas Preud'homme
<address@hidden> wrote:
Hi Jovanovic,
I've just seen your commit to add more tests in the test2 directory and I have
2 suggestions:
1) it might be good to indicate the license (BSD 2-clauses if I'm right) in a
file or add the copyright notice at the top of the files.
2) as you say in the commit message, one of the purpose of tests is to avoid
regressions. Thus, it might be good for tests which fails on some architecture
to disable them on these architectures since they are not regressions and a
failure might prevent someone from commiting although he didn't break
anything. In Debian for example the package is not uploaded if the testsuite
fails.
Best regards,
Thomas Preud'homme