[Bug 972943] Review Request: lpf - Local package factory - build non-redistributable rpms

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=972943

--- Comment #15 from Alec Leamas <leamas.alec@xxxxxxxxx> ---
(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@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]