[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761 --- Comment #8 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). No need to request rawhide, it's automatic. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761 MartinKG mgans...@alice.de changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE Last Closed||2015-02-03 09:11:35 --- Comment #9 from MartinKG mgans...@alice.de --- package has been built successfully on fc20, fc21 and rawhide. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761 --- Comment #7 from MartinKG mgans...@alice.de --- New Package SCM Request === Package Name: vdr-weatherforecast Short Description: A VDR plugin which provides a weather forecast Owners: martinkg Branches: f20 f21 rawhide InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761 MartinKG mgans...@alice.de changed: What|Removed |Added Flags||fedora-cvs? --- Comment #6 from MartinKG mgans...@alice.de --- (In reply to Golo Fuchert from comment #5) Martin, the package is already in a very good shape. There is only one thing that has to be changed before including it in the repos: Surely you are aware of the latest changes to the Packaging Guidelines that the license files should not be included as %doc any longer, but as %license (see https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/ LicensingGuidelines#License_Text). Please change that before pushing the package to the repos. done Additionally, but that is only a matter of taste, I am not so happy with the frequent explicit mentioning of the package name in the spec file due to the mismatch of the package name and the source name. Maybe you want to consider introducing additional macro names. done @Golo Thanks for the review. Spec URL: https://www.dropbox.com/s/a5r0ymwnafz2xw6/vdr-weatherforecast.spec?dl=0 SRPM URL: https://www.dropbox.com/s/6i3b0w5q8byz1sw/vdr-weatherforecast-0.0.2-2.fc21.src.rpm?dl=0 %changelog * Mon Feb 02 2015 Martin Gansser marti...@fedoraproject.org - 0.0.2-2 - Mark license files as %%license where available - Defined global macro pname for program name -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761
Golo Fuchert packa...@golotop.de changed:
What|Removed |Added
Flags||fedora-review+
--- Comment #5 from Golo Fuchert packa...@golotop.de ---
Martin, the package is already in a very good shape. There is only one thing
that has to be changed before including it in the repos:
Surely you are aware of the latest changes to the Packaging Guidelines that the
license files should not be included as %doc any longer, but as %license (see
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/LicensingGuidelines#License_Text).
Please change that before pushing the package to the repos.
Additionally, but that is only a matter of taste, I am not so happy with the
frequent explicit mentioning of the package name in the spec file due to the
mismatch of the package name and the source name. Maybe you want to consider
introducing additional macro names.
Here is the official review now:
[+] No errors reported by rpmlint:
rpmlint vdr-weatherforecast-0.0.2-1.fc21.x86_64.rpm
vdr-weatherforecast.x86_64: W: spelling-error %description -l en_US io - oi,
Io, ii
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
rpmlint vdr-weatherforecast-0.0.2-1.fc21.src.rpm
vdr-weatherforecast.src: W: spelling-error %description -l en_US io - oi, Io,
ii
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
[+] package name follows the naming guidelines.
[+] spec file name matches the base package %{name}.
[+] package meets the packaging guidelines.
[+] package is licensced with a Fedora approved license (GPLv2+)
[+] Licencse field is correct.
[!] the package contains a license file, but this is not included using
%license
[+] spec file is written in American English.
[+] spec file is legible
[+] Packaged sources agree with upstream sources.
b7afd15376303c1ad7940d09cdf81b8a7642e145bf913e35609f9e5be1bb7f2c
vdr-plugin-weatherforecast-0.0.2.tar.bz2-upstream
b7afd15376303c1ad7940d09cdf81b8a7642e145bf913e35609f9e5be1bb7f2c
vdr-plugin-weatherforecast-0.0.2.tar.bz2-packaged
[+] successfully compiles and builds on at least one primary architecture
(tested using mock)
[+] no need for an ExcludeArch (ARM not tested)
[+] all build dependencies listed in the BuildRequires
[+] locales are handled properly using %find_lang
[+] no need to run ldconfig (no shared libraries in the linker's default paths)
[+] no copies of system libraries
[+] package not relocatable
[+] all installed file owend by the package (some directories are owned by the
vdr package)
[+] no file listed more than once in the %files section
[+] file permissions are set properly
[+] consistent use of macros (may be improved concerning the package name,
though)
[+] no need for a -doc subpackage
[+] reasonable use of %doc
[+] no need for a -static or -devel subpackage
[+] package does not contain .la libtool files
[+] no gui application
[+] no files packaged already owned by other packages
[+] all filenames valid UTF-8
# Package Approved #
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761 Golo Fuchert packa...@golotop.de changed: What|Removed |Added Assignee|nob...@fedoraproject.org|packa...@golotop.de -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761 Golo Fuchert packa...@golotop.de changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #4 from Golo Fuchert packa...@golotop.de --- You are right. I don't know if I like this but it seems to be the convention for Fedora indeed. I can take this review, I guess I can finish it this weekend. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761
--- Comment #3 from MartinKG mgans...@alice.de ---
(In reply to Golo Fuchert from comment #2)
Is there any particular reason why the package is not called
vdr-plugin-weatherforecast? That would allow to use the %{name} macro in a
more consistent way.
i don't know if it is a historically reason
I do not know if it is a historical reason, but the Fedora package name
contains no plugin.
'yum search vdr-' gives you a list of the vdr packages in Fedora.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761
Golo Fuchert packa...@golotop.de changed:
What|Removed |Added
CC||packa...@golotop.de
--- Comment #2 from Golo Fuchert packa...@golotop.de ---
Is there any particular reason why the package is not called
vdr-plugin-weatherforecast? That would allow to use the %{name} macro in a more
consistent way.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182761] Review Request: vdr-weatherforecast - A VDR plugin which provides a weather forecast
https://bugzilla.redhat.com/show_bug.cgi?id=1182761 MartinKG mgans...@alice.de changed: What|Removed |Added CC||pikachu.2...@gmail.com --- Comment #1 from MartinKG mgans...@alice.de --- Spec URL: https://martinkg.fedorapeople.org/Review/SPECS/vdr-weatherforecast.spec SRPM URL: https://martinkg.fedorapeople.org/Review/SRPMS/vdr-weatherforecast-0.0.2-1.fc21.src.rpm Description: WeatherForecast provides a weather forecast based onforecast.io data. Sat Jan 17 2015 Martin Gansser marti...@fedoraproject.org - 0.0.2-1 - Update to 0.0.2-1 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
