Skip to content

Conversation

Tachi107
Copy link
Member

Closes: #289

@guillemj
Copy link
Contributor

Thanks for the MR. I think this is good in general, but I'm not sure whether the golang packages will currently take much benefit from specifying the dpkg-build-api, given that at least two of the behavior changes for v1 are now the default, and the others might not apply to golang packages in the common case. My concern would be with introducing unnecessary churn.

In any case, this would be a +1 from me (in gerrit terms, as in "LGTM but someone else needs to approve with +2") given that I'm clearly biased here (from an implementer PoV), but if others think this is good and the churn is not a major concern, then let's go for it?

@Tachi107
Copy link
Member Author

Tachi107 commented Aug 12, 2025 via email

Copy link
Contributor

@ottok ottok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Motivations for this change seem reasonable after reading policy and dpkg-build-api man page as mentioned in #289. Approving, so it has +1 from me.

@guillemj Do you want to try approve as well so we can mimick the +1+2 functionality you longer from gerrit?

@guillemj
Copy link
Contributor

@ottok As I tried to convey (but perhaps I was not clear) in my earlier comment, I feel I have a conflict of interests here, as the implementer of this feature in dpkg. So if the team is fine with the potential burden this might involve, I'm good with having this merged, but otherwise, this something I'd let the project at large adopt organically w/o any push from my side, that's why I mentioned the +1 (I'll give a thumbs up, which is the closes this maps to the gerrit semantics, where a +2 would be an approval that lets someone merge it). I'm thinking that perhaps this should be mentioned on the go-team mailing lists, in case some one has an issue with it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add dpkg-build-api (= 1) to the default list of build dependencies
3 participants