[Bug 457947] Review Request: oldstandard-sfd-fonts - Old Standard Fonts

bugzilla at redhat.com bugzilla at redhat.com
Wed Nov 19 22:58:25 UTC 2008


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=457947





--- Comment #21 from Martin-Gomez Pablo <pablo.martin-gomez at laposte.net>  2008-11-19 17:58:24 EDT ---
> The doc on changelog doesnt tell me which ones dist and which ones version.. :(
> so i havent changed that in this one.. (pablo please help me with this)
It's quite simple to understand, for exemple, your spec:
Version:        1.1
Release:        1%{?dist}
The number after "Version:" is the version one, it's given by upstream and you
must not change it. So put "1.0" as it's the version of the font.
The "Release:" number is a distro/package specific number, so each time you add
an entry in the changelog, you increment by one this number. So currently, it
should be "3%{?dist}".

Also your changelog format is not correct, you have to add your email address
as said in the guidelines
(https://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs).


> The warnings for the docs is still present.. How do i correct that?
You should add the code proposed by Nicolas in the review request of the
bonveno fonts in the %prep section  .

> You have many font substitution examples in the dejavu font packages. Do not
> forget to declare your font is an acceptable substitute for all the different
> Old Standard variant out there (TTF, OTF, etc)
Ok, so you have to duplicate the XML code I gave you, and just change the first
<family>Old Standard</family> to <family>Old Standard TTF</family>.
For register this font as a serif one, add the following to the fontconfig
file:
<alias>
  <family>serif</family>
  <prefer>
    <family>Old Standard SFD</family>
  </prefer>
</alias>

-- 
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 Fedora-package-review mailing list