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 580329 - Use show_uri() to display help window
Use show_uri() to display help window
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Vincent Legoll
Orca Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-04-26 20:02 UTC by Vincent Legoll
Modified: 2009-05-01 14:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
use show_uri() (743 bytes, patch)
2009-04-26 20:02 UTC, Vincent Legoll
committed Details | Review

Description Vincent Legoll 2009-04-26 20:02:24 UTC
As per http://live.gnome.org/GnomeGoals/RemoveGnomeOpenGnomeHelp

Replace use of gnome.help_display_with_doc_id() by gtk.show_uri()

Here is a patch that does the requested migration.
Please review. Comments appreciated.
Comment 1 Vincent Legoll 2009-04-26 20:02:50 UTC
Created attachment 133361 [details] [review]
use show_uri()
Comment 2 Willie Walker 2009-04-30 18:11:08 UTC
Many thanks!  Committed to master and gnome-2-26.
Comment 3 Vincent Legoll 2009-05-01 08:43:02 UTC
I think you should change the author filed when committing someone else's work, so that attribution is properly kept.

If I recall correctly you can do this by adding a "From:" line to the commit message metadata.

But thanks anyways to have taken care of it.
Comment 4 Willie Walker 2009-05-01 13:35:09 UTC
(In reply to comment #3)
> I think you should change the author filed when committing someone else's work,
> so that attribution is properly kept.

Crap.  Sorry about that.  With SVN, I was very religious about providing proper attribution.  I'm still getting used to the 'conveniences' provided by GIT and haven't yet learned all the hoops I need to jump through.  :-)  I make sure it ends up in the ChangeLog, though.
Comment 5 Willie Walker 2009-05-01 13:37:02 UTC
BTW, I *think* the git format-patch operation should create a patch you can attach that I can apply with git am.  If done this way, the patch provider essentially ensures proper attribution as I understand it.
Comment 6 Vincent Legoll 2009-05-01 14:30:41 UTC
Yes I think you're right about git format-patch,
I didn't use it since I didn't commit the patch
in my cloned repository yet. And I also intended
to commit it myself and create a proper changelog
at that time...

Case closed, no harm done, that was a small
patch anyways.