[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-12-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2015-12-09 15:51:24



-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-12-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211



--- Comment #10 from Fedora Update System  ---
python-gradunwarp-1.0.2-3.fc23 has been pushed to the Fedora 23 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-12-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211
Bug 1279211 depends on bug 1276871, which changed state.

Bug 1276871 Summary: Review Request: python-nibabel - Python package to access 
a cacophony of neuro-imaging file formats
https://bugzilla.redhat.com/show_bug.cgi?id=1276871

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA



-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #9 from Fedora Update System  ---
python-gradunwarp-1.0.2-3.fc23 has been pushed to the Fedora 23 testing
repository. If problems still persist, please make note of it in this bug
report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update python-gradunwarp'
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2015-203b00ea80

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211



--- Comment #8 from Fedora Update System  ---
python-gradunwarp-1.0.2-3.fc23 has been submitted as an update to Fedora 23.
https://bodhi.fedoraproject.org/updates/FEDORA-2015-203b00ea80

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211



--- Comment #7 from Till Maas  ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/python-gradunwarp

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211



--- Comment #6 from Igor Gnatenko  ---
Spec URL:
https://ignatenkobrain.fedorapeople.org/neurofedora/python-gradunwarp.spec
SRPM URL:
https://ignatenkobrain.fedorapeople.org/neurofedora/python-gradunwarp-1.0.2-2.fc24.src.rpm

I had some minutes to add py3 subpkg (with sending patches to upstream) and I
also fixed issues found by you.

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211



--- Comment #4 from Igor Gnatenko  ---
> the script in /usr/bin/ should not be suffixed
> with -2 or -2.7.

it is not.

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Zbigniew Jędrzejewski-Szmek  ---
Oh, sorry, too many reviews in parallel :)

Pasting rmplint and requires/provides for reference:

$ rpmlint python-gradunwarp
python-gradunwarp.x86_64: W: spelling-error %description -l en_US unwarp ->
unwrap, unwary
python-gradunwarp.x86_64: W: spelling-error %description -l en_US
inhomogenities -> homogenizes
python-gradunwarp.x86_64: E: non-standard-executable-perm
/usr/lib64/python2.7/site-packages/gradunwarp/core/interp3_ext.so 775
python-gradunwarp.x86_64: E: non-standard-executable-perm
/usr/lib64/python2.7/site-packages/gradunwarp/core/legendre_ext.so 775
python-gradunwarp.x86_64: E: non-standard-executable-perm
/usr/lib64/python2.7/site-packages/gradunwarp/core/transform_coordinates_ext.so
775
python-gradunwarp.x86_64: W: no-manual-page-for-binary gradient_unwarp
1 packages and 0 specfiles checked; 3 errors, 3 warnings.

All OK.

Requires:
/usr/bin/env
/usr/bin/python2
libc.so.6()(64bit)
libc.so.6(GLIBC_2.2.5)(64bit)
libc.so.6(GLIBC_2.3.4)(64bit)
libc.so.6(GLIBC_2.4)(64bit)
libpthread.so.0()(64bit)
libpython2.7.so.1.0()(64bit)
numpy
python(abi) = 2.7
python2-nibabel
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PartialHardlinkSets) <= 4.0.4-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(PayloadIsXz) <= 5.2-1
rtld(GNU_HASH)
scipy

Provides:
python-gradunwarp = 1.0.2-1.fc24
python-gradunwarp(x86-64) = 1.0.2-1.fc24

Oh, so this uncovers another issue:
- please add Provides: python2-gradunwrap when importing.

Package is 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211



--- Comment #3 from Zbigniew Jędrzejewski-Szmek  ---
It has the same problem as other packages: the script in /usr/bin/ should not
be suffixed
with -2 or -2.7.

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211



--- Comment #2 from Igor Gnatenko  ---
Can you approve and I will fix in import stage if there are no other issues?

Thanks for 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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||zbys...@in.waw.pl
   Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl
  Flags||fedora-review?



--- Comment #1 from Zbigniew Jędrzejewski-Szmek  ---
#!/usr/bin/env /usr/bin/python2 → #!/usr/bin/python2.

Looks good otherwise.

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1279211] Review Request: python-gradunwarp - Gradient Unwarping

2015-11-08 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1279211

Igor Gnatenko  changed:

   What|Removed |Added

 Blocks||1276941 (fedora-neuro)
 Depends On||1276871 (python-nibabel)
  Alias||python-gradunwarp




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1276871
[Bug 1276871] Review Request: python-nibabel - Python package to access a
cacophony of neuro-imaging file formats
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
https://admin.fedoraproject.org/mailman/listinfo/package-review