After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 790543 - Error in the help's profiling page
Error in the help's profiling page
Status: RESOLVED FIXED
Product: sysprof
Classification: Other
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Sysprof maintainer(s)
Sysprof maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2017-11-18 17:53 UTC by Adrien Plazas
Modified: 2017-11-22 01:08 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
help: fix cross-reference link title (1.15 KB, patch)
2017-11-22 01:08 UTC, Christian Hergert
committed Details | Review

Description Adrien Plazas 2017-11-18 17:53:31 UTC
https://git.gnome.org/browse/sysprof/tree/help/C/profiling.page#n15

The link says "profiling an existing process" but it links to the "Profile a new process" section.
Comment 1 Christian Hergert 2017-11-19 23:39:10 UTC
What version of yelp do you have? It works fine here?
Comment 2 Adrien Plazas 2017-11-21 13:19:03 UTC
I have yelp 3.26.0 but I doubt this is relevant as the problem is that the text of the link and the section it points to don't match.

I would have happily submitted a patch but I don't know which of the text and the link is correct, which one should it be?
- <link href="profiling#new-process-profiling">profiling a new process</link>
- <link href="profiling#existing-process-profiling">profiling an existing process</link>
Comment 3 Christian Hergert 2017-11-22 01:08:04 UTC
Created attachment 364159 [details] [review]
help: fix cross-reference link title

This was linking to the right section, but had the incorrect
link text.
Comment 4 Christian Hergert 2017-11-22 01:08:51 UTC
Okay, I see what you mean now. I thought you meant the cross-referencing links
generated by Yelp where incorrect.

Attachment 364159 [details] pushed as a07ee1e - help: fix cross-reference link title