D3650: serve: add an option to open in the default browser

2020-01-24 Thread baymax (Baymax, Your Personal Patch-care Companion)
This revision now requires changes to proceed. baymax added a comment. baymax requested changes to this revision. There seems to have been no activities on this Diff for the past 3 Months. By policy, we are automatically moving it out of the `need-review` state. Please, move it back t

D3650: serve: add an option to open in the default browser

2018-06-16 Thread yuja (Yuya Nishihara)
yuja added a comment. (Resend because I forgot to remove the "someone wrote:" line again, sorry.) > For laziness, this patch could be also useful. It is also better for text shell beginners, because they may not know that they can combine commands like that (even if it is basic).

Re: D3650: serve: add an option to open in the default browser

2018-06-16 Thread Yuya Nishihara
(Resend because I forgot to remove the "someone wrote:" line again, sorry.) > For laziness, this patch could be also useful. It is also better for text > shell beginners, because they may not know that they can combine commands > like that (even if it is basic). For laziness, we can add it t

Re: D3650: serve: add an option to open in the default browser

2018-06-15 Thread Yuya Nishihara
On Fri, 15 Jun 2018 08:40:33 +, nspanti-logilab (Nicola Spanti) wrote: > For laziness, this patch could be also useful. It is also better for text > shell beginners, because they may not know that they can combine commands > like that (even if it is basic). Moreover, you can still use `hg

D3650: serve: add an option to open in the default browser

2018-06-15 Thread nspanti-logilab (Nicola Spanti)
nspanti-logilab added a comment. For laziness, this patch could be also useful. It is also better for text shell beginners, because they may not know that they can combine commands like that (even if it is basic). Moreover, you can still use `hg serve` in foreground or use it in background

D3650: serve: add an option to open in the default browser

2018-06-14 Thread pulkit (Pulkit Goyal)
pulkit added a comment. Do we need this patch any more since https://phab.mercurial-scm.org/D3649 has landed and we can do `xdg-open "$(hg serve --print-url)"`? REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D3650 To: nspanti-logilab, #hg-reviewers Cc: pulkit, a

D3650: serve: add an option to open in the default browser

2018-06-02 Thread av6 (Anton Shestakov)
av6 added a comment. https://phab.mercurial-scm.org/D3649 mentions `xdg-open "$(hg serve --print-url)"`, which shows that --print-url can nearly replace --graphical and is more versatile. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D3650 To: nspanti-logilab,

D3650: serve: add an option to open in the default browser

2018-05-31 Thread durin42 (Augie Fackler)
durin42 added a comment. I don't love the flag name, and am -0 on the feature in general. All terminal emulators I've used in the last 10 years have URL detection logic and make clicking the link trivial... REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D3650 T

D3650: serve: add an option to open in the default browser

2018-05-24 Thread nspanti-logilab (Nicola Spanti)
nspanti-logilab created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY It is inspired by a similar functionality in pydoc. REPOSITORY rHG Mercurial BRANCH default REVISION DETAIL https://phab.mercurial-scm.org/D3650 AF