[Bug 843997] Review Request: mlpack - scalable C++ machine learning library

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

 



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

--- Comment #13 from José Matos <jamatos@xxxxxxxx> ---
I am running out of excuses to approve this package. ;-)

One small issue, no need to generate a new srpm for this. There still a
leftover from the previous license installation:

# Put the license file in place.
mkdir -p $RPM_BUILD_ROOT/%{_docdir}/%{name}-%{version}/


Regarding the rpmlint output note that it issues a warning and not an error,
and as you stated in this case _exit_ should really be there so we can ignore
rpmlint in this case.

I will complete the review tomorrow since in my time zone I am past midnight
and tired. :-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
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]