SourceForge.net
2010-09-01 08:33:00 UTC
Patches item #2991548, was opened at 2010-04-23 21:50
Message generated for change (Comment added) made by knowledgejunkie
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424137&aid=2991548&group_id=39046
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Private: No
Submitted By: Chris Butler (crispygoth)
Initial Comment:
Just reviewing the patches I have in the Debian xmltv package, I noticed this one which I've forgotten to forward on.
Debian policy states that everything we install needs a manpage, so as part of the Debian package build we generate the man pages with pod2man. Because of this, we also need to make sure that the NAME section of the POD fits the format required by man ("module/app name - description"), so that tools such as whatis and apropos function correctly.
The attached patch fixes the NAME section in a few PODs to fit this format. There are also a few files where the description starts with lower-case, whereas most of them are capitalised, so this patch also fixes those.
I've also fixed the POD for xmltv/choose/tv_pick/tv_pick_cgi as it referred to the obsolete name for the _uk grabber.
----------------------------------------------------------------------
Message:
Closing as patches were applied by Chris in April 2010.
----------------------------------------------------------------------
Comment By: Robert Eden (rmeden)
Date: 2010-04-23 22:26
Message:
Chris, you should have commit access. I say go ahead and commit them and
make a devel list post to make sure the devs get a heads up on what was
modified.
I don't think you need to use the patch tracker for it.
Robert
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424137&aid=2991548&group_id=39046
Message generated for change (Comment added) made by knowledgejunkie
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424137&aid=2991548&group_id=39046
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Closed
Resolution: Fixed
Priority: 1Resolution: Fixed
Private: No
Submitted By: Chris Butler (crispygoth)
Assigned to: Chris Butler (crispygoth)
Summary: Fixes for the NAME sections of the PODsInitial Comment:
Just reviewing the patches I have in the Debian xmltv package, I noticed this one which I've forgotten to forward on.
Debian policy states that everything we install needs a manpage, so as part of the Debian package build we generate the man pages with pod2man. Because of this, we also need to make sure that the NAME section of the POD fits the format required by man ("module/app name - description"), so that tools such as whatis and apropos function correctly.
The attached patch fixes the NAME section in a few PODs to fit this format. There are also a few files where the description starts with lower-case, whereas most of them are capitalised, so this patch also fixes those.
I've also fixed the POD for xmltv/choose/tv_pick/tv_pick_cgi as it referred to the obsolete name for the _uk grabber.
----------------------------------------------------------------------
Comment By: Nick Morrott (knowledgejunkie)
Date: 2010-09-01 09:33Message:
Closing as patches were applied by Chris in April 2010.
----------------------------------------------------------------------
Comment By: Robert Eden (rmeden)
Date: 2010-04-23 22:26
Message:
Chris, you should have commit access. I say go ahead and commit them and
make a devel list post to make sure the devs get a heads up on what was
modified.
I don't think you need to use the patch tracker for it.
Robert
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=424137&aid=2991548&group_id=39046