[Bug 1193923] Review Request: mbedtls - polarssl package renaming process

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

 



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



--- Comment #6 from Morten Stevens <mstevens@xxxxxxxxxxxxxxx> ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #5)

Here is the latest mbedtls.spec file:

https://mstevens.fedorapeople.org/mbedtls/mbedtls.spec
https://mstevens.fedorapeople.org/mbedtls/mbedtls-1.3.10-1.fc23.src.rpm

> - Consider using %make_install instead of the open-coded make install ...

fixed

> fedora-review complains:
> - Sources used to build the package match the upstream source, as provided
>   in the spec URL.
>   Note: Upstream MD5sum check error, diff is in /var/tmp/review-
>   rc/1193923-mbedtls/diff.txt
>   See: http://fedoraproject.org/wiki/Packaging/SourceURL

The URL has been changed to: Source0:
https://tls.mbed.org/download/%{name}-%{version}-gpl.tgz

> - Static libraries in -static or -devel subpackage, providing -devel if
>   present.
>   Note: Package has .a files: mbedtls-devel. Does not provide -static:
>   mbedtls-devel.
>   See: http://fedoraproject.org/wiki/Packaging/Guidelines#StaticLibraries

new subpackage added: mbedtls-static

> - Large documentation must go in a -doc subpackage. Large could be size
>   (~1MB) or number of files.
>   Note: Documentation size is 40683520 bytes in 1423 files.
>   See:
>   http://fedoraproject.org/wiki/Packaging/Guidelines#PackageDocumentation

new subpackage added: mbedtls-doc

-- 
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@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





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