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

Ankur Sinha (FranciscoD) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(sanjay.ankur@gmai |
                   |l.com)                      |



--- Comment #11 from Ankur Sinha (FranciscoD) <[email protected]> ---
Hi Anil,

The package needs to be built correctly so we can complete the review. Quite a
few issues outstanding with the current spec:

Are you a package maintainer already? I can help you with the package and
sponsor you as a maintainer if you wish: please e-mail me and we can work on
that.


Issues:
=======
- Package installs properly.
  Note: Installation errors (see attachment)
  See: https://docs.fedoraproject.org/en-US/packaging-guidelines/
^ Needs to be checked

- I've filed a ticket asking upstream to add tests and licenses to the release,
  or make sure the github releases/tags match pypi so we can use them:
  https://github.com/BlueBrain/BluePyOpt/issues/280

We should use the github tar here. It contains:

- license files
- tests
- docs.


- Please complete the description. It is incomplete. Probably worth defining a
  macro and re-using it. Example here:
 
https://pagure.io/neuro-sig/NeuroFedora/blob/master/f/spec-templates/python.spec#_10

Some more work needed. Unfortunately, the mock build seemed to have failed
here.


-- 
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 -- [email protected]
To unsubscribe send an email to [email protected]
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/[email protected]

Reply via email to