[Bug 1723013] Review Request: mne-cpp - A Framework for Electrophysiology

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

 



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



--- Comment #4 from David Auer <dreua@xxxxxxxxx> ---
AFAIK you can not edit previous messages. Just post the URLs in a new comment.
Unfortunately my tries to convert your URLs to direct download weren't
successful. 

When I last commented I took a short peek at your spec file and if I recall
correctly there are some things that need to be fixed, if you haven't already
please make sure your Version/Release, License and makro usage meets the
guidelines (https://docs.fedoraproject.org/en-US/packaging-guidelines/). I will
try to make a full unofficial review in the next days. (I am not a packager yet
so all of this is unofficial!) 

I think downloading sources via curl in %setup is not allowed, but not entirely
sure in this case, because it wouldn't happen for Fedora but only for rhel
builds. 

Additionally it would be a good idea to do a koji sratch build (also post a
link when it's done) and I personally ran the fedora-review tool on my own
package, it might just help you to find some mistakes and fix them yourself.

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




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

  Powered by Linux