Bug 220782 - devel/gcc-arm-embedded: Remove unnecessary LIB_DEPENDS on libelf
Summary: devel/gcc-arm-embedded: Remove unnecessary LIB_DEPENDS on libelf
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Torsten Zuehlsdorff
URL:
Keywords: easy, needs-qa
Depends on:
Blocks:
 
Reported: 2017-07-16 20:05 UTC by Jonathan Chen
Modified: 2017-08-17 14:45 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (lev)
tz: merge-quarterly+


Attachments
removes libelf dependancy (433 bytes, patch)
2017-07-16 20:05 UTC, Jonathan Chen
koobs: maintainer-approval+
Details | Diff
Updated patch for qa (1.70 KB, patch)
2017-08-08 23:21 UTC, Jonathan Chen
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Chen 2017-07-16 20:05:10 UTC
Created attachment 184411 [details]
removes libelf dependancy

devel/gcc-arm-embedded has a LIB_DEPENDS of libelf.so:devel/libelf, but this dependancy will never be built on FreeBSD-10 and FreeBSD-11 which bundles this in the base system.

Additionally, the dependancy results in a package validation failure during a synth  build of the port.
Comment 1 Jonathan Chen 2017-07-30 19:28:29 UTC
Any response to this PR?
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2017-08-06 09:18:54 UTC
Comment on attachment 184411 [details]
removes libelf dependancy

Approved by: portmgr (maintainer timeout, 2 weeks)

Pending committer review/QA
Comment 3 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2017-08-08 14:35:33 UTC
Hello Jonathan,

it currently fails the QA stage with the following errors:
====> Running Q/A tests (stage-qa)
Warning: 'gcc-arm-embedded-6-2017-q2-update/lib/libcc1.so.0' is not stripped consider trying INSTALL_TARGET=install-strip or using ${STRIP_CMD}
Error: /usr/local/gcc-arm-embedded-6-2017-q2-update/bin/arm-none-eabi-gdb-py is linked to /usr/local/lib/libintl.so.8 from devel/gettext-runtime but it is not declared as a dependency
Warning: you need USES+=gettext-runtime
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: %%PORTDOCS%%%%DOCSDIR%%/man/man1/%%TARGET%%-gcov-dump.1.gz
Error: Orphaned: %%PORTDOCS%%%%DOCSDIR%%/man/man1/%%TARGET%%-gcov-tool.1.gz
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.
*** Error code 1

Stop.
make: stopped in /usr/ports/devel/gcc-arm-embedded

Can you please have a look at it?

It is very helpful to add DEVELOPER=yes to /etc/make.conf because than every qa is executed. Otherwise try make stage-qa and make check-plist to detect this kind of issues.
Comment 4 Jonathan Chen 2017-08-08 23:21:33 UTC
Created attachment 185176 [details]
Updated patch for qa

I've:
- added the missing plist entries
- updated the USES
- corrected minor spelling mistake
Comment 5 commit-hook freebsd_committer freebsd_triage 2017-08-09 15:36:44 UTC
A commit references this bug:

Author: tz
Date: Wed Aug  9 15:36:28 UTC 2017
New revision: 447629
URL: https://svnweb.freebsd.org/changeset/ports/447629

Log:
  devel/gcc-arm-embedded: Remove unnecessary LIB_DEPENDS on libelf

  Also:
  - added missing plist entries
  - add missing gettext-runtime to USES
  - corrected minor spelling mistake

  PR:           220782
  Submitted by: Jonathan Chen <jonc@chen.org.nz>
  Approved by:  portmgr (maintainer timeout, 2 weeks)
  MFH:          2017Q3

Changes:
  head/devel/gcc-arm-embedded/Makefile
  head/devel/gcc-arm-embedded/pkg-plist
Comment 6 commit-hook freebsd_committer freebsd_triage 2017-08-17 14:41:49 UTC
A commit references this bug:

Author: tz
Date: Thu Aug 17 14:40:43 UTC 2017
New revision: 448107
URL: https://svnweb.freebsd.org/changeset/ports/448107

Log:
  MFH: r447629

  devel/gcc-arm-embedded: Remove unnecessary LIB_DEPENDS on libelf

  Also:
  - added missing plist entries
  - add missing gettext-runtime to USES
  - corrected minor spelling mistake

  PR:           220782
  Submitted by: Jonathan Chen <jonc@chen.org.nz>
  Approved by:  portmgr (maintainer timeout, 2 weeks)

  Approved by:	ports-secteam (feld)

Changes:
_U  branches/2017Q3/
  branches/2017Q3/devel/gcc-arm-embedded/Makefile
  branches/2017Q3/devel/gcc-arm-embedded/pkg-plist
Comment 7 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2017-08-17 14:45:39 UTC
Now also in quarterly, thanks! :)