Opened 11 years ago
Closed 11 years ago
#39853 closed defect (fixed)
bash-completion(-devel): some completions expect GNU sed in $PATH
Reported by: | neverpanic (Clemens Lang) | Owned by: | raimue (Rainer Müller) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.2.0 |
Keywords: | Cc: | dale+macports@…, cooljeanius (Eric Gallager) | |
Port: | bash-completion |
Description
:) clemens@cSchlepptop:[..]/Haskell/UE/KVB$ make con^Ised: illegal option -- r usage: sed script [-Ealn] [-i extension] [file ...] sed [-Ealn] [-i extension] [-e script] ... [-f script_file] ... [file ...] ^C
Apparently caused by some rules using the -r
flag to sed. egrep -R "sed\s+-\S*r"
in /opt/local/share/bash-completion/completions suggests the culprits are colormake, gmake, gnumake, make and pmake. We should add a runtime dep on gsed and patch the sed path.
Change History (5)
comment:1 Changed 11 years ago by raimue (Rainer Müller)
Status: | new → assigned |
---|
comment:4 Changed 11 years ago by neverpanic (Clemens Lang)
Any progress? I recall you were already working on this, did you just forget to close the ticket?
comment:5 Changed 11 years ago by raimue (Rainer Müller)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Upstream: https://alioth.debian.org/tracker/?func=detail&atid=413095&aid=314345&group_id=100114
Thanks for the reminder. I actually forgot to finalize the patch. The fix has been commmitted in r109658.
Note: See
TracTickets for help on using
tickets.
Thanks for the report. All make variants use the same completions, so it's only a single instance that needs to get fixed. As this only affects make I would think we could also patch the regex instead of adding the additional dependency on GNU sed. I am working on a patch.