[Bug 2215421] Review Request: x2gokdrive - KDrive graphical server backend for X2GoServer

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

 



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



--- Comment #10 from Orion Poplawski <orion@xxxxxxxx> ---
(In reply to Jos de Kloe from comment #9)
> Issues:
> =======
> - Package must not depend on deprecated() packages.
>   Note: openssl1.1-devel is deprecated, you must not depend on it.
>   See: https://docs.fedoraproject.org/en-US/packaging-
>   guidelines/deprecating-packages/

I honestly have no idea where this comment is coming from.  The package BRs
pkgconfig(openssl), which on rawhide brings in:

 openssl-devel                x86_64   1:3.1.1-4.fc40            fedora   2.6 M

Although the binaries don't require either of libcrypto or libssl so I'm not
entirely sure it's used, despite being checked for:

checking for openssl... yes

I see no evidence of openssl1.1-devel being involved.

> - License field in the package spec file matches the actual license.
>   ==>the copyright file that you included describes GPL-2 and GPL-2+
>      but the spec file gives GPL-3.0-or-later
>      You added a comment to explain, but if this package actually
>      uses a mixture of GPL-2, GPL-2+, and GPL-3+
>      then maybe the license flag should be GPL-2.0-or-later ?

According to https://docs.fedoraproject.org/en-US/legal/license-field/:

The spec file License tag consists of an enumeration of all licenses covering
any code or other material contained in the corresponding binary RPM. This
enumeration must take the form of an SPDX license expression. No further
analysis as to the "effective" license should be done.

> -rpmspec complains about permissions of the source files.
>  please fix this.

Fixed.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2215421

Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202215421%23c10
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux