[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Jon Ciesla 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 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #31 from Jon Ciesla --- Git done (by process-git-requests). -- 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 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Alec Leamas changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? --- Comment #30 from Alec Leamas --- Package Change Request == Package Name: lpf New Branches: el6 Owners: slaanesh InitialCC: slaanesh -- 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 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #29 from Fedora Update System --- lpf-0-10.d18db6d.fc19 has been pushed to the Fedora 19 stable repository. If problems still persist, please make note of it in this bug report. -- 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 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #28 from Fedora Update System --- lpf-0-10.d18db6d.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/lpf-0-10.d18db6d.fc19 -- 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 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #27 from Alec Leamas --- Per fpc decision in ticket 362 lpf-spotify-client should be moved to a more appropriate repository. The package is retired. -- 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 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #26 from Fedora Update System --- lpf-0-9.b40e846.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/lpf-0-9.b40e846.fc19 -- 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 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||lpf-0-8.ff50a5b.fc20 Resolution|--- |ERRATA Last Closed||2013-11-10 02:02:20 --- Comment #25 from Fedora Update System --- lpf-0-8.ff50a5b.fc20 has been pushed to the Fedora 20 stable repository. -- 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 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #24 from Fedora Update System --- lpf-0-8.ff50a5b.fc20 has been pushed to the Fedora 20 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #23 from Fedora Update System --- lpf-0-8.ff50a5b.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/lpf-0-8.ff50a5b.fc19 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #22 from Fedora Update System --- lpf-0-8.ff50a5b.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/lpf-0-8.ff50a5b.fc20 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #21 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Alec Leamas changed: What|Removed |Added Flags||fedora-cvs? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #20 from Alec Leamas --- New Package SCM Request === Package Name: lpf Short Description: Local package factory - build non-redistributable rpms Owners: leamas Branches: f19 f20 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Alec Leamas changed: What|Removed |Added Assignee|nob...@fedoraproject.org|kcham...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Alec Leamas changed: What|Removed |Added Status|NEW |ASSIGNED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #19 from Alec Leamas --- Just as a clarification, I don't plan to actually import this until a actual lpf package is approved. Currently, I'm working with lpf-spotify-client. If someone else is interested of this package I'll import it asap, though. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=q0Zc2K9Fyw&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Alec Leamas changed: What|Removed |Added Flags|needinfo?(leamas.alec@gmail | |.com) | --- Comment #18 from Alec Leamas --- Cristopher: still on holiday, back in August (date to ber defined). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Mml35pfmpH&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Christopher Meng changed: What|Removed |Added CC||cicku...@gmail.com, ||leamas.a...@gmail.com Flags||needinfo?(leamas.alec@gmail ||.com) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=FAFYc2qC6B&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Kashyap Chamarthy changed: What|Removed |Added Flags||fedora-review+ --- Comment #17 from Kashyap Chamarthy --- (In reply to Alec Leamas from comment #15) > (In reply to Kashyap Chamarthy from comment #14) > [cut] > > (1) Lincences > > > > Unknown or generated > > > > > > /var/lib/mock/fedora-rawhide-x86_64/root/builddir/build/BUILD/lpf- > > 46ae0c3b2791013a2b5b1d03137538b9bc906350/scripts/build_error.py > > > > /var/lib/mock/fedora-rawhide-x86_64/root/builddir/build/BUILD/lpf- > > 46ae0c3b2791013a2b5b1d03137538b9bc906350/scripts/update.py > Even if these files are not marked properly, the overall licensing > situation for this package should be clear form the README and the LICENSE > file IMHO. If you insist, I can patch the files. However, I've approved > some packages myself containing files with these licenses since I havn't > found anything in the guidelines which says I shouldn't in cases like this?! Ok, ACK, I'm not rigid here. > > > > (2) No %check? > Unit tests are on the todo-list but yet not in place. Is %check required in > the guidelines? Well, not mandatory. Once you have them, you can update the spec. > > > > (3) Rationale for the non-standard-dir-perm (from rpmlint errors) > Because user modifies files owned by the lpg user pkg-build using group > permissions, the 775 permissions is needed to allow this. There's some more > in the README on this. ACK. APPROVED. (Sorry for the delay.) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Bp2QKO0TbH&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #16 from Alec Leamas --- Just FYI I'll be on vacation (disconnected on a sailing yacht) from Thursday until some day in August yet to be defined :) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=fuPD7R7h8m&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #15 from Alec Leamas --- (In reply to Kashyap Chamarthy from comment #14) [cut] > (1) Lincences > > Unknown or generated > > > /var/lib/mock/fedora-rawhide-x86_64/root/builddir/build/BUILD/lpf- > 46ae0c3b2791013a2b5b1d03137538b9bc906350/scripts/build_error.py > > /var/lib/mock/fedora-rawhide-x86_64/root/builddir/build/BUILD/lpf- > 46ae0c3b2791013a2b5b1d03137538b9bc906350/scripts/update.py Even if these files are not marked properly, the overall licensing situation for this package should be clear form the README and the LICENSE file IMHO. If you insist, I can patch the files. However, I've approved some packages myself containing files with these licenses since I havn't found anything in the guidelines which says I shouldn't in cases like this?! > (2) No %check? Unit tests are on the todo-list but yet not in place. Is %check required in the guidelines? > (3) Rationale for the non-standard-dir-perm (from rpmlint errors) Because user modifies files owned by the lpg user pkg-build using group permissions, the 775 permissions is needed to allow this. There's some more in the README on this. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MrBBsJ0V3h&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943
--- Comment #14 from Kashyap Chamarthy ---
Manual Review
~
Mostly looks good, just a couple of things:
TL;DR:
(1) Lincences
Unknown or generated
/var/lib/mock/fedora-rawhide-x86_64/root/builddir/build/BUILD/lpf-46ae0c3b2791013a2b5b1d03137538b9bc906350/scripts/build_error.py
/var/lib/mock/fedora-rawhide-x86_64/root/builddir/build/BUILD/lpf-46ae0c3b2791013a2b5b1d03137538b9bc906350/scripts/update.py
(2) No %check?
(3) Rationale for the non-standard-dir-perm (from rpmlint errors)
= MUST items =
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[-]: Package is not known to require ExcludeArch.
[x]: Package complies to the Packaging Guidelines
[!]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
"Unknown or generated". 2 files have unknown license. Detailed output of
licensecheck in /home/kashyap/rpmbuild/SRPMS/lpf/licensecheck.txt
[x]: Package consistently uses macro is (instead of hard-coded directory
names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: update-desktop-database is invoked when required
Note: desktop file(s) in lpf
- {No 'MimeType key' in lpf.desktop, that's fine}
[x]: gtk-update-icon-cache is invoked when required
Note: icons in lpf
- $ grep gtk-update srpm-unpacked/lpf.spec
/usr/bin/gtk-update-icon-cache %{_datadir}/icons/hicolor &>/dev/null || :
/usr/bin/gtk-update-icon-cache %{_datadir}/icons/hicolor &>/dev/null || :
[-]: Large documentation must go in a -doc subpackage.
Python:
[-]: Python eggs must not download any dependencies during the build process.
[-]: A package which is used by another package via an egg interface should
provide egg info.
[x]: Package meets the Packaging Guidelines::Python
= SHOULD items =
Generic:
[x]: If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
- {I didn't test the package extensively.}
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
architectures.
[x]: %check is present and all tests pass.
[-]: Packages should try to preserve timestamps of original installed files.
Rpmlint (installed packages)
# rpmlint lpf
lpf.noarch: W: spelling-error Summary(en_US) redistributable -> redistribute,
redistribution, attributable
lpf.noarch: W: spelling-error %description -l en_US akmod -> Kodak
lpf.noarch: W: spelling-error %description -l en_US dkms -> dims, dams, DBMS
lpf.noarch: W: incoherent-version-in-changelog 0-1.c4bc5a2 ['0-2.7df703a.fc20',
'0-2.7df703a']
lpf.noarch: W: non-standard-uid /var/lib/lpf/rpms pkg-build
lpf.noarch: W: non-standard-gid /var/lib/lpf/rpms pkg-build
lpf.noarch: E: non-standard-dir-perm /var/lib/lpf/rpms 0775L
lpf.noarch: W: non-standard-uid /var/lib/lpf pkg-build
lpf.noarch: W: non-standard-gid /var/lib/lpf pkg-build
lpf.noarch: E: non-standard-dir-perm /var/lib/lpf 0775L
lpf.noarch: W: non-standard-uid /var/lib/lpf/log pkg-build
lpf.noarch: W: non-standard-gid /var/lib/lpf/log pkg-build
lpf.noarch: E: non-standard-dir-perm /var/lib/lpf/log 0775L
lpf.noarch: W: non-standard-uid /var/lib/lpf/packages pkg-build
lpf.noarch: W: non-standard-gid /var/lib/lpf/packages pkg-build
lpf.noarch: E: non-standard-dir-perm /var/lib/lpf/packages 0775L
lpf.noarch: W: non-standard-uid /var/lib/lpf/approvals pkg-build
lpf.noarch: W: non-standard-gid /var/lib/lpf/approvals pkg-build
lpf.noarch: E: non-standard-dir-perm /var/lib/lpf/a
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #13 from Kashyap Chamarthy --- Manual Review in progress. Koji scratch build successful: $ koji build --scratch rawhide lpf-0-3.46ae0c3.fc18.src.rpm Uploading srpm: lpf-0-3.46ae0c3.fc18.src.rpm [] 100% 00:00:03 176.65 KiB 57.87 KiB/sec Created task: 5581936 Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=5581936 Watching tasks (this may be safely interrupted)... 5581936 build (rawhide, lpf-0-3.46ae0c3.fc18.src.rpm): open (ppc12.phx2.fedoraproject.org) 5581937 buildArch (lpf-0-3.46ae0c3.fc18.src.rpm, noarch): free 5581937 buildArch (lpf-0-3.46ae0c3.fc18.src.rpm, noarch): free -> open (buildvm-07.phx2.fedoraproject.org) 5581937 buildArch (lpf-0-3.46ae0c3.fc18.src.rpm, noarch): open (buildvm-07.phx2.fedoraproject.org) -> closed 0 free 1 open 1 done 0 failed 5581936 build (rawhide, lpf-0-3.46ae0c3.fc18.src.rpm): open (ppc12.phx2.fedoraproject.org) -> closed 0 free 0 open 2 done 0 failed 5581936 build (rawhide, lpf-0-3.46ae0c3.fc18.src.rpm) completed successfully -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=KnTdgRD21C&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #12 from Alec Leamas --- Tom: thanks! Kashyap: new links: spec: http://leamas.fedorapeople.org/lpf/3/lpf.spec srpm: http://leamas.fedorapeople.org/lpf/3/lpf-0-3.46ae0c3.fc18.src.rpm Changelog: * Sun Jun 23 2013 Alec Leamas - 0-3.fe3defcf9 - Removed examples, added lpf spec template. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=QhCY6GmoSs&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #11 from Kashyap Chamarthy --- Thanks Tom, for your comment. So, Alec, I assume you'd be providing updated SPEC/SRPM with the said examples elided (is that the right assumption?), so that this review can proceed further. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=XlTMnbFBNf&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Tom "spot" Callaway changed: What|Removed |Added Blocks|182235 (FE-Legal) | --- Comment #10 from Tom "spot" Callaway --- The tooling itself is not an issue. Any lpf examples/recipes/packages must be legally reviewed on a case-by-case basis to minimize risk. Lifting FE-Legal here, but please be sure to remove the examples before committing. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=WjlqAhsXwj&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #9 from Alec Leamas --- Tom: ping? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=8R2QdvgeRn&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #8 from Alec Leamas --- The examples are silly and should be removed not only for legal reasons. They are needed upstream, but not in the package. My bad. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=pKDJefduVT&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #7 from Tom "spot" Callaway --- I don't think there is any reason not to permit the lpf tooling itself, but the examples might be problematic (along with any lpf helper "packages"). I'm going to have to discuss this scenario with Red Hat Legal before we can move anything forward. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Hh11PAtay7&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Kashyap Chamarthy changed: What|Removed |Added Blocks|973069 |182235 (FE-Legal) --- Comment #6 from Kashyap Chamarthy --- Alec, thanks for the update. Yes, made a FE-Legal blocker. (Sorry, I was occupied with other higher priority work, and didn't mean to ignore this review request.) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=DAh23DCElU&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #5 from Alec Leamas --- Deadlock? If you need Tom's opinion on this, wouldn't it be better to block the bug on FE-Legal? If not, as I think, could we proceed? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=InM3YX7Kse&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #4 from Alec Leamas --- My judgment is that this package as such is not problematic from legal point of view - it's just a tool simplifying the rpm build process similar in scope to e. g., rpmbuild. OTOH, I'm definitely not the lawyer here. Upcoming lpf-* packages is another issue. E. g., the blocked bug #973069, which I already have blocked on FE-Legal. Perhaps also the examples in this package could be problematic, but they could then just be excluded from the package. If necessary, I could comment on the rpmlint output. However, my overall feeling is that the messages are harmless (admittedly, I could get rid of the last by writing a manpage, though). Did some specfile nit-picking (mid-air collision), same links. Changelog: * Thu Jun 13 2013 Alec Leamas - 0-3.3c3c216 - Added BR: python2-devel - Simplified Source0 (https://fedorahosted.org/fpc/ticket/284) - Using 2775 instead of 775 perms (https://fedorahosted.org/fpc/ticket/286) New links: spec: http://leamas.fedorapeople.org/lpf/3/lpf.spec srpm: http://leamas.fedorapeople.org/lpf/3/lpf-0-3.4b9f706.fc18.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=akTx0tu7Og&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Kashyap Chamarthy changed: What|Removed |Added CC||tcall...@redhat.com --- Comment #3 from Kashyap Chamarthy --- Before I proceed with manual review, we might need a comment from legal folks given the package description. Added Tom Callaway to CC. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=M0jOjOmF2m&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943
Kashyap Chamarthy changed:
What|Removed |Added
CC||kcham...@redhat.com
--- Comment #2 from Kashyap Chamarthy ---
Running fedora-review tool against rawhide. Manual review upcoming..
$ fedora-review -m fedora-rawhide-x86_64 \
--rpm-spec -n lpf-0-2.7df703a.fc18.src.rpm
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- Package contains BR: python2-devel or python3-devel
See: http://fedoraproject.org/wiki/Packaging:Python#BuildRequires
= MUST items =
Generic:
[ ]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[ ]: Package contains no bundled libraries without FPC exception.
[ ]: Changelog in prescribed format.
[ ]: Sources contain only permissible code or content.
[ ]: Development files must be in a -devel package
[ ]: Package requires other packages for directories it uses.
[ ]: Package uses nothing in %doc for runtime.
[ ]: Package is not known to require ExcludeArch.
[ ]: Package complies to the Packaging Guidelines
[ ]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
"Unknown or generated". 2 files have unknown license. Detailed output of
licensecheck in /home/kashyap/rpmbuild/SRPMS/lpf/licensecheck.txt
[ ]: Package consistently uses macro is (instead of hard-coded directory
names).
[ ]: Package is named according to the Package Naming Guidelines.
[ ]: Package does not generate any conflict.
[ ]: Package obeys FHS, except libexecdir and /usr/target.
[ ]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[ ]: Package must own all directories that it creates.
[ ]: Package does not own files or directories owned by other packages.
[ ]: Requires correct, justified where necessary.
[ ]: Spec file is legible and written in American English.
[ ]: Package contains systemd file(s) if in need.
[ ]: update-desktop-database is invoked when required
Note: desktop file(s) in lpf
[ ]: gtk-update-icon-cache is invoked when required
Note: icons in lpf
[ ]: Large documentation must go in a -doc subpackage.
Note: Documentation size is 20480 bytes in 2 files.
[x]: All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: %config files are marked noreplace or the reason is justified.
[x]: Each %files section contains %defattr if rpm < 4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package contains desktop file if it is a GUI application.
[x]: Package installs a %{name}.desktop using desktop-file-install if there is
such a file.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Fully versioned dependency in subpackages, if present.
[x]: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %doc.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
work.
[x]: Package is named using only allowed ASCII characters.
[x]: No %config files under /usr.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
[x]: Package successfully compiles and builds into binary rpms on at least one
supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
Python:
[ ]: Python eggs must not download any dependencies during the build process.
[ ]: A package which is used by another package via an egg interface should
provide egg info.
[ ]: Package meets the Packaging Guidelines::Python
[x]: Binary eggs must be removed in %prep
= SHOULD items =
Generic:
[ ]: If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[ ]: Final provides and requires are sane (see attachments).
[ ]: Package functions as described.
[ ]: Latest version is packaged.
[ ]: Package does not include license text files s
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 --- Comment #1 from Alec Leamas --- Simplified spec using a Makefile. new links: spec: http://leamas.fedorapeople.org/lpf/2/lpf.spec srpm: http://leamas.fedorapeople.org/lpf/2/lpf-0-2.7df703a.fc18.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=wYmpWWyJhI&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms
https://bugzilla.redhat.com/show_bug.cgi?id=972943 Alec Leamas changed: What|Removed |Added Blocks||973069 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=35mVBjLhQC&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
