Bug#1076211: RFS: msc-generator/8.6.1-1 -- Draws signalling charts from textual description
On 2024-07-22 21:08, Boyuan Yang wrote:
> 在 2024-07-22星期一的 06:18 +0000,Gábor Németh写道:
>> Hi,
>>
>> On 2024-07-22 03:05, Boyuan Yang wrote:
>> > > https://mentors.debian.net/debian/pool/main/m/msc-generator/msc-generator_8.6.2-1.dsc
>> >
>> > I think all changes look okay except for adding the usage of dpkg-vendor and
>> > exporting a specific environment variable only on Debian in debian/rules.
>>
>> I'll try to explain. This env var:
>>
>> +VENDOR ?= $(shell dpkg-vendor --query vendor)
>> ...
>> +ifneq ($(VENDOR),Debian)
>> +export PNGDIFF_LUMA_DIFF = 10
>> +endif
>>
>> In short, `dpkg-vendor` here only controls precision of PNG-generating
>> tests and has no effect on the binary tool itself whatsoever.
>>
>> [1] https://bugs.launchpad.net/ubuntu/+source/msc-generator/+bug/2061755
>
> If you are tackling [1], I have to remind you that your dpkg-vendor comparison
> only applies to Debian and won't go into effect on Ubuntu. In other words,
I understand that and in no case want to handle any other distros at
all. That is why I'm setting the variable for all cases *except* Debian.
> My current understanding is that this environment variable loosens the
> comparison threshold in tests.
That is right: I want to loosen the tests in every distro except Debian.
The rationale is that upstream's CI works on Debian so there we expect
the default (=stricter) test settings to pass.
Can you agree?
BR,
Gábor
Reply to: