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 681643 - Port to new documentation infrastructure
Port to new documentation infrastructure
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: documentation
3.4.x
Other All
: Normal normal
: ---
Assigned To: Maintainers of Gnome user documentation
Maintainers of Gnome user documentation
Depends on:
Blocks: 681609
 
 
Reported: 2012-08-11 09:24 UTC by Javier Jardón (IRC: jjardon)
Modified: 2012-09-25 12:18 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Use-new-documentation-infrastructure (3.21 KB, patch)
2012-08-11 09:24 UTC, Javier Jardón (IRC: jjardon)
needs-work Details | Review
Use-new-documentation-infrastructure.v2 (115.34 KB, patch)
2012-08-21 15:18 UTC, Javier Jardón (IRC: jjardon)
committed Details | Review

Description Javier Jardón (IRC: jjardon) 2012-08-11 09:24:03 UTC
Created attachment 220921 [details] [review]
Use-new-documentation-infrastructure

Use yelp-tools instead of gnome-doc-utils.

https://live.gnome.org/GnomeGoals/NewDocumentationInfrastructure
Comment 1 Philip Withnall 2012-08-20 23:49:21 UTC
Review of attachment 220921 [details] [review]:

I think the “ghelp” URIs in src/totem-object.c, help/C/totem.xml, help/C/legal.xml and their translations all need converting to “help” URIs.

::: help/Makefile.am
@@ +5,2 @@
+HELP_FILES = \
+	index.docbook

Shouldn’t this patch also rename C/totem.xml to C/index.docbook?
Comment 2 Javier Jardón (IRC: jjardon) 2012-08-21 15:18:38 UTC
Created attachment 222031 [details] [review]
Use-new-documentation-infrastructure.v2

Yeah, here a new patch. Thanks for the review
Comment 3 Javier Jardón (IRC: jjardon) 2012-08-31 04:21:04 UTC
friendly ping :)
Comment 4 Philip Withnall 2012-09-03 18:26:07 UTC
Review of attachment 222031 [details] [review]:

Looks good — please commit to master!

(Sorry for not re-reviewing it sooner; Bugzilla didn’t e-mail me about the updated patch. :-( )
Comment 5 Javier Jardón (IRC: jjardon) 2012-09-03 19:28:42 UTC
Comment on attachment 222031 [details] [review]
Use-new-documentation-infrastructure.v2

commit 59a6bd6064fe7dd047f7b61642aba73e0ba72acf
Comment 6 Javier Jardón (IRC: jjardon) 2012-09-03 19:29:11 UTC
Thanks for the review! :)
Comment 7 Jeremy Bicha 2012-09-21 02:21:22 UTC
It looks like this was applied to master but no the gnome-3-6 branch. I presume it's too late for 3.6 as this would upset translations, right?
Comment 8 Philip Withnall 2012-09-25 12:18:20 UTC
(In reply to comment #7)
> It looks like this was applied to master but no the gnome-3-6 branch. I presume
> it's too late for 3.6 as this would upset translations, right?

Yes, it was accidentally not committed to the 3.6 branch, and it’s too late to fix that now.

http://git.gnome.org/browse/totem/commit/?h=gnome-3-6&id=0d4a17ff31b4dcb9c902136ab80dd177a295dba8