[Bug 682666] Review Request: DeTex - A program to remove TeX constructs from a text file
bugzilla at redhat.com
bugzilla at redhat.com
Thu Apr 28 10:15:36 UTC 2011
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=682666
--- Comment #12 from Christoph Wickert <cwickert at fedoraproject.org> 2011-04-28 06:15:35 EDT ---
(In reply to comment #11)
> (In reply to comment #10)
> > There is still a small bug in the spec: the release was not bumped and -2 is in
> > the changelog twie.
>
> That isn't a bug afaics:
>
> http://fedoraproject.org/wiki/PackagingGuidelines#Multiple_Changelog_Entries_per_Release
But we should have had two releases actually, even during a review. And if you
substitute multiple versions in a changelog entry, it should actually be
* Thu Apr 28 2011 Ankur Sinha <ankursinha AT fedoraproject DOT org> - 2.8-2
- removed patch
- renamed man page to .1 instead of .1l
- corrected compilation flags
- Review ticket #682666
- Added *comments*
Highest version/release and latest date contain all changes.
I suggest to leave it at -2 and just change the changelog.
> Before I do an SCM request, should I change the *entire* package name to DeTeX
> Mohamed?
Please don't, leave the spec and the package "detex" please. LaTeX is also
written lowercase in our package names
"While case sensitivity is not a mandatory requirement, case should only be
used where necessary."
from http://fedoraproject.org/wiki/PackageNamingGuidelines#Case_Sensitivity
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
More information about the package-review
mailing list