Merge ~xnox/germinate/+git/builtusing2:master into germinate:master

Proposed by Dimitri John Ledkov
Status: Needs review
Proposed branch: ~xnox/germinate/+git/builtusing2:master
Merge into: germinate:master
Diff against target: 102 lines (+25/-6)
3 files modified
debian/changelog (+5/-1)
germinate/germinator.py (+19/-5)
germinate/tests/test_germinator.py (+1/-0)
Reviewer Review Type Date Requested Status
Colin Watson Needs Fixing
Adam Conrad Pending
Steve Langasek Pending
Ubuntu Package Archive Administrators Pending
Review via email: mp+308743@code.staging.launchpad.net

Description of the change

built-using in components mismatches try #2.

http://people.canonical.com/~xnox/germinate-output/builtusing2/components-missmatches.html

No changes to .svg

To post a comment you must log in.
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Shall I upload this to the archive? But it would need deployment on snake fruit too...

Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Note the:
golang-go-xdg: golang-go-xdg-dev
[Reverse-Depends: Rescued from golang-go-xdg (Reverse Built-Using ubuntu-push-client,golang-ubuntu-push-dev)]

in the new html.

Currently it says for this package:
golang-go-xdg: golang-go-xdg-dev
[Reverse-Depends: Rescued from golang-go-xdg]

No other changes to any other logic, just the comment that is pushed out to the .html output has more useful information.

Revision history for this message
Colin Watson (cjwatson) wrote :

This looks broadly OK, but I've commented on a few things I'd like to see fixed before we land this.

review: Needs Fixing
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Thank you for this review and comments.

Thinking about this now, I think the most user-friendly result here would be to do the following:

 - inspect built-using
 - if built-using binaries, are in build-deps
 - turn follow-build-depends back on for that build-dep

Plus, all the things you mentioned in your review comments including go and non-go test cases.

Revision history for this message
Adam Conrad (adconrad) wrote :

> - if built-using binaries, are in build-deps

Built-Using contains sources, not binaries. We don't require build-deps to be in main, and I don't see why binaries of Built-Using should be any more special. We should be concerned with promoting sources here, not binaries.

ie: If I build foo-main against libbaruniverse-static, Canonical is committing to support the baruniverse SOURCE that produced libbaruniverse (to make sure we get it updates with security fixes, etc), and we have to ship it alongside foo for GPL compliance reasons, but that doesn't mean we're saying we support users installing the binary package libuniverse-static.

Trying to guess at binaries seems like a pointless exercise, really, and it's entirely possible to promote a source but none of its binaries.

Revision history for this message
Dimitri John Ledkov (xnox) wrote :

ok. This makes more sense.

On Fri, 18 Oct 2019 at 17:09, Adam Conrad <adconrad@0c3.net> wrote:
>
> > - if built-using binaries, are in build-deps
>
> Built-Using contains sources, not binaries. We don't require build-deps to be in main, and I don't see why binaries of Built-Using should be any more special. We should be concerned with promoting sources here, not binaries.
>
> ie: If I build foo-main against libbaruniverse-static, Canonical is committing to support the baruniverse SOURCE that produced libbaruniverse (to make sure we get it updates with security fixes, etc), and we have to ship it alongside foo for GPL compliance reasons, but that doesn't mean we're saying we support users installing the binary package libuniverse-static.
>
> Trying to guess at binaries seems like a pointless exercise, really, and it's entirely possible to promote a source but none of its binaries.
> --
> https://code.launchpad.net/~xnox/germinate/+git/builtusing2/+merge/308743
> You are the owner of ~xnox/germinate/+git/builtusing2:master.

--
Regards,

Dimitri.

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches