Re: Request for Rawtran packaging review
Hi Ole,
On Thu, 2016-04-07 at 09:37 +0200, Ole Streicher wrote:
> One point here: In the moment, the check gives no error, but it actually
> checks the wrong thing: it rebuilds the rawtran executable and checks it
> instead of the installed executable.
>
> This is not what the CI test is intended to do. Imagine f.e. if cfitsio
> breaks its ABI but not the API, then the recompiled executable would
> work, but the packaged one not -- and these are exactly the errors one
> would like to find with the CI test.
>
> I would just pragmatically compile the fitslint program manually (into
> $ADTTMP) and run the test by hand there.
>
You're right. The way violated the rule:
"""
The cwd of each test is guaranteed to be the root of the source package,
which will have been unpacked but not built. *However* note that the
tests must test the *installed* version of the program. Tests may not
modify the source tree (and may not have write access to it).
"""
So I uploaded corrected version. Please, check it.
Thank you
FH
--
F. Hroch e-mail, jabber: hroch@physics.muni.cz, tel.: +420549494470
Dept. of theor. physics and astrophysics, MU Brno, Kotlarska 2,CZ-611 37
Reply to: