https://bugzilla.redhat.com/show_bug.cgi?id=1208835
Bug ID: 1208835 Summary: Review Request: gdouros-akkadian-fonts - A font for Sumero-Akkadian cuneiform Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: alex.ploumistos@gmail.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: https://alexpl.fedorapeople.org/packages/fonts/gdouros/gdouros-akkadian-font...
SRPM URL: https://alexpl.fedorapeople.org/packages/fonts/gdouros/gdouros-akkadian-font...
Description: Akkadian covers the following scripts and symbols supported by The Unicode Standard 5.2: Basic Latin, Greek and Coptic, some Punctuation and other Symbols, Cuneiform, Cuneiform Numbers and Punctuation.
It was created by George Douros.
Fedora Account System Username: alexpl
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=9402789
The font is already in fedora, but the current maintainer has been non-responsive for quite some time, so this is the last step of the relevant policy - a takeover request.
I have cleaned up the spec files from deprecated commands, added AppStream metadata and I have included a sample file provided by upstream in a separate doc package. Following a brief discussion on devel ML and after having contacted upstream for clarifications, I have decided not to include the hinted font in the source package.
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
--- Comment #1 from Alexander Ploumistos alex.ploumistos@gmail.com --- Updated files, without doc sub-package:
https://alexpl.fedorapeople.org/packages/fonts/gdouros/gdouros-akkadian-font...
https://alexpl.fedorapeople.org/packages/fonts/gdouros/gdouros-akkadian-font...
I also noticed that upstream has forgotten to update the version number internally, so in a font viewer it appears that the font is in version 7.13. I intend to inform him about that, along with a few other issues.
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |panemade@gmail.com See Also| |https://bugzilla.redhat.com | |/show_bug.cgi?id=532231 Assignee|nobody@fedoraproject.org |panemade@gmail.com Summary|Review Request: |Re-Review Request: |gdouros-akkadian-fonts - A |gdouros-akkadian-fonts - A |font for Sumero-Akkadian |font for Sumero-Akkadian |cuneiform |cuneiform Flags| |fedora-review?
--- Comment #2 from Parag AN(पराग) panemade@gmail.com --- Review:
+ mock build is successful for F23 (x86_64)
+ rpmlint on generated rpms gave output 2 packages and 0 specfiles checked; 0 errors, 0 warnings.
+ Source verified with upstream as (sha256sum) srpm Source0: fd00741c13e0e822a97a6451b1fe406fcdab09f9e4bd77fe6a3eede3d8f5ea61 upstream Source0 : fd00741c13e0e822a97a6451b1fe406fcdab09f9e4bd77fe6a3eede3d8f5ea61 srpm Source1 : 15115616d2e04f36ac53b7d45e87dbfc77a1d810f6b4659dc70127935bae7e59 upstream Source1 : 15115616d2e04f36ac53b7d45e87dbfc77a1d810f6b4659dc70127935bae7e59
+ License is "Public Domain" as per given on upstream website "Fonts and documents in this site are not pieces of property or merchandise items; they carry no trademark, copyright, license or other market tags; they are free for any use". So no license text is included in font file or any text file
+ follows fonts packaging guidelines.
+ fontconfig file is present at priority 65.
+ Appdata metainfo file is present.
Suggestions: 1) I propose to follow https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Pre-Release_packag... with following changes
%global checkout 20150430
Release: 0.2.%{checkout}%{?dist}
also add changelog that uses 7.13-0.2.20150430
The reason why I am not considering 7.15 as version is that it can be mistake by upstream to write the archive as 7.15.
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
--- Comment #3 from Alexander Ploumistos alex.ploumistos@gmail.com --- (In reply to Parag AN(पराग) from comment #2)
- I propose to follow
https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Pre- Release_packages with following changes
%global checkout 20150430
Release: 0.2.%{checkout}%{?dist}
a) Why is the leading zero necessary? b) Is it OK to use the date on which I downloaded the font as opposed to the date on which it was updated?
The reason why I am not considering 7.15 as version is that it can be mistake by upstream to write the archive as 7.15.
So I should pay attention only to the version number declared inside the font and not the one on his website?
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
--- Comment #4 from Alexander Ploumistos alex.ploumistos@gmail.com --- (In reply to Alexander Ploumistos from comment #3)
a) Why is the leading zero necessary?
Never mind, I figured it out, I was looking at the wrong versioning example.
Here are the updated files:
https://alexpl.fedorapeople.org/packages/fonts/gdouros/gdouros-akkadian-font...
https://alexpl.fedorapeople.org/packages/fonts/gdouros/gdouros-akkadian-font...
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
--- Comment #5 from Parag AN(पराग) panemade@gmail.com --- Many times upstream developers of fonts do not update meta information like version number, License url, Copyright etc. in fonts also not release versioned tarball. So best way to identify that upstream silently changed the font file but not information we should add maximum identification to the packaged font file.
If you still think upstream is right in writing 7.15 then good to clarify with him again but if he says its 7.15 but forgot to update then our package naming is better 7.13-0.2.20150430 that will say that packaged font archive is downloaded on 20150430 date and inside font information say 7.13 as also archive tarball is not versioned and there is no information in other supporting text files in that tarball.
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
--- Comment #6 from Alexander Ploumistos alex.ploumistos@gmail.com --- (In reply to Parag AN(पराग) from comment #5)
Many times upstream developers of fonts do not update meta information like version number, License url, Copyright etc. in fonts also not release versioned tarball. So best way to identify that upstream silently changed the font file but not information we should add maximum identification to the packaged font file.
During these past few months, I noticed that upstream had changed the fonts, without altering the date or incrementing the version, so I wholeheartedly agree with you and as you can see, I have changed the naming on all the gdouros- font packages.
If you still think upstream is right in writing 7.15 then good to clarify with him again but if he says its 7.15 but forgot to update then our package naming is better 7.13-0.2.20150430 that will say that packaged font archive is downloaded on 20150430 date and inside font information say 7.13 as also archive tarball is not versioned and there is no information in other supporting text files in that tarball.
I have sent him a message, if and when he fixes the mismatch I will update the package. I am leaving it as 7.13-0.2.20150430 for now.
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
--- Comment #7 from Parag AN(पराग) panemade@gmail.com --- Thanks. Btw, Since last few days fedora-review and dnf is giving me problem which is taking some time to review your packages. Just installed new mock package from f22 testing repo and still some issues with fedora-review.
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+
--- Comment #8 from Parag AN(पराग) panemade@gmail.com --- The updated srpm in comment#4 looks good now.
Package APPROVED.
You want to submit here package change request to own this package. See https://fedoraproject.org/wiki/Package_SCM_admin_requests#Package_Change_Req...
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
--- Comment #9 from Alexander Ploumistos alex.ploumistos@gmail.com --- Package Change Request ====================== Package Name: gdouros-akkadian-fonts Branches: f20 f21 f22 master Owners: alexpl InitialCC: fonts-sig
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
Alexander Ploumistos alex.ploumistos@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-cvs?
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
--- Comment #10 from Alexander Ploumistos alex.ploumistos@gmail.com --- Turns out the right version was 7.13 all along. He set it as 7.15 on the web page and in the other files by mistake.
No need to change anything, your hunch was spot on!
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
--- Comment #11 from Jon Ciesla limburgher@gmail.com --- Complete.
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
Jon Ciesla limburgher@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-cvs? |
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
--- Comment #12 from Fedora Update System updates@fedoraproject.org --- gdouros-akkadian-fonts-7.13-0.2.20150430.fc22 has been submitted as an update for Fedora 22. https://admin.fedoraproject.org/updates/gdouros-akkadian-fonts-7.13-0.2.2015...
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #13 from Fedora Update System updates@fedoraproject.org --- Package gdouros-akkadian-fonts-7.13-0.2.20150430.fc22: * should fix your issue, * was pushed to the Fedora 22 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing gdouros-akkadian-fonts-7.13-0.2.20150430.fc22' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2015-7543/gdouros-akkadian-fo... then log in and leave karma (feedback).
https://bugzilla.redhat.com/show_bug.cgi?id=1208835
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |gdouros-akkadian-fonts-7.13 | |-0.2.20150430.fc22 Resolution|--- |ERRATA Last Closed| |2015-05-11 15:07:18
--- Comment #14 from Fedora Update System updates@fedoraproject.org --- gdouros-akkadian-fonts-7.13-0.2.20150430.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.
package-review@lists.fedoraproject.org