[Bug 1216279] Review Request: cppformat - Small, safe and fast formating library for C++

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

 



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



--- Comment #8 from Dave Johansen <davejohansen@xxxxxxxxx> ---
(In reply to Antonio Trande from comment #7)
> - Exist some hidden directories in the -doc package.
>   Please check if they can be removed.

I removed them and notified upstream.

> - Source0 link is not valid; use this form:
> 
> https://github.com/cppformat/cppformat/archive/1.1.0.zip#/%{name}-%{version}.
> zip

The link is actually valid. It's just that github doesn't like the way that
rpmlint does the request. I would switch to the recommended URL but the URL I'm
using right now is a release that contains the built documentation. I'm
watching on packaging the requirements to build the packaging in Fedora, but
for now the current URL is probably the beset option.

> - cppformat.x86_64: W: private-shared-object-provides
> /usr/lib64/libformat.so.1.1.0 libformat.so.1()(64bit)
> 
> See
> https://fedoraproject.org/wiki/Common_Rpmlint_issues?rd=PackageMaintainers/
> Common_Rpmlint_Issues#private-shared-object-provides

I believe that this is actually a false warning. I opened a bugzilla (see
https://bugzilla.redhat.com/show_bug.cgi?id=1253917 ).

The latest version of the .spec and source .rpm can be found at:
Spec URL: https://daveisfera.fedorapeople.org/cppformat_1.1.0/cppformat.spec
SRPM URL:
https://daveisfera.fedorapeople.org/cppformat_1.1.0/cppformat-1.1.0-1.fc22.src.rpm

-- 
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]