Summary: | x11-wm/piewm: Fix link error: "ld: error: duplicate symbol: yylineno" | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | david | ||||||
Component: | Individual Port(s) | Assignee: | Kyle Evans <kevans> | ||||||
Status: | Closed FIXED | ||||||||
Severity: | Affects Many People | CC: | dhw, emaste, kevans, ports-bugs | ||||||
Priority: | --- | Keywords: | needs-qa | ||||||
Version: | Latest | Flags: | koobs:
merge-quarterly?
|
||||||
Hardware: | Any | ||||||||
OS: | Any | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 248756 | ||||||||
Attachments: |
|
Description
david
2020-08-23 20:37:24 UTC
^Triage: - [tags] in issue Titles are deprecated - Please set the maintainer-approval attachment flag (to +) on patches for ports you maintain to signify approval Attachment -> Details -> maintainer-approval [+] Oh -- thanks. It looks as if the patch (now that you've updated things) has the "maintainer approval" flag, so I don't need to be trying to set it (but for next time, I should do it correctly), yeah? (In reply to david from comment #2) Hi David, I didn't set the maintainer-approval flag in the previous update,. so please feel free to set it (to +) on this and future attachments for ports you maintain Comment on attachment 217466 [details] Patch to x11-wm/piewm/Makefile to add the "-fcommon" flag. >Index: x11-wm/piewm/Makefile >=================================================================== >--- x11-wm/piewm/Makefile (revision 545815) >+++ x11-wm/piewm/Makefile (working copy) >@@ -14,6 +14,6 @@ > DISABLE_SIZE= yes > USE_XORG= xbitmaps xpm xmu > >-CFLAGS+= -Wno-return-type >+CFLAGS+= -Wno-return-type -fcommon > > .include <bsd.port.mk> Created attachment 217487 [details]
Better fix for x11-wm/piewm
This patch is a result from having issued "svn rm x11-wm/piewm/files/patch-gram.y"
The new patch is a result of advice from dim@; after verifying that I was able to reproduce the observed failure under head @r364650, I tested and observed that removing the offending file allows the port to build in that environment. A commit references this bug: Author: kevans Date: Thu Aug 27 14:33:29 UTC 2020 New revision: 546348 URL: https://svnweb.freebsd.org/changeset/ports/546348 Log: x11-wm/piewm: fix the build with LLVM 11 This patch de-extern'd yylineno to actually cause the build failure that popped up with the new -fno-common default that debuted in LLVM 11 and GCC 10. Simply remove the patch to fix the build. PR: 248859 MFH: 2020Q3 (-fno-common build fix) Changes: head/x11-wm/piewm/files/patch-gram.y A commit references this bug: Author: kevans Date: Thu Aug 27 14:34:54 UTC 2020 New revision: 546349 URL: https://svnweb.freebsd.org/changeset/ports/546349 Log: MFH: r546348 x11-wm/piewm: fix the build with LLVM 11 This patch de-extern'd yylineno to actually cause the build failure that popped up with the new -fno-common default that debuted in LLVM 11 and GCC 10. Simply remove the patch to fix the build. PR: 248859 Approved by: ports-secteam (implicit, -fno-common build fix) Changes: _U branches/2020Q3/ branches/2020Q3/x11-wm/piewm/files/patch-gram.y Committed, thanks! Note that you may still get another fallout e-mail or two if a new build had started prior to the commit, but I reviewed the patch and also confirmed that it was OK. Apologies for the delay. |