Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

lesspipe 1.72 #8642

Closed
wants to merge 1 commit into from
Closed

lesspipe 1.72 #8642

wants to merge 1 commit into from

Conversation

msabramo
Copy link
Contributor

@Sharpie Sharpie closed this in b7a43ff Nov 17, 2011
@Sharpie
Copy link
Contributor

Sharpie commented Nov 17, 2011

Thanks.

@tcurdt
Copy link
Contributor

tcurdt commented Jan 3, 2012

Hm. I get an error with deb archives now (on 10.6 at least - haven't tried on 10.7 yet)

==> use Deb_file:contained_file to view a file in the Deb

sed: illegal option -- r
usage: sed script [-Ealn] [-i extension] [file ...]
       sed [-Ealn] [-i extension] [-e script] ... [-f script_file] ... [file ...]

drwxr-xr-x  0 loader loader      0 Jan  4 00:23 ./usr/
drwxr-xr-x  0 loader loader      0 Jan  4 00:23 ./usr/share/
drwxr-xr-x  0 loader loader      0 Jan  4 00:23 ./usr/share/jdeb/
drwxr-xr-x  0 loader loader      0 Jan  4 00:23 ./usr/share/jdeb/lib/
-rw-r--r--  0 loader loader   2644 Jan  4 00:23 ./usr/share/jdeb/lib/jdeb-example-1.0.jar
drwxr-xr-x  0 loader loader      0 Jan  4 00:23 ./etc/
drwxr-xr-x  0 loader loader      0 Jan  4 00:23 ./etc/init.d/
-rw-r--r--  0 loader loader      0 Jan  4 00:23 ./etc/init.d/myservice
/Users/tcurdt/Projects/jdeb/src/examples/maven/target/jdeb-example_1.0.deb (END)

Can someone else confirm? This used to work fine.

@msabramo
Copy link
Contributor Author

@tcurdt I'm not getting that error, but I looked in the source and it looks like that path with sed would only be triggered if you don't have dpkg. It then tries to call sed and the BSD/OS X version doesn't have a -r option (GNU sed does).

So the formula could probably be improved to make it use gsed (in the coreutils brew if it's installed) instead of sed. But you might as well install the dpkg brew.

Sure enough, I just tried removing the dpkg brew and I was able to reproduce your error. Installing it makes it go away.

@tcurdt
Copy link
Contributor

tcurdt commented Jan 19, 2012

Thanks for digging into this. I've also just installed dpkg.

Well, I would call it bug in lesspipe as it makes assumptions that it is gsed vs sed. Not sure how to handle this best from the brew point of view.

Sharpie pushed a commit to Sharpie/homebrew that referenced this pull request Sep 12, 2012
Closes Homebrew#8642.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>
snakeyroc3 pushed a commit to snakeyroc3/homebrew that referenced this pull request Dec 17, 2012
Closes Homebrew#8642.

Signed-off-by: Charlie Sharpsteen <source@sharpsteen.net>
@Homebrew Homebrew locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants