Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: jsMath-fonts - A collection of Math symbol fonts
https://bugzilla.redhat.com/show_bug.cgi?id=531040
Summary: Review Request: jsMath-fonts - A collection of Math symbol fonts Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: rdieter@math.unl.edu QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://rdieter.fedorapeople.org/rpms/jsMath-fonts/jsMath-fonts.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/jsMath-fonts/jsMath-fonts-0.0-1.fc12.sr... Description: A collection of Math symbol fonts.
Usable by wxMaxima, see bug #530915
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=531040
Nicolas Mailhot nicolas.mailhot@laposte.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |nicolas.mailhot@laposte.net Flag| |fedora-review?, | |needinfo?(rdieter@math.unl. | |edu)
--- Comment #1 from Nicolas Mailhot nicolas.mailhot@laposte.net 2009-11-03 16:04:25 EDT --- 1. Please use a lowercase package name as per Fedora font packaging guidelines http://fedoraproject.org/wiki/Packaging:FontsPolicy#Naming
2. Please use the font packaging templates in fontpackages-devel as per Fedora font packaging guidelines http://fedoraproject.org/wiki/Packaging:FontsPolicy#Technical_implementation
(otherwise no font metadata will be generated at build time; note that it won't work on mixed case packages)
3. Please check what repo-font-audit says about your package,
a. install fontpackages-tools from http://koji.fedoraproject.org/koji/buildinfo?buildID=139320 b. put your rpms in a dir c. run createrepo on it d. run repo-font-audit test file://absolute-path-to-the-dir
it will catch this kind of mistake (also identify other problems that need to be relayed upstream)
4. the font splitup and naming seems a giant pre-unicode pre-wws mess. the different files are probably all parts of the same font family, except they're not properly named and their symbols not properly mapped at the right unicode points (granted, some math codepoints are only being standardised today thanks to the nice work of the stix project). For this reason, it's probably better to suff all the files in the same package, and not create a subpackage for each of them (you gotta love when upstreams decide to create their own non-standard conventions to confuse everyone else)
5. a fontconfig file would be nice, if only to document the font should be put at a very low priority in fontconfig stacks (but since fontconfig relies on font naming, and this font naming is weird, that may not be sufficient)
6. Please check with spot this font is not on his TEX blacklist
7. When there is no obvious version in a font set we usually use the timestamp of the most recent font file in the archive upstream released as version
8. Other font packages use User Interface/X as group (which is arguably not ideal, but not worth creating inconsistencies with other packages)
9. If you wanted to package this for non-fedora distros such as epel, it'd probably be more effective to push fontpackages there than add conditionals in specs. Pushing fontpackages is not difficult you only need to : 1. put back in the spec templates the warts that could be removed in Fedora thanks to recent rpm enhancements 2. check the yum-utils is recent enough to run repo-font-audit (else remove this part) 3. check the fontconfig is recent enough to accept split config files in /etc/fonts/conf.d (else remove the fontconfig templates)
fontpackages has few deps, only the templates included in it depend on particular enhancements and need "porting"
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=531040
Rex Dieter rdieter@math.unl.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(rdieter@math.unl. | |edu) |
--- Comment #2 from Rex Dieter rdieter@math.unl.edu 2009-11-09 14:55:23 EDT --- Oh boy, ok, here's a start anyway,
%changelog * Mon Nov 09 2009 Rex Dieter rdieter@fedoraproject.org 20090708-1 - lower case pkg name - Version: 20090708 (time stamp of newest included font) - use fontpackages-devel - drop subpkg baggage - Group: User Interface/X
SPEC: http://rdieter.fedorapeople.org/rpms/jsMath-fonts/jsmath-fonts.spec SRPM: http://rdieter.fedorapeople.org/rpms/jsMath-fonts/jsmath-fonts-20090708-1.fc...
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=531040
Rex Dieter rdieter@math.unl.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: |Review Request: |jsMath-fonts - A collection |jsmath-fonts - A collection |of Math symbol fonts |of Math symbol fonts
--- Comment #3 from Rex Dieter rdieter@math.unl.edu 2009-11-09 15:04:47 EDT --- Adjusting summary.
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=531040
Nicolas Mailhot nicolas.mailhot@laposte.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(rdieter@math.unl. | |edu)
--- Comment #4 from Nicolas Mailhot nicolas.mailhot@laposte.net 2009-11-13 06:27:55 EDT --- /me cries for all the ifdefing
At this point you should use the simple font template. The multi style of packaging is only useful if you produce more than one font subpackage
That's probably the only thing needed to bring it to "good enough" level of packaging. Though you will receive regularly packaging alerts because of the non-standard font naming (see attached file, the only pure packager alert is test #3 and it's a consequence of this font set state ; the rest needs relaying upstream)
I hope STIX finally publishes a non-beta version so we can tell TeXies to drop this kind of gross legacy font hack
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=531040
--- Comment #5 from Nicolas Mailhot nicolas.mailhot@laposte.net 2009-11-13 06:29:36 EDT --- Created an attachment (id=369405) --> (https://bugzilla.redhat.com/attachment.cgi?id=369405) repo-font-audit report for this package
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=531040
Rex Dieter rdieter@math.unl.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(rdieter@math.unl. | |edu) |
--- Comment #6 from Rex Dieter rdieter@math.unl.edu 2009-12-23 16:18:19 EDT --- SPEC: http://rdieter.fedorapeople.org/rpms/jsmath-fonts/jsmath-fonts.spec SRPM: http://rdieter.fedorapeople.org/rpms/jsMath-fonts/jsmath-fonts-20090708-2.fc...
%changelog * Wed Dec 23 2009 Rex Dieter rdieter@fedoraproject.org 20090708-2 - use simple template
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=531040
Nicolas Mailhot nicolas.mailhot@laposte.net changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nicolas.mailhot@laposte.net |rdieter@math.unl.edu Flag|fedora-review? |fedora-review+
--- Comment #7 from Nicolas Mailhot nicolas.mailhot@laposte.net 2010-02-22 13:21:47 EST --- Thanks
I freely admit I fail to be excited about this package, but there's probably little that can be done given the context
䷷䷷䷷ APPROVED ䷷䷷䷷
⇒ reassiging
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=531040
Rex Dieter rdieter@math.unl.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #8 from Rex Dieter rdieter@math.unl.edu 2010-02-23 14:50:06 EST --- New Package CVS Request ======================= Package Name: jsmath-fonts Short Description: A collection of Math symbol fonts Owners: rdieter Branches: EL-5 F-11 F-12 InitialCC:
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=531040
--- Comment #9 from Jason Tibbitts tibbs@math.uh.edu 2010-02-23 14:58:24 EST --- This review ticket is assigned to the person submitting it, which points to potential weirdness that I'd like to figure out before doing CVS. What's going on here?
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=531040
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|rdieter@math.unl.edu |nicolas.mailhot@laposte.net
--- Comment #10 from Jason Tibbitts tibbs@math.uh.edu 2010-02-23 15:08:46 EST --- OK, checking the history (the CVS processing tool doesn't have a way to show that) I see that for whatever reason Nicholas reassigned this to Rex. Please don't do that. I've assigned it back and will take care of CVS now.
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=531040
--- Comment #11 from Jason Tibbitts tibbs@math.uh.edu 2010-02-23 15:10:30 EST --- CVS done (by process-cvs-requests.py).
I added an F-13 branch as well.
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=531040
--- Comment #12 from Fedora Update System updates@fedoraproject.org 2010-03-21 12:39:08 EDT --- jsmath-fonts-20090708-2.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/jsmath-fonts-20090708-2.fc13
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=531040
--- Comment #13 from Fedora Update System updates@fedoraproject.org 2010-03-21 12:39:44 EDT --- jsmath-fonts-20090708-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/jsmath-fonts-20090708-2.fc12
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=531040
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |ON_QA
--- Comment #14 from Fedora Update System updates@fedoraproject.org 2010-03-23 19:34:15 EDT --- jsmath-fonts-20090708-2.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update jsmath-fonts'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/jsmath-fonts-20090708-2.fc12
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=531040
--- Comment #15 from Fedora Update System updates@fedoraproject.org 2010-03-23 19:37:52 EDT --- jsmath-fonts-20090708-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update jsmath-fonts'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/jsmath-fonts-20090708-2.fc13
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=531040
--- Comment #16 from Fedora Update System updates@fedoraproject.org 2010-03-29 22:13:47 EDT --- jsmath-fonts-20090708-2.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
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=531040
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |jsmath-fonts-20090708-2.fc1 | |2 Resolution| |ERRATA
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=531040
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|jsmath-fonts-20090708-2.fc1 |jsmath-fonts-20090708-2.fc1 |2 |3
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=531040
--- Comment #17 from Fedora Update System updates@fedoraproject.org 2010-04-09 00:26:37 EDT --- jsmath-fonts-20090708-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
package-review@lists.fedoraproject.org