[Bug 1651397] Review Request: python-efel - Electrophys Feature Extraction Library

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651397



--- Comment #10 from Fedora Update System  ---
python-efel-3.0.22-1.fc29 has been pushed to the Fedora 29 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-1531eeb1bd

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1619458] Review Request: python-pyphi - A library for computing integrated information

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1619458



--- Comment #14 from Fedora Update System  ---
python-pyphi-1.1.0-2.fc29 has been pushed to the Fedora 29 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-a352df747a

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650937] Review Request: python-dipy - Diffusion MRI utilities in python

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650937



--- Comment #10 from Fedora Update System  ---
python-dipy-0.15.0-0.3.git756b519.fc29 has been pushed to the Fedora 29 testing
repository. If problems still persist, please make note of it in this bug
report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-b3d5c7b663

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649874] Review Request: hpx - a general purpose C++ runtime system for parallel and distributed applications of any scale

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649874



--- Comment #19 from Fedora Update System  ---
hpx-1.2.0-3.fc29 has been pushed to the Fedora 29 testing repository. If
problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-a9b53adbdc

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651766] Review Request: python-pymatreader - Convenient reader for Matlab mat files

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651766



--- Comment #7 from Fedora Update System  ---
python-mne-0.17-1.fc29, python-pymatreader-0.0.17-1.fc29 has been pushed to the
Fedora 29 testing repository. If problems still persist, please make note of it
in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-e65d3f09c0

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650633] Review Request: pam_2fa - Second factor authentication for PAM

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650633



--- Comment #15 from Fedora Update System  ---
pam_2fa-1.0-2.fc29 has been pushed to the Fedora 29 testing repository. If
problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-18c38b185e

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651766] Review Request: python-pymatreader - Convenient reader for Matlab mat files

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651766

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #6 from Fedora Update System  ---
python-mne-0.17-1.fc28, python-pymatreader-0.0.17-1.fc28 has been pushed to the
Fedora 28 testing repository. If problems still persist, please make note of it
in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-f9dc990324

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651397] Review Request: python-efel - Electrophys Feature Extraction Library

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651397

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
python-efel-3.0.22-1.fc28 has been pushed to the Fedora 28 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-91dbe4e2b6

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1619458] Review Request: python-pyphi - A library for computing integrated information

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1619458

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #13 from Fedora Update System  ---
python-pyphi-1.1.0-2.fc28 has been pushed to the Fedora 28 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-c8dd8701a6

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649874] Review Request: hpx - a general purpose C++ runtime system for parallel and distributed applications of any scale

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649874

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #18 from Fedora Update System  ---
hpx-1.2.0-3.fc28 has been pushed to the Fedora 28 testing repository. If
problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-8367209bd8

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650937] Review Request: python-dipy - Diffusion MRI utilities in python

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650937

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
python-dipy-0.15.0-0.3.git756b519.fc28 has been pushed to the Fedora 28 testing
repository. If problems still persist, please make note of it in this bug
report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-373dc3e3a0

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650633] Review Request: pam_2fa - Second factor authentication for PAM

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650633

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #14 from Fedora Update System  ---
pam_2fa-1.0-2.fc28 has been pushed to the Fedora 28 testing repository. If
problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-127affe1d7

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651849] New: Review Request: python-pure-sasl - A pure python SASL client

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651849

Bug ID: 1651849
   Summary: Review Request: python-pure-sasl - A pure python SASL
client
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: tdeca...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Hello, this is a new requirement of recent python-kazoo version.

Spec URL: https://fedorapeople.org/~tdecacqu/python-pure-sasl.spec
SRPM URL:
https://fedorapeople.org/~tdecacqu/python-pure-sasl-0.5.1-1.fc30.src.rpm
Description: A pure python SASL client
Fedora Account System Username: tdecacqu

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1639186] Review Request: libcerf - a library that provides complex error functions

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1639186

Dmitri A. Sergatskov  changed:

   What|Removed |Added

 CC||dasergats...@gmail.com



--- Comment #13 from Dmitri A. Sergatskov  ---
libcerf 1.8 and 1.9 are broken. Please downggrade to 1.7 before 1.10 release.
http://apps.jcns.fz-juelich.de/doku/sc/libcerf

Dmitri.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1631987] Review Request: python-deepdiff - Deep Difference and search of any Python object/data

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1631987

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2018-11-20 22:10:59



--- Comment #15 from Fedora Update System  ---
python-deepdiff-3.3.0-3.fc28 has been pushed to the Fedora 28 stable
repository. If problems still persist, please make note of it in this bug
report.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1639186] Review Request: libcerf - a library that provides complex error functions

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1639186

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2018-11-20 22:10:00



--- Comment #12 from Fedora Update System  ---
libcerf-1.9-3.fc28 has been pushed to the Fedora 28 stable repository. If
problems still persist, please make note of it in this bug report.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651759] Review Request: rust-winapi-x86_64-pc-windows-gnu - Import libraries for the x86_64-pc-windows-gnu target.

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651759



--- Comment #2 from Luis Segundo  ---
Hello Robert,
I would like include this Package "dutree" more info
https://docs.rs/dutree/0.2.9/dutree/

this package need the follow Dependencies
getopts ^0.2
regex ^0.2
terminal_size ^0.1.7
unicode-segmentation ^1.2.0
unicode-width ^0.1.1

now "terminal_size" it's not include yet on fedora this one requiere the follow
Dependencies

libc ^0.2
winapi ^0.3

"winapi" need the lib "winapi-x86_64-pc-windows-gnu" that I try to include


referecen
http://eminence.github.io/terminal-size/doc/terminal_size/index.html
https://docs.rs/winapi/0.3.6/winapi/

I will be waiting your comments

Thanks

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651824] Review Request: python-elephant - Elephant is a package for analysis of electrophysiology data in Python

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651824



--- Comment #2 from Luis Bazan  ---
Reference for test:
https://elephant.readthedocs.io/en/latest/developers_guide.html

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651824] Review Request: python-elephant - Elephant is a package for analysis of electrophysiology data in Python

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651824



--- Comment #1 from Luis Bazan  ---
I excluded test because failed. (need contact upstream to check these)

%{__python2} setup.py nosetests
%{__python3} setup.py nosetests

Or

%{__python2} setup.py nosetests --with-coverage --cover-package=elephant
--cover-erase
%{__python3} setup.py nosetests --with-coverage --cover-package=elephant
--cover-erase

Or using
%{__python2} setup.py -m unittest discover
%{__python3} setup.py -m unittest discover

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651824] Review Request: python-elephant - Elephant is a package for analysis of electrophysiology data in Python

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651824

Luis Bazan  changed:

   What|Removed |Added

  Alias||python-elephant



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651824] Review Request: python-elephant - Elephant is a package for analysis of electrophysiology data in Python

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651824

Luis Bazan  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|l...@blackfile.net



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651824] Review Request: python-elephant - Elephant is a package for analysis of electrophysiology data in Python

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651824

Luis Bazan  changed:

   What|Removed |Added

 Blocks||1276941 (fedora-neuro)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1276941
[Bug 1276941] Fedora NeuroImaging and NeuroScience tracking bug
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651824] New: Review Request: python-elephant - Elephant is a package for analysis of electrophysiology data in Python

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651824

Bug ID: 1651824
   Summary: Review Request: python-elephant - Elephant is a
package for analysis of electrophysiology data in
Python
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: bazanlui...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://lbazan.fedorapeople.org/python-elephant.spec
SRPM URL: https://lbazan.fedorapeople.org/python-elephant-0.6.0-1.fc29.src.rpm

Description:
Elephant - Electrophysiology Analysis Toolkit Elephant is a package for the
analysis of neurophysiology data, based on Neo.Code status -- :target:

Fedora Account System Username: lbazan

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651808] Review Request: python-hkdf - HMAC-based Extract-and-Expand Key Derivation Function (HKDF )

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651808

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin  ---
Package approved.



Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated". 6 files have unknown license. Detailed
 output of licensecheck in /home/bob/packaging/review/python-hkdf
 /review-python-hkdf/licensecheck.txt
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[x]: Package requires other packages for directories it uses.
[x]: Package does not own files or directories owned by other packages.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

Python:
[x]: Python eggs must not download any dependencies during the build
 process.
[x]: A package which is used by another package via an egg interface should
 provide egg info.
[x]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Packages MUST NOT have dependencies (either build-time or runtime) on
 packages named with the unversioned python- prefix unless no properly
 versioned package exists. Dependencies on Python packages instead MUST
 use names beginning with python2- or python3- as appropriate.
[x]: Python packages must not contain %{pythonX_site(lib|arch)}/* in %files
[x]: Binary eggs must be removed in %prep

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package 

[Bug 1405548] Review Request: emacs-json-mode - Major mode for editing JSON files with Emacs

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1405548

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin  ---
 - Group: is not used in Fedora


Package approved.



Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "*No copyright* GPL (v3 or later)", "GPL (v3 or later)".
 Detailed output of licensecheck in /home/bob/packaging/review/emacs-
 json-mode/review-emacs-json-mode/licensecheck.txt
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for directories it uses.
[x]: Package does not own files or directories owned by other packages.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: SourceX is a working URL.
[x]: Spec use %global instead of 

[Bug 1405547] Review Request: emacs-json-snatcher - Get the path to a JSON element in Emacs

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1405547

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin  ---
 - Group is not needed in Fedora

Package approved

Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "GPL (v3 or later)", "Unknown or generated". 2 files have
 unknown license. Detailed output of licensecheck in
 /home/bob/packaging/review/emacs-json-snatcher/review-emacs-json-
 snatcher/licensecheck.txt
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for directories it uses.
[x]: Package does not own files or directories owned by other packages.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: SourceX is a working URL.
[x]: 

[Bug 1590431] Review Request: qemu-lite - OpenBIOS development utilities

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1590431

Jose Carlos Venegas Munoz  changed:

   What|Removed |Added

  Flags|needinfo?(berrange@redhat.c |
   |om) |



--- Comment #4 from Jose Carlos Venegas Munoz 
 ---
Hi, 
Today Kata uses some packages as part of its CI:
qemu-lite: qemu with additional patches to improve boot time and memory usage.
qemu-vanilla: This is a qemu that is not modified. The idea is that yes, kata
could work with default qemu package from a distro.

I think it is ok as intial work to use qemu from vedora, to avoid add an
additional package to maintain. Be aware qemu with a lot of features enabled
may decrease the boot time or memory footprint.

But as part of a initial work we could test the current qemu version for f28
and f29 and request any change or patch is needed to the qemu package.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1405546] Review Request: emacs-json-reformat - Reformatting tool for JSON

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1405546

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin  ---
Package approved.



Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Expat License", "Unknown or generated". 9 files have unknown
 license. Detailed output of licensecheck in /home/bob/packaging/review
 /emacs-json-reformat/review-emacs-json-reformat/licensecheck.txt
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for directories it uses.
[x]: Package does not own files or directories owned by other packages.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
 files.
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: SourceX is a working URL.
[x]: Spec use %global instead of %define unless 

[Bug 1651759] Review Request: rust-winapi-x86_64-pc-windows-gnu - Import libraries for the x86_64-pc-windows-gnu target.

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651759

Luis Bazan  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651808] New: Review Request: python-hkdf - HMAC-based Extract-and-Expand Key Derivation Function (HKDF )

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651808

Bug ID: 1651808
   Summary: Review Request: python-hkdf - HMAC-based
Extract-and-Expand Key Derivation Function (HKDF)
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: decatho...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: https://decathorpe.fedorapeople.org/packages/python-hkdf.spec
SRPM URL:
https://decathorpe.fedorapeople.org/packages/python-hkdf-0.0.3-1.fc29.src.rpm

Description:
This module implements the HMAC Key Derivation function, defined at
http://tools.ietf.org/html/draft-krawczyk-hkdf-01. There are two
interfaces: a functional interface, with separate extract and expand
functions as defined in the draft RFC, and a wrapper class for these
functions.

Fedora Account System Username: decathorpe

koji scratch build for rawhide:
https://koji.fedoraproject.org/koji/taskinfo?taskID=31025263


The .spec file was generated with pyp2rpm and adapted by me.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1183169] Review Request: ykfde - opening LUKS with yubikey

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1183169

Orion Poplawski  changed:

   What|Removed |Added

 CC||bper...@fedoraproject.org,
   ||or...@nwra.com
  Flags||needinfo?(bpereto@fedorapro
   ||ject.org)



--- Comment #7 from Orion Poplawski  ---
Benjamin - Are you still interested in packaging this?

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1647354] Review Request: wfdb - Software working with recordings of physiologic signals

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1647354



--- Comment #5 from Robert-André Mauchin  ---
Regarding rman:

https://src.fedoraproject.org/rpms/rman/blob/master/f/dead.package


Package was removed due to licensing issues (it was only available under
Artistic 1.0).

Every effort was made to contact upstream for relicensing, but no response was
ever received.


See: https://fedoraproject.org/wiki/Features/Artistic1Removal


No luck there.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651759] Review Request: rust-winapi-x86_64-pc-windows-gnu - Import libraries for the x86_64-pc-windows-gnu target.

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651759

Robert-André Mauchin  changed:

   What|Removed |Added

 CC||zebo...@gmail.com



--- Comment #1 from Robert-André Mauchin  ---
Update your rust2rpm to the very latest version (from Rawhide), your generated
SPEC use the old format. Also I don't see the point of packaging this, it seems
Windows only like Winapi, why do you need it?

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651739] Review Request: caribou EPEL7 - EPEL7 branch of caribou ( cinnamon prereq)

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651739

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||zebo...@gmail.com
 Resolution|--- |CANTFIX
Last Closed||2018-11-20 16:54:39



--- Comment #1 from Robert-André Mauchin  ---
If you want an EPEL branch to be made, contact the current maintainer and ask
him for one, maybe ask him to comaintain it. Making a Review request is not the
way to proceed.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650633] Review Request: pam_2fa - Second factor authentication for PAM

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650633



--- Comment #13 from Fedora Update System  ---
pam_2fa-1.0-2.fc29 has been submitted as an update to Fedora 29.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-18c38b185e

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650633] Review Request: pam_2fa - Second factor authentication for PAM

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650633

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |MODIFIED



--- Comment #12 from Fedora Update System  ---
pam_2fa-1.0-2.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-127affe1d7

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651766] Review Request: python-pymatreader - Convenient reader for Matlab mat files

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651766

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #4 from Fedora Update System  ---
python-pymatreader-0.0.17-1.fc28 python-mne-0.17-1.fc28 has been submitted as
an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-f9dc990324

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651766] Review Request: python-pymatreader - Convenient reader for Matlab mat files

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651766



--- Comment #5 from Fedora Update System  ---
python-pymatreader-0.0.17-1.fc29 python-mne-0.17-1.fc29 has been submitted as
an update to Fedora 29.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-e65d3f09c0

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1647354] Review Request: wfdb - Software working with recordings of physiologic signals

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1647354



--- Comment #4 from Robert-André Mauchin  ---
Also, it seems some tests only work with network access, which won't work in
Mock/Koji.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1647354] Review Request: wfdb - Software working with recordings of physiologic signals

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1647354



--- Comment #3 from Robert-André Mauchin  ---
 - Vendor: is forbidden in Fedora

 - Not needed anymore:

%post -p /sbin/ldconfig

%postun -p /sbin/ldconfig

 Use %ldconfig_scriptlets for F27 or EPEL. See
https://fedoraproject.org/wiki/Changes/Removing_ldconfig_scriptlets

 - Not needed, it is the default:

%defattr(-,root,root,-)

 - You forgot %{?_isa} in the Requires for the devel subpackage:

Requires: %{name}%{?_isa} = %{version}-%{release}

 - This part should be in %build:

cd doc/wpg-src
%__make 
cd ../wag-src
# package pdftk  was removed from distribution (in F21) replace it by qpdf
command
sed -i -e 's/pdftk/qpdf --empty wag.pdf --pages wag1.pdf wag2.pdf wag3.pdf
wag4.pdf --  # /g' Makefile
%__make
cd ../wug-src
%__make

 - Use make, not %__make. These kind of macros are not useful.

 - You could use pushd/popd instead of cd ../ in that section

 - GCC should be a BR rot a RR

BuildRequires:   gcc

 - Remove the glob here to own the include dir:

%files devel
%{_includedir}/%{name}/
%{_libdir}/libwfdb.so

 - Same here and own %{_datadir}/%{name}

%dir %{_datadir}/%{name}
%{_datadir}/%{name}/database/
%{_datadir}/%{name}/ps/

 - Your build must respect Fedora's CCFLAGS and LDFLAGS

 - Build fails:

DEBUG util.py:490:  BUILDSTDERR: No matching package to install: 'rman'
DEBUG util.py:490:  BUILDSTDERR: Not all dependencies satisfied
DEBUG util.py:490:  BUILDSTDERR: Error: Some packages could not be found.

I have no idea what rman is in this context.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651388] Review Request: python-xnat - A new XNAT client that exposes XNAT objects/ functions as python objects/functions

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651388



--- Comment #2 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/python-xnat

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651766] Review Request: python-pymatreader - Convenient reader for Matlab mat files

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651766



--- Comment #3 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/python-pymatreader

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1648728] Review Request: python-progressbar2 - A Progressbar library to provide visual progress to long running operations

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1648728

Luis Bazan  changed:

   What|Removed |Added

 Blocks||1651388 (python-xnat)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1651388
[Bug 1651388] Review Request: python-xnat - A new XNAT client that exposes
XNAT objects/functions as python objects/functions
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651388] Review Request: python-xnat - A new XNAT client that exposes XNAT objects/ functions as python objects/functions

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651388

Luis Bazan  changed:

   What|Removed |Added

 Depends On||1648728
   ||(python-progressbar2)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1648728
[Bug 1648728] Review Request: python-progressbar2 - A Progressbar library
to provide visual progress to long running operations
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651365] Review Request python-pynwb - PyNWB is a Python package for working with NWB files.

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651365

Luis Bazan  changed:

   What|Removed |Added

Summary|python-pynwb|Review Request python-pynwb
   ||- PyNWB is a Python package
   ||for working with NWB files.



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651407] Review Request: python-wxnatpy - wxnatpy is a wxPython widget which allows users to browse the contents of a XNAT repository.

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651407

Luis Bazan  changed:

   What|Removed |Added

Summary|python-wxnatpy  |Review Request:
   ||python-wxnatpy - wxnatpy is
   ||a wxPython widget which
   ||allows users to browse the
   ||contents of a XNAT
   ||repository.



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651388] Review Request: python-xnat - A new XNAT client that exposes XNAT objects/ functions as python objects/functions

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651388

Ankur Sinha (FranciscoD)  changed:

   What|Removed |Added

Summary|python-xnat: A new XNAT |Review Request: python-xnat
   |client that exposes XNAT|- A new XNAT client that
   |objects/functions as python |exposes XNAT
   |objects/functions   |objects/functions as python
   ||objects/functions



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651388] python-xnat: A new XNAT client that exposes XNAT objects/ functions as python objects/functions

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651388

Ankur Sinha (FranciscoD)  changed:

   What|Removed |Added

 CC||sanjay.an...@gmail.com
Summary|python-xnat |python-xnat: A new XNAT
   ||client that exposes XNAT
   ||objects/functions as python
   ||objects/functions



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651766] Review Request: python-pymatreader - Convenient reader for Matlab mat files

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651766



--- Comment #2 from Ankur Sinha (FranciscoD)  ---
(In reply to Robert-André Mauchin from comment #1)
>  - Don't think there should be a 0 here:
> 
> %bcond_without py2 0

Ah, of course not. Corrected.

> 
> Package approved.
> 

Thanks for the quick review Robert. SCM requested:
https://pagure.io/releng/fedora-scm-requests/issue/8937

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651766] Review Request: python-pymatreader - Convenient reader for Matlab mat files

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651766

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zebo...@gmail.com
   Assignee|nob...@fedoraproject.org|zebo...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Robert-André Mauchin  ---
 - Don't think there should be a 0 here:

%bcond_without py2 0

Package approved.

Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "BSD 2-clause "Simplified" License", "Unknown or generated",
 "*No copyright* BSD (unspecified)". 43 files have unknown license.
 Detailed output of licensecheck in /home/bob/packaging/review/python-
 pymatreader/review-python-pymatreader/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package does not own files or directories owned by other packages.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

Python:
[x]: Python eggs must not download any dependencies during the build
 process.
[x]: A package which is used by another package via an egg interface should
 provide egg info.
[x]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Packages MUST NOT have dependencies (either build-time or runtime) on
 packages named with the unversioned python- prefix unless no properly
 versioned package exists. Dependencies on Python packages instead MUST
 use names beginning with python2- or python3- as appropriate.
[x]: Python packages must not contain %{pythonX_site(lib|arch)}/* in %files
[x]: Binary eggs must be removed in %prep

= SHOULD items =

Generic:
[x]: Uses parallel make %{?_smp_mflags} macro.
[-]: Avoid bundling fonts in non-fonts packages.
 Note: Package contains font files
[-]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.

[Bug 1649574] Review Request: jaf - JavaBeans Activation Framework

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649574



--- Comment #5 from Jie Kang  ---
The notice file [1] has:

SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause OR GPL-2.0 WITH
Classpath-exception-2.0

I believe the spec should follow this.

[1] https://github.com/eclipse-ee4j/jaf/blob/master/NOTICE.md

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650633] Review Request: pam_2fa - Second factor authentication for PAM

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650633



--- Comment #11 from Fedora Update System  ---
pam_2fa-1.0-1.fc28 has been pushed to the Fedora 28 testing repository. If
problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-127affe1d7

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650947] Review Request: python-pygiftiio - Python bindings for Gifti

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650947



--- Comment #14 from Fedora Update System  ---
python-pygiftiio-1.0.4-2.fc28 has been pushed to the Fedora 28 testing
repository. If problems still persist, please make note of it in this bug
report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-625dee4559

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651759] Review Request: rust-winapi-x86_64-pc-windows-gnu - Import libraries for the x86_64-pc-windows-gnu target.

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651759

Luis Bazan  changed:

   What|Removed |Added

 CC||bazanlui...@gmail.com
   Assignee|nob...@fedoraproject.org|bazanlui...@gmail.com



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649576] Review Request: fix-info-plist-maven-plugin - Fix Info.plist file generated by p2-maven-plugin during Tycho build

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649576



--- Comment #5 from Jie Kang  ---
The package will need to have a runtime dependency on dd-plist and xz-java as
it does not include the packages in the jar it builds. It looks like the rpm
build system adds the requires automatically but I think it's best to specify
it explicitly.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651388] python-xnat

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651388

Luis Segundo  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651388] python-xnat

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651388

Luis Segundo  changed:

   What|Removed |Added

  Flags|fedora-review+  |fedora-review?



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651388] python-xnat

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651388

Luis Bazan  changed:

   What|Removed |Added

 Depends On|1648728 |
   |(python-progressbar2)   |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1648728
[Bug 1648728] Review Request: python-progressbar2 - A Progressbar library
to provide visual progress to long running operations
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1648728] Review Request: python-progressbar2 - A Progressbar library to provide visual progress to long running operations

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1648728

Luis Bazan  changed:

   What|Removed |Added

 Blocks|1651388 (python-xnat)   |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1651388
[Bug 1651388] python-xnat
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649797] Review Request: imwheel - Mouse Event to Key Event Mapper Daemon

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649797



--- Comment #6 from Robert-André Mauchin  ---
(In reply to sensor.wen from comment #4)
> Robert. I want to know... How long does it take to update rawhide
> repository? From `fedpkg build` to package can be downloaded.
> 
> I use following command to monitor package. :)
> # curl
> https://dl.fedoraproject.org/pub/fedora/linux/development/rawhide/Everything/
> x86_64/os/Packages/g/ | grep go-lib

I don't know what go-lib is, but packages are added every couple of days at
each "compose" which are announced on the devel mailing list:
https://lists.fedoraproject.org/archives/list/de...@lists.fedoraproject.org/message/6IH2EOQCC2TFWXHNJDJJYGZVZSSOQ35V/

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649797] Review Request: imwheel - Mouse Event to Key Event Mapper Daemon

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649797



--- Comment #5 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/imwheel

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650633] Review Request: pam_2fa - Second factor authentication for PAM

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650633



--- Comment #10 from Fedora Update System  ---
pam_2fa-1.0-1.el7 has been pushed to the Fedora EPEL 7 testing repository. If
problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2018-f6b899126d

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651766] Review Request: python-pymatreader - Convenient reader for Matlab mat files

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651766

Ankur Sinha (FranciscoD)  changed:

   What|Removed |Added

 Blocks||1435035, 1276941
   ||(fedora-neuro)
  Alias||python-pymatreader




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1276941
[Bug 1276941] Fedora NeuroImaging and NeuroScience tracking bug
https://bugzilla.redhat.com/show_bug.cgi?id=1435035
[Bug 1435035] python-mne-v0.17 is available
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651766] New: Review Request: python-pymatreader - Convenient reader for Matlab mat files

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651766

Bug ID: 1651766
   Summary: Review Request: python-pymatreader - Convenient reader
for Matlab mat files
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: sanjay.an...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://ankursinha.fedorapeople.org/python-pymatreader/python-pymatreader.spec
SRPM URL:
https://ankursinha.fedorapeople.org/python-pymatreader/python-pymatreader-0.0.17-1.fc29.src.rpm

Description:
A Python module to read Matlab files. This module works with both the old (<
7.3) and the new (>= 7.3) HDF5 based format. The output should be the same for
both kinds of files.

Documentation can be found here: http://pymatreader.readthedocs.io/en/latest/

Fedora Account System Username: ankursinha

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1590423] Review Request: kata-proxy - Kata proxy

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1590423



--- Comment #4 from Jose Carlos Venegas Munoz 
 ---

This is a review *template*. Besides handling the [ ]-marked tests you are
also supposed to fix the template before pasting into bugzilla:
- Add issues you find to the list of issues on top. If there isn't such
  a list, create one.
- Add your own remarks to the template checks.
- Add new lines marked [!] or [?] when you discover new things not
  listed by fedora-review.
- Change or remove any text in the template which is plain wrong. In this
  case you could also file a bug against fedora-review
- Remove the "[ ] Manual check required", you will not have any such lines
  in what you paste.
- Remove attachments which you deem not really useful (the rpmlint
  ones are mandatory, though)
- Remove this text



Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


Issues:
===
- All build dependencies are listed in BuildRequires, except for any that
  are listed in the exceptions section of Packaging Guidelines.
  Note: These BR are not needed: make
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2


= MUST items =

C/C++:
[ ]: Package does not contain kernel modules.
[ ]: Package contains no static executables.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[ ]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[ ]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated", "MIT/X11 (BSD like)", "*No copyright*
 MPL (v2.0)", "BSD (3 clause)", "ISC", "*No copyright* Apache (v2.0)".
 290 files have unknown license. Detailed output of licensecheck in
 /home/jcvenega/1590423-kata-proxy/licensecheck.txt
[ ]: License file installed when any subpackage combination is installed.
[ ]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by: /usr/lib/.build-id(cups-
 client, efivar-libs, qemu-block-gluster, rpm-plugin-selinux, abrt,
 ghc-transformers, libmediaart, libimagequant, giflib, libreoffice-
 graphicfilter, trousers, libdb-utils, tracker-miners, xorg-x11-font-
 utils, PackageKit-command-not-found, libcom_err, libseccomp, p11-kit,
 libabw, libvirt-daemon-driver-storage-iscsi, libmodman, argyllcms,
 parted, gnome-disk-utility, libdvdread, zstd, bluez-libs, libwebp,
 libxslt, system-config-printer-udev, perl-Encode, libverto-libev,
 python2-cryptography, libevdev, libgpg-error, lvm2-libs, perl-Digest-
 SHA, libzstd, qemu-system-ppc-core, libdazzle, avahi, lua-lpeg,
 symlinks, libcdr, cdparanoia-libs, e2fsprogs-libs, pulseaudio-libs,
 libdwarf, lcms2, qemu-guest-agent, annobin, ntfs-3g, libxcb,
 libyubikey, sssd-common-pac, python3-brlapi, openconnect, xcb-util-
 keysyms, m4, expat, libgphoto2, xcb-util, gvfs-fuse, udisks2-iscsi,
 python3-coverage, weechat, anaconda-widgets, http-parser, file-roller-
 nautilus, libeot, systemd-bootchart, dbusmenu-qt, ncurses-libs,
 libidn2, checkpolicy, libreport-plugin-logger, gnome-color-manager,
 fcoe-utils, xorg-x11-xkb-utils, ibus-hangul, perl-IO-Tty,
 NetworkManager-openvpn, libxkbfile, hplip-libs, libwvstreams, ghc-
 binary, freerdp-libs, xmlrpc-c-client, sane-backends-drivers-cameras,
 libwpg, libwpd, enchant2, libvirt-daemon-driver-qemu, perl-Time-HiRes,
 qemu-block-ssh, libwps, gssdp, gnome-calendar, libacl, xorg-x11-drv-
 ati, libSM, policycoreutils, mcelog, abrt-plugin-bodhi,
 libmusicbrainz5, ghc-unix, pptp, sane-backends-drivers-scanners,
 libXxf86misc, foomatic, minizip, colord-gtk, nspr, rpm-build, libvirt-
 daemon-driver-storage-sheepdog, libglvnd-gles, python3-audit, rpm-
 plugin-systemd-inhibit, gstreamer1, gom, gmime30, webkit2gtk3-plugin-
 process-gtk2, dbus-libs, libreoffice-gtk3, libreoffice-gtk2, cairo-
 gobject, gnome-screenshot, libkkc, gutenprint, usermode, qpdf-libs,
 linux-atm-libs, libedit, openssh, subversion-libs, openssl,
 python3-cairo, accountsservice, ghc-containers, isl, hfsplus-tools,
 gssproxy, libblockdev-btrfs, libatomic_ops, gupnp-av, librepo, gtk-
 update-icon-cache, python3-iscsi-initiator-utils, libvirt-daemon-
 driver-interface, NetworkManager, liblangtag, NetworkManager-glib,
 libdatrie, glade-libs, setuptool, libcryptui, guile, pcre-utf16, tree,
 gvfs-goa, pipewire, gnupg, cronie-anacron, libserf, xfsprogs,
 authselect, libcroco, telepathy-glib, plymouth-plugin-label,
 libisoburn, colord-libs, totem-pl-parser, gpgmepp, libgcab1,
 libzhuyin, gvnc, hypervfcopyd, cronie, libexttextcat, nettle,
 libcacard, 

[Bug 1649797] Review Request: imwheel - Mouse Event to Key Event Mapper Daemon

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649797



--- Comment #4 from sensor@gmail.com ---
Robert. I want to know... How long does it take to update rawhide repository?
From `fedpkg build` to package can be downloaded.

I use following command to monitor package. :)
# curl
https://dl.fedoraproject.org/pub/fedora/linux/development/rawhide/Everything/x86_64/os/Packages/g/
| grep go-lib

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649797] Review Request: imwheel - Mouse Event to Key Event Mapper Daemon

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649797

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Robert-André Mauchin  ---
Package approved.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651759] New: Review Request: rust-winapi-x86_64-pc-windows-gnu - Import libraries for the x86_64-pc-windows-gnu target.

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651759

Bug ID: 1651759
   Summary: Review Request: rust-winapi-x86_64-pc-windows-gnu -
Import libraries for the x86_64-pc-windows-gnu target.
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: l...@blackfile.net
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://blackfile.fedorapeople.org/rust/rust-winapi-pcwg/rust-winapi-x86_64-pc-windows-gnu.spec
SRPM URL:
https://blackfile.fedorapeople.org/rust/rust-winapi-pcwg/rust-winapi-x86_64-pc-windows-gnu-0.4.0-1.fc29.src.rpm
Description: Import libraries for the x86_64-pc-windows-gnu target. 
Fedora Account System Username:blackfile

Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=31020644

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651397] Review Request: python-efel - Electrophys Feature Extraction Library

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651397



--- Comment #8 from Fedora Update System  ---
python-efel-3.0.22-1.fc29 has been submitted as an update to Fedora 29.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-1531eeb1bd

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651397] Review Request: python-efel - Electrophys Feature Extraction Library

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651397

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #7 from Fedora Update System  ---
python-efel-3.0.22-1.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-91dbe4e2b6

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650633] Review Request: pam_2fa - Second factor authentication for PAM

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650633

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
pam_2fa-1.0-1.fc29 has been pushed to the Fedora 29 testing repository. If
problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-18c38b185e

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650947] Review Request: python-pygiftiio - Python bindings for Gifti

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650947

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #13 from Fedora Update System  ---
python-pygiftiio-1.0.4-2.fc29 has been pushed to the Fedora 29 testing
repository. If problems still persist, please make note of it in this bug
report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2018-aa97f027d7

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1648728] Review Request: python-progressbar2 - A Progressbar library to provide visual progress to long running operations

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1648728

Luis Bazan  changed:

   What|Removed |Added

 Blocks||1651388 (python-xnat)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1651388
[Bug 1651388] python-xnat
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651388] python-xnat

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651388

Luis Bazan  changed:

   What|Removed |Added

 Depends On||1648728
   ||(python-progressbar2)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1648728
[Bug 1648728] Review Request: python-progressbar2 - A Progressbar library
to provide visual progress to long running operations
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1648728] Review Request: python-progressbar2 - A Progressbar library to provide visual progress to long running operations

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1648728

Luis Bazan  changed:

   What|Removed |Added

 Blocks|1651388 (python-xnat)   |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1651388
[Bug 1651388] python-xnat
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651388] python-xnat

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651388

Luis Bazan  changed:

   What|Removed |Added

 Depends On|1648728 |
   |(python-progressbar2)   |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1648728
[Bug 1648728] Review Request: python-progressbar2 - A Progressbar library
to provide visual progress to long running operations
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651739] Review Request: caribou EPEL7 - EPEL7 branch of caribou ( cinnamon prereq)

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651739

Pat Riehecky  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651739] New: Review Request: caribou EPEL7 - EPEL7 branch of caribou (cinnamon prereq)

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651739

Bug ID: 1651739
   Summary: Review Request: caribou EPEL7 - EPEL7 branch of
caribou (cinnamon prereq)
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: riehe...@fnal.gov
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://src.fedoraproject.org/rpms/caribou.git#ae1aaf05d779ad77727e17e2ca05b55d6235
SRPM URL:
https://copr.fedorainfracloud.org/coprs/jcpunk/caribou-epel7/build/827089/
Description: This is a direct fork of the Fedora 29 package without changes
Fedora Account System Username: jcpunk

The EPEL7 branch of caribou is required to reintroduce cinnamon to EPEL7.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1612092] Review Request: mailman3 - The GNU mailing list manager

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1612092



--- Comment #18 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/mailman3

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1619458] Review Request: python-pyphi - A library for computing integrated information

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1619458



--- Comment #12 from Fedora Update System  ---
python-pyphi-1.1.0-2.fc29 has been submitted as an update to Fedora 29.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-a352df747a

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1619458] Review Request: python-pyphi - A library for computing integrated information

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1619458

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #11 from Fedora Update System  ---
python-pyphi-1.1.0-2.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-c8dd8701a6

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1612092] Review Request: mailman3 - The GNU mailing list manager

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1612092

Neal Gompa  changed:

   What|Removed |Added

 Status|NEW |POST
  Flags|fedora-review?  |fedora-review+



--- Comment #17 from Neal Gompa  ---
Package looks good to me.

PACKAGE APPROVED.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1645766] Review Request: python-azure-devtools - Devtools for Azure SDK and CLI for Python

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1645766

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags|fedora-review?  |fedora-review+



--- Comment #8 from Robert-André Mauchin  ---
Package approved.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1228503] Review Request: springframework-amqp - Support for Spring programming model with AMQP

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1228503
Bug 1228503 depends on bug 1231430, which changed state.

Bug 1231430 Summary: Missing jinterface maven metadata
https://bugzilla.redhat.com/show_bug.cgi?id=1231430

   What|Removed |Added

 Status|CLOSED  |NEW
 Resolution|EOL |---



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1650987] Review Request: libgenht - A simple generic hash table implementation in C

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1650987



--- Comment #4 from Fabio Valentini  ---
Initial comments:

1) Package name

Did you consider naming the main package just "genht", and adding a "-libs"
sub-package? The upstream project is called "genht", after all. That would also
make defining "generic_name" unnecessary, since you could just use "%{name}"
everywhere, instead.


2) Build system problems

a) The upstream project seems to have some isses with their build system. I
don't think it honors build flags from the environment (e.g. CFLAGS, LDFLAGS) -
but you didn't set them in the .spec file, either.

b) You should use the "%set_build_flags" macro (see [0]) at the top of %build
(it sets CFLAGSm LDFLAGS, etc. to the fedora defaults), and change "make" to
"%make_build", and "make install" to "%make_install" - those macros set some
important flags for fedora builds (including DESTDIR).

c) The upstream build system also doesn't define an SONAME for the library
("-Wl,-soname,libfoo.so.1" flag) - that's missing from the upstream Makefile as
well, and will throw RPM off - right now, it renders the "-devel" package
uninstallable. If upstream won't introduce an SONAME, you will have to add one
for the fedora build.

d) Don't include the static library in your package (and drop the "-static"
sub-package), unless you are sure that other packages will want to use it, and
have a good reason to do so.

e) The Makefile also defines a binary, is that not used for anything?

I'm curious if you could work with the upstream project to add support for a
proper build system ("porting" the Makefile to meson could probably be done
within an hour, and it would make it more portable than a crusty Makefile).


3) .spec issues

You don't need to use macros in summary, description, etc. It just makes the
.spec less readable, IMO. But that's a matter of taste, I guess.

I also think you can just remove all the comments. They are pretty redundant
with the code below them. If you want to keep them, move the ones above %prep
and %install into the sections, otherwise RPM will probably treat them as
belonging to the previous section.

a) By using a glob for the shared library ("%{_libdir}/libgenht.so.*") in line
54, which conceals the library version, you increase the likelihood of
unintended breakage because of SONAME bumps. Use "%{_libdir}/libgenht.so.1*")
instead, as documented in the Guidelines.

b) The summary should not start with "A" and not end with ".". Change it to:
"Simple generic hash table implementation in C".

c) The "-devel" sub-package must depend on the package containing the shared
library (right now, libgenht; if you decide to change the package name to
"genht", then "genht-libs") like this:

  Requires: %{name}%{?_isa} = %{version}-%{release} # for libgenht
or this:
  Requires: %{name}-libs%{?_isa} = %{version}-%{release} # for genht-libs

d) What does the comment "# Untar the correct  tar.gz  file" mean? I think you
can just drop it. (The benefit of renaming the main package to "genht" would be
that you can drop the "-n" argument from the "%autosetup" call.)

e) The following entries in %files lists are equivalent:

  %dir %{_includedir}/%{generic_name}
  %{_includedir}/%{generic_name}/*
and
  %{_includedir}/%{generic_name}/

This is probably just a stylistic issue, so you decide which one you'd like to
use.


[0]:
https://src.fedoraproject.org/rpms/redhat-rpm-config/blob/master/f/buildflags.md

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1636019] Review Request: java-runtime-decompiler - a tool for extraction and decompilation of bytecode from JVM memory

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1636019



--- Comment #17 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/java-runtime-decompiler

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651397] Review Request: python-efel - Electrophys Feature Extraction Library

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651397



--- Comment #6 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/python-efel

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1619458] Review Request: python-pyphi - A library for computing integrated information

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1619458



--- Comment #10 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/python-pyphi

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1590428] Review Request: kata-shim - Kata shim

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1590428



--- Comment #6 from Lokesh Mandvekar  ---
Filed Bug 1650620 since fedora-review doesn't seem to be working for me.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649797] Review Request: imwheel - Mouse Event to Key Event Mapper Daemon

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649797



--- Comment #2 from sensor@gmail.com ---
SRPM:
https://copr-be.cloud.fedoraproject.org/results/mosquito/deepin/fedora-rawhide-x86_64/00827039-imwheel/imwheel-1.0.0-0.1.pre12.fc30.src.rpm

SPEC:
https://copr-be.cloud.fedoraproject.org/results/mosquito/deepin/fedora-rawhide-x86_64/00827039-imwheel/imwheel.spec

Fixed ~

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1612092] Review Request: mailman3 - The GNU mailing list manager

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1612092



--- Comment #16 from Aurelien Bompard  ---
I added the license file. The rpm does not build at the moment because it
requires a more recent aiosmtpd. I've updated it in rawhide but it's not
available just yet.

Spec URL: https://abompard.fedorapeople.org/reviews/mailman3/mailman3.spec
SRPM URL:
https://abompard.fedorapeople.org/reviews/mailman3/mailman3-3.2.0-1.fc29.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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1626134] Review Request: fips - OpenGL-based FITS image viewer

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1626134

Peter Lemenkov  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2018-11-20 08:55:11



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649874] Review Request: hpx - a general purpose C++ runtime system for parallel and distributed applications of any scale

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649874



--- Comment #17 from Fedora Update System  ---
hpx-1.2.0-3.fc28 has been submitted as an update to Fedora 28.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-8367209bd8

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649874] Review Request: hpx - a general purpose C++ runtime system for parallel and distributed applications of any scale

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649874

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #16 from Fedora Update System  ---
hpx-1.2.0-3.fc29 has been submitted as an update to Fedora 29.
https://bodhi.fedoraproject.org/updates/FEDORA-2018-a9b53adbdc

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1645766] Review Request: python-azure-devtools - Devtools for Azure SDK and CLI for Python

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1645766



--- Comment #7 from Mohamed El Morabity  ---
Spec URL:
https://melmorabity.fedorapeople.org/packages/python-azure-devtools/python-azure-devtools.spec
SRPM URL:
https://melmorabity.fedorapeople.org/packages/python-azure-devtools/python-azure-devtools-1.0.0-2.fc30.src.rpm

Changes:
- Don't glob everything under the Python sitelib directory

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651397] Review Request: python-efel - Electrophys Feature Extraction Library

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651397



--- Comment #5 from Ankur Sinha (FranciscoD)  ---
Thanks for the review Luis. SCM requested:
https://pagure.io/releng/fedora-scm-requests/issue/8927

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1651397] Review Request: python-efel - Electrophys Feature Extraction Library

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1651397

Luis Segundo  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
  Flags||fedora-review+



--- Comment #4 from Luis Segundo  ---
(In reply to Ankur Sinha (FranciscoD) from comment #3)
> (In reply to Luis Segundo from comment #2)

> I think this is a false positive. 
Hello Ankur, if it makes sense

Package approved!

Cheers,

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649573] Review Request: directory-maven-plugin - Establish locations for files in multi-module builds

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649573

Severin Gehwolf  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649573] Review Request: directory-maven-plugin - Establish locations for files in multi-module builds

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649573



--- Comment #14 from Severin Gehwolf  ---
APPROVED.

Please fix the 'ASL 2.0' vs. 'ASL-2.0' issue before you push.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org


[Bug 1649573] Review Request: directory-maven-plugin - Establish locations for files in multi-module builds

2018-11-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1649573



--- Comment #13 from Severin Gehwolf  ---
Issues:

- Please use 'ASL 2.0' instead of 'ASL-2.0' in
  License field.


Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Package is
 ASL 2.0
[?]: License file installed when any subpackage combination is installed.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
 names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
 one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
 provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

Java:
[x]: Bundled jar/class files should be removed before build
[x]: Packages have proper BuildRequires/Requires on jpackage-utils
 Note: Maven packages do not need to (Build)Require jpackage-utils. It
 is pulled in by maven-local
[x]: Javadoc documentation files are generated and included in -javadoc
 subpackage
[x]: Javadoc subpackages should not have Requires: jpackage-utils
[x]: Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlink)

Maven:
[x]: If package contains pom.xml files install it (including metadata) even
 when building with ant
[x]: POM files have correct Maven mapping
[x]: Maven packages should use new style packaging
[x]: Old add_to_maven_depmap macro is not being used
[x]: Packages DO NOT have Requires(post) and Requires(postun) on jpackage-
 utils for %update_maven_depmap macro
[x]: Package DOES NOT use %update_maven_depmap in %post/%postun
[x]: Packages use .mfiles file list instead of %{_datadir}/maven2/poms

= SHOULD items =

Generic:
[x]: If the source package does not include license text(s) as a separate
 file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
 directory-maven-plugin-javadoc
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from 

  1   2   >