[Bug c/87488] hyperlink filenames in diagnostics

2020-02-20 Thread msl0000023508 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #24 from WHR --- Looks like commit r10-6643-g458c8d6459c4005fc9886b6e25d168a6535ac415 is a well fix, and my terminal is no longer nosiy running GCC 10. I have a suggestion, consider check for a few more 'TERM' types that would certai

[Bug c/87488] hyperlink filenames in diagnostics

2020-02-14 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #23 from CVS Commits --- The master branch has been updated by Bernd Edlinger : https://gcc.gnu.org/g:458c8d6459c4005fc9886b6e25d168a6535ac415 commit r10-6643-g458c8d6459c4005fc9886b6e25d168a6535ac415 Author: Bernd Edlinger Date:

[Bug c/87488] hyperlink filenames in diagnostics

2019-12-10 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #22 from Jakub Jelinek --- Indeed, RHEL5 (admittedly in extended lifecycle support only now) and RHEL6 use old VTE though, dunno about the other long term support distros. Dunno what are the probabilities users will use new GCC on th

[Bug c/87488] hyperlink filenames in diagnostics

2019-12-08 Thread ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 Eric Botcazou changed: What|Removed |Added CC||ebotcazou at gcc dot gnu.org --- Comment

[Bug c/87488] hyperlink filenames in diagnostics

2019-12-07 Thread egmont at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #20 from Egmont Koblinger --- GNOME Terminal's behavior mainly depends on VTE. Hyperlinks are supported since VTE 0.50 and a corresponding GNOME Terminal 3.26. VTE 0.48.2 and 0.46.3, and newer versions within these stable series sile

[Bug c/87488] hyperlink filenames in diagnostics

2019-12-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #19 from David Malcolm --- (In reply to Jakub Jelinek from comment #18) > I came up independently with the same patch (sorry, wasn't aware a patch is > discussed in this PR, I've searched just the mailing list and nothing has > been p

[Bug c/87488] hyperlink filenames in diagnostics

2019-12-07 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #18

[Bug c/87488] hyperlink filenames in diagnostics

2019-12-07 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #17 from Jakub Jelinek --- Author: jakub Date: Sat Dec 7 08:24:14 2019 New Revision: 279073 URL: https://gcc.gnu.org/viewcvs?rev=279073&root=gcc&view=rev Log: PR c/87488 * pretty-print.c (pp_begin_url, pp_end_url, te

[Bug c/87488] hyperlink filenames in diagnostics

2019-12-05 Thread egmont at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #16 from Egmont Koblinger --- (In reply to Tobias Burnus from comment #13) > but using KDE Konsole, the warnings now have: > >warning: control reaches end of non-void function [\-Wreturn-type\] > > That is: There are spurious \

[Bug c/87488] hyperlink filenames in diagnostics

2019-12-05 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #15 from David Malcolm --- I guess the other thing to test it on is on older gnome terminals that predate the support - but I don't have one handy.

[Bug c/87488] hyperlink filenames in diagnostics

2019-12-05 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #14 from David Malcolm --- Thanks. systemd also uses '\a': https://github.com/systemd/systemd/blob/master/src/shared/pretty-print.c#L53 n = strjoin("\x1B]8;;", url, "\a", text, "\x1B]8;;\a"); I didn't see your update to this B

[Bug c/87488] hyperlink filenames in diagnostics

2019-12-05 Thread burnus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 Tobias Burnus changed: What|Removed |Added CC||burnus at gcc dot gnu.org --- Comment #1

[Bug c/87488] hyperlink filenames in diagnostics

2019-10-10 Thread egmont at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 Egmont Koblinger changed: What|Removed |Added CC||egmont at gmail dot com --- Comment #

[Bug c/87488] hyperlink filenames in diagnostics

2019-10-10 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c/87488] hyperlink filenames in diagnostics

2019-10-10 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #10 from David Malcolm --- Author: dmalcolm Date: Thu Oct 10 17:03:46 2019 New Revision: 276843 URL: https://gcc.gnu.org/viewcvs?rev=276843&root=gcc&view=rev Log: Documentation hyperlinks for [-Wname-of-option] (PR 87488) This patch

[Bug c/87488] hyperlink filenames in diagnostics

2019-10-10 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #9 from David Malcolm --- Author: dmalcolm Date: Thu Oct 10 16:57:30 2019 New Revision: 276841 URL: https://gcc.gnu.org/viewcvs?rev=276841&root=gcc&view=rev Log: pretty-print: support URL escape sequences (PR 87488) https://gist.git

[Bug c/87488] hyperlink filenames in diagnostics

2019-10-09 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 David Malcolm changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug c/87488] hyperlink filenames in diagnostics

2019-01-08 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #7 from David Malcolm --- I'm unconvinced that doing it for filenames is a good idea (based on the objections in comment #3), but I think that there could be other good uses tagging URLs into the output. For example, a static analysi

[Bug c/87488] hyperlink filenames in diagnostics

2018-10-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #6 from Manuel López-Ibáñez --- I think the utility of this is similar to colors, that is when the output is a terminal, not when it is emacs or vim or any other non-tty. In remote hosts, I think we should do whatever GNU coreutils (

[Bug c/87488] hyperlink filenames in diagnostics

2018-10-02 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #5 from Jonathan Wakely --- Vim already Does The Right Thing for a filename like file.c:20:23

[Bug c/87488] hyperlink filenames in diagnostics

2018-10-02 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #4 from David Malcolm --- Interesting. Is there a way to encode the line number in the URL? (and the column number?) Or does it just give the file?

[Bug c/87488] hyperlink filenames in diagnostics

2018-10-02 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #3 from Jonathan Wakely --- https://gist.github.com/egmontkob/eb114294efbcd5adb1944c9f3cb5feda#file-uris-and-the-hostname says there should be a hostname part, so not just file:///path but file://hostname/path If I compile on a remot

[Bug c/87488] hyperlink filenames in diagnostics

2018-10-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #2 from Manuel López-Ibáñez --- Actually, according to https://gist.github.com/egmontkob/eb114294efbcd5adb1944c9f3cb5feda#detecting-availability-of-the-feature "almost all terminal emulators ignore the OSC sequences they don't know a

[Bug c/87488] hyperlink filenames in diagnostics

2018-10-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 Manuel López-Ibáñez changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Co