firecat53 / urlscan

Mutt and terminal url selector (similar to urlview)
GNU General Public License v2.0
214 stars 38 forks source link

Please add non interactive interface #8

Closed ghost closed 9 years ago

ghost commented 9 years ago

Forwarded from: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=513548


--- Please enter the report below this line. ---

why not work this?

cat message | urlscan | cat > urllist

In message: Hello, visit my site: http://site.org/user http://google.com/

in urllist after: http://site.org/user http://google.com/


It's not an absolutely required feature but still useful.

firecat53 commented 9 years ago

Available for testing in the 'argparse' branch

ghost commented 9 years ago

After some tests, it works fine. Well done.

Concerning the argparse switch, I found a little trick to avoid the interface change ('-' not more needed to specify stdin). I'm doing some tests on this and I will submit a PR.

ghost commented 9 years ago

PR done #15

firecat53 commented 9 years ago

Fixed with 6be713e. I thought I'd tried that trick to remove the '-' and it didn't work at the time...must have been a competing issue. Thanks!