Sean Whitton <spwhit...@spwhitton.name> writes:

> Hello,
>
> On Sun 10 Dec 2023 at 09:09pm -08, Xiyue Deng wrote:
>
>> So a little further reading from the policy[1] and the lintian bug[2]
>> helped me understand the usage of "Built-Using" a bit better: it's used
>> to include other source package required for building without having to
>> depend on them.  So technically it's not mutually exclusive with
>> arch:all as stated in the bug.  However, in the case of
>> persp-perspective, I tried with or without it and it doesn't make any
>> difference.  What's important is that ${elpa:Depends} correctly added
>> elpa-perspective and elpa-projectile to the dependency list of the
>> binary package.  So I think in the end dropping it should be OK.
>>
>> Still, it makes sense to clarify the actual reason to drop it, so I've
>> updated the changelog entry to reflect this fact[3].  PTAL, TIA!
>
> Well, it's more about ensuring that those source package versions aren't
> dropped from the archive by dak, rendering us license-incompliant.
> Thanks for looking into it further.  I've made a further change to your
> changelog message.  Please take a look.

LGTM.  Thanks!

>
> I've also noticed that there has been an upload to the archive,
> 1:0.2.0-4, which is not accounted for in our history.  Please merge it
> in.  'gbp import-dsc apt:persp-projectile/sid', and then a manual merge,
> is probably what you want, because of how the patches are unapplied.

Not sure how I missed this, sorry about that.  Somehow `apt source`
cannot find persp-projectile, and I see that there is actually a
"debian/1:0.2.0-4" tag created but the change is not merged to master
since I worked on it, so I just merged from the tag and resolved the
conflicts.  Also rebuilt and pushed to mentors[1].  PTAL, TIA!

[1] https://mentors.debian.net/package/persp-projectile/

-- 
Xiyue Deng

Attachment: signature.asc
Description: PGP signature

Reply via email to