Summary: | archivers/xarchiver fails to built with bsdgrep | ||||||
---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Piotr Smyrak <ps.ports> | ||||
Component: | Individual Port(s) | Assignee: | Danilo Egea Gondolfo <danilo> | ||||
Status: | Closed Overcome By Events | ||||||
Severity: | Affects Only Me | CC: | danfe, kevans | ||||
Priority: | --- | Flags: | bugzilla:
maintainer-feedback?
(danilo) |
||||
Version: | Latest | ||||||
Hardware: | Any | ||||||
OS: | Any | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 230332 | ||||||
Attachments: |
|
Description
Piotr Smyrak
2020-10-01 10:35:45 UTC
Created attachment 218888 [details]
configure patch
Adding a configure patch that changes the one liner used to return gettext version. Instead of using perl regexp unsupported by bsdgrep, it uses awk to print the latest column of the xgettext output.
(In reply to Piotr Smyrak from comment #1) Hi, This is actually a GNU extension to BREs rather than Perl -- I would recommend having your patch do: -xgversion="`xgettext --version | head -1 | grep -o '[0-9.]\+$'`" +xgversion="`xgettext --version | head -1 | grep -Eo '[0-9.]+$'`" This may be more palatable to upstream as it's just POSIXifying their expression. bsdgrep in -CURRENT now enables GNU extensions by default and the original incantation works again, but 12.x will be broken until 12.3. IMO, it is worth it. (In reply to Kyle Evans from comment #2) Nice! Thanks for looking into it. I will try to upstream the change then. The change suggested by Kyle has been upstreamed in https://github.com/ib/xarchiver/pull/106 This bug has been fixed by bsdgrep improvements and an upstream fix added before 0.5.4.16 of xarchiver has been released. Closing per comment #5. |