Bug 230278 - devel/binutils: Plist contains files that aren't present.
Summary: devel/binutils: Plist contains files that aren't present.
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: John Baldwin
Depends on:
Reported: 2018-08-02 00:20 UTC by Andrey Cherkashin
Modified: 2018-08-20 12:29 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (bapt)

fix plist (990 bytes, text/plain)
2018-08-02 00:20 UTC, Andrey Cherkashin
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Cherkashin 2018-08-02 00:20:31 UTC
Created attachment 195738 [details]
fix plist

https://svnweb.freebsd.org/ports/head/devel/binutils/Makefile?annotate=476186&pathrev=476186#l156  Borked the build.

Patch fixes it.
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2018-08-02 03:57:29 UTC
Over to committer of r476186.
Comment 2 John Baldwin freebsd_committer freebsd_triage 2018-08-02 15:24:33 UTC
Patch looks correct.  It seems the port was not removing these manpages that should have been removed before.  bapt@ is the maintainer so should probably approve it though.
Comment 3 John Baldwin freebsd_committer freebsd_triage 2018-08-03 15:54:40 UTC
Antoine already reverted the commit, but I'll include a fixed plist when I reapply it.
Comment 4 commit-hook freebsd_committer 2018-08-20 09:18:46 UTC
A commit references this bug:

Author: jhb
Date: Mon Aug 20 09:17:46 UTC 2018
New revision: 477638
URL: https://svnweb.freebsd.org/changeset/ports/477638

  Cleanup packing lists.

  - Use BUREMOVE to strip binutils tools not installed by the base/binutils
  - Update BUREMOVE logic in devel/binutils to cope with the base package
    which installs tools without a BUTARGET- prefix.
  - Use MANPREFIX for BUREMOVE to handle the PREFIX=/usr case used by
  - Remove binutils headers and libraries explicitly from the staging area
    for base/binutils.
  - Add missing plist entries for binutils binaries installed under a
    BUTARGET subdirectory.
  - Drop plist entries from devel/binutils that are now properly removed.
    Previously the binaries for Windows tools like dlltool were removed
    from the staging area but the manpages were still left in the package.

  This is a recommit of r476186 but with the update to the pkg-plist of
  devel/binutils and PORTREVISION bump.

  PR:		230278
  Reviewed by:	antoine
  Differential Revision:	https://reviews.freebsd.org/D16582