Bug 231332 - Mk/Scripts/qa.sh: Fix flavor detection for LIB_DEPENDS
Summary: Mk/Scripts/qa.sh: Fix flavor detection for LIB_DEPENDS
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Ports Framework (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Port Management Team
Depends on:
Reported: 2018-09-12 17:36 UTC by David Naylor
Modified: 2019-01-11 12:51 UTC (History)
1 user (show)

See Also:
koobs: merge-quarterly?

qa.sh: use package name instead of port origin (682 bytes, patch)
2018-09-12 17:36 UTC, David Naylor
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Naylor freebsd_committer 2018-09-12 17:36:58 UTC
Created attachment 197060 [details]
qa.sh: use package name instead of port origin

qa.sh uses `pkg annotate -q -S $origin flavor` to get the flavor of file's package.  However, if there are multiple flavors of a port origin installed then things brake:

# pkg info -E multimedia/phonon
# pkg annotate -q -S multimedia/phonon flavor

The attached patch ensures ensures the full package name is used, instead of the origin.
Comment 1 Mathieu Arnold freebsd_committer 2018-09-14 12:54:16 UTC
Could you provide a port that is broken, to test this?

Or a port that can be broken to test it?
Comment 2 David Naylor freebsd_committer 2018-09-14 14:09:27 UTC
If you have both versions of phonon installed then audio/soundkonverter will fail stage-qa.
Comment 3 David Naylor freebsd_committer 2018-12-11 07:14:11 UTC
Any objections against committing this patch?
Comment 4 commit-hook freebsd_committer 2019-01-11 12:51:49 UTC
A commit references this bug:

Author: mat
Date: Fri Jan 11 12:50:52 UTC 2019
New revision: 489958
URL: https://svnweb.freebsd.org/changeset/ports/489958

  Fix proxydeps QA check when multiple flavor of a package are installed.

  Previously it would do this:

  $ pkg which -q -o /usr/local/lib/libphonon4qt5.so
  $ pkg pkg annotate -q -S multimedia/phonon flavor

  And that would break things in an interesting way.

  This changes makes it use the package name of the required file, which
  is unique.

  Note that this problem would probably only ever happen on a live system.
  I was not able to find a single port that would trigger this it in a
  clean environment (poudriere testport).

  PR:		231332
  Submitted by:	dbn