[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Added autopkgtests for gifticlib




On 6/2/22 01:46, Étienne Mollier wrote:
Hi Mohd Bilal,

Mohd Bilal, on 2022-06-01:
I've added autopkgtests for gifticlib[1].However the reprotest fails
currently. The error log looks similar to that of nifticlib. So should I
disable reprotest as done in nifticlib ?

[1] - https://salsa.debian.org/med-team/gifticlib/

I had a quick lookup at it and I'm happy with the autopkgtest,
the way data are introduced, their relatively light weight
compared to the rest of the package, their declaration in the
copyright file (modulo the syntax error reported by lintian:
"Continuation line not in paragraph (line 13)", but that is just
a matter of glueing the copyright paragraph to the "License:"
line I believe).

On the reproducible build front, I failed to reproduce the
failure to build reproducibly (if you follow me ;).  The
following output in build logs seemed suspicious to me:

	/tmp/reprotest.uo7yHg/const_build_path/const_build_path/gifti_io.c: In function 'gifti_disp_lib_version':
	/tmp/reprotest.uo7yHg/const_build_path/const_build_path/gifti_io.c:1701:48: warning: macro "__DATE__" might prevent reproducible builds [-Wdate-time]
	 1701 |     printf("%s, compiled %s\n", gifti_version, __DATE__);
	      |                                                ^~~~~~~~

However that might be a red herring if circumvented already by
an automated mechanism, and a diffoscope pass in salsa ci build
artifacts did not show me anything more than a few sections
shifted by a byte or so.  I wouldn't disable the test before at
least trying how goes the salsa ci with the warning corrected,
to answer your question.

Thanks for the pointer ! I tried removing the usage of '__DATE__' via a patch but still I'm getting this error in CI[2] . I'm having trouble finding out what is causing this error . Can you help me out here ? :)

I think once the copyright file is fixed, and eventually fixing
a couple of lintian informational messages, the package should
be in good shape for upload.


I fixed the copyright file as suggested.

Thanks for your contributions!
Have a nice day,  :)

[2] - https://salsa.debian.org/med-team/gifticlib/-/jobs/2840927#L815

--
╭────────────────────────────────────────────────────╮
│ Mohammed Bilal                                     │
│ 2D65 BC1E B966 5A6E 97F9  730A B3F5 9452 8521 9E1F │
╰────────────────────────────────────────────────────╯

Attachment: OpenPGP_signature
Description: OpenPGP digital signature


Reply to: