DeCarabas / fwd

MIT License
6 stars 2 forks source link

Include url in all Browse error conditions #5

Closed quodlibetor closed 8 months ago

quodlibetor commented 1 year ago

This makes it safer to use fwd-browse as your BROWSER. Even if you haven't started a fwd session at least you'll get the url the program you were using wanted to open.

$ cargo run -q --bin fwd-browse -- http://zombo.com
fwd-browse: Error connecting to socket (is fwd actually connected here?)
Start a fwd session to open: http://zombo.com
DeCarabas commented 1 year ago

Why not just print out the URL in browse_url in browse/mod.rs?

quodlibetor commented 1 year ago

Because I didn't actually look at the surrounding code. Updated to use that module.