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
signature.asc
Description: PGP signature