[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #14 from Steve Traylen 2011-04-11 04:02:10 EDT --- Sorry, any comment. Steve -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Jason Tibbitts changed: What|Removed |Added Status|ASSIGNED|NEW AssignedTo|ti...@math.uh.edu |nob...@fedoraproject.org Flag|fedora-review? | --- Comment #15 from Jason Tibbitts 2011-04-11 11:14:06 EDT --- Looks like you need a review before I'll be able to provide one. Hopefully you can find someone else to do this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Jerry James changed: What|Removed |Added Status|NEW |ASSIGNED CC||loganje...@gmail.com AssignedTo|nob...@fedoraproject.org|loganje...@gmail.com Flag||fedora-review? --- Comment #16 from Jerry James 2011-04-28 17:24:31 EDT --- I'll take this review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619355
--- Comment #17 from Jerry James 2011-04-28 18:35:18 EDT
---
I have a couple of observations before getting to the standard review items.
First, I saw a lot of warnings about code that breaks strict aliasing rules. I
suggest adding -fno-strict-aliasing to CFLAGS, else you may see some subtly
weird behavior at runtime.
Second, I don't understand why both lapack-devel and atlas-devel are BRs. The
atlas package provides a lapack replacement (%{_libdir}/atlas/liblapack.so.3).
The only ELF object I can find that is linked to liblapack,
linalg/lapack_lite.so, is indeed linked to %{_libdir}/atlas/liblapack.so.3.
Why is lapack-devel a BR?
Third, totally trivial, but 's/in in/be in/' on line 118 of the spec file.
rpmlint output:
python26-numpy.x86_64: W: no-soname
/usr/lib64/python2.6/site-packages/numpy/lib/_compiled_base.so
python26-numpy.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/python2.6/site-packages/numpy/lib/_compiled_base.so
/lib64/libpthread.so.0
python26-numpy-devel.x86_64: W: no-documentation
python26-numpy-f2py.x86_64: W: manpage-not-compressed bz2
/usr/share/man/man1/f2py2.6.1.gz
python26-numpy-f2py.x86_64: E: devel-dependency python26-devel
python26-numpy-f2py.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/python2.6/site-packages/numpy/f2py/tests/src/array_from_pyobj/wrapmodule.c
python26-numpy-f2py.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/python2.6/site-packages/numpy/f2py/src/fortranobject.h
python26-numpy-f2py.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/python2.6/site-packages/numpy/f2py/src/fortranobject.c
python26-numpy-f2py-tests.x86_64: E: devel-dependency python26-devel
python26-numpy-f2py-tests.x86_64: W: no-documentation
python26-numpy-f2py-tests.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/python2.6/site-packages/numpy/f2py/tests/src/array_from_pyobj/wrapmodule.c
python26-numpy-tests.x86_64: W: no-documentation
6 packages and 1 specfiles checked; 2 errors, 10 warnings.
#1, #3, #10, #12: appear inconsequential.
#2: it would be good to not link lib/_compiled_base.so against -lpthread if
that isn't impossible to arrange.
#4: rpmlint has apparently gone insane, since ALL man pages are gz compressed
#5-#8: these appear to be a consequence of python26-numpy-f2py being a
"stealth" devel package
#9, #11: ditto for python26-numpy-f2py-tests
So it looks like only #2 needs attention. You only get that warning if you run
rpmlint on the installed package, by the way; checking the binary rpm won't
trigger this warning.
+: OK
-: Must fix
=: Needs attention (but not a review blocker)
N: Not applicable
MUST items:
[=] rpmlint output: shown above
[+] Package name follows the Package Naming Guidelines
[+] Spec file name matches the base package name
[+] Package meets the Packaging Guidelines
[+] Package has a Fedora-approved license
[=] License field matches the actual license
[+] Iff the license appears in a file, that file is in %doc.
[+] Spec file is in American Engligh
[+] Spec file is legible
[+] Sources match the upstream sources: md5sum is
376ef150df41b5353944ab742145352d
[+] Source RPM builds on a least one arch: x86_64
[N] Appropriate use of ExcludeArch
[+] BuildRequires for all build-time dependencies
[N] Proper handling of locales
[N] Proper use of ldconfig in %post and %postun
[+] No bundled copies of system libraries
[N] No relocatable packages
[+] Package owns all directories it creates
[+] No duplicates in %files
[+] Appropriate permissions on files
[+] Consistent use of macros
[+] Package contains code or permissible content
[+] Large documentation goes into a -doc subpackage
[+] No runtime dependencies in %doc
[+] Header files in -devel
[N] Static libraries in -static
[N] If a shared library has a suffix, then a .so symlink is in -devel
[=] -devel has a fully versioned Requires on the main package: this should be
%{name}%{?_isa} in Fedora, but I don't know if that is the case in EPEL
[+] No libtool archives in the binary packages
[N] GUI applications install a desktop file
[+] Does not own any files/dirs owned by other packages
[+] All filenames are valid UTF-8
SHOULD items:
[N] Query upstream to include a missing license file
[N] Description and summary should contain translations, if available
[+] Package builds in mock: tried epel-5-x86_64
[=] Package builds on all supported arches: only tried x86_64
[=] Package functions as described: minimal testing only
[N] Sane scriptlets
[+] All subpackages require the main package
[N] Good pkgconfig file placement
[+] Package dependencies instead of file dependencies
[+] Package has man pages for binaries/scripts
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #18 from Steve Traylen 2011-04-30 17:32:43 EDT --- Hi Jerry, Thanks for the comments so far. New packages: http://cern.ch/straylen/rpms/python26-numpy/python26-numpy-1.5.1-3.el5.src.rpm http://cern.ch/straylen/rpms/python26-numpy/python26-numpy.spec The easy ones: * The isa tags indeed do nothing on EPEL5 * s/in in/be in/ done. * Added -fno-strict-aliasing. The other ones: Concerning the liblapack in atlas and lapack: I've been reading around the history of this, good news is we can drop the requirement on lapack since indeed everything and more is also in atlas. This also allows the specification of library locations to be dropped in the %build and %install sections as well. They were there from a time after EPEL atlas when lapack in atlas was not including all of lapack, it's just not needed for EPEL5 versions of these packages. The current situation with liblapack is definitely ugly and is detailed in the bug #478856. Hopefully unrelated to this review I added a comment that there may now be a way out of this situation but the fix will come from atlas and lapack and will take time if it comes at all. I'll look at the /usr/lib64/python2.6/site-packages/numpy/lib/_compiled_base.so /lib64/libpthread.so.0 shortly. I noticed you added a look more for the licensing, was it something in particular, I will look more anyway. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619355
--- Comment #19 from Jerry James 2011-05-03 14:05:28 EDT
---
Argh, I didn't copy in my notes about licenses. Sorry about that. I was just
poking around in the sources, and found a few things with licenses other than
BSD. Here are some things that need to be looked at:
numpy/core/_mx_datetime_parser.py carries the eGenix public license 1.1.0.
numpy/lib/utils.py contains a SafeEval class which carries the Python license.
numpy/random/mtrand/{distributions.c,distributions.h,mtrand.pyx,randomkit.h}
carry MIT licenses.
numpy/random/mtrand/randomkit.c is a mixture of BSD and MIT licenses.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #20 from Steve Traylen 2011-05-24 14:48:54 EDT --- http://cern.ch/straylen/rpms/python26-numpy/python26-numpy-1.5.1-4.el5.src.rpm http://cern.ch/straylen/rpms/python26-numpy/python26-numpy.spec These change the license field basically and add your notes plus a bit more describing the license situation. Concerning the eGenix license this is a modified python license but should be approved of course. I'll do that now. As for the: /usr/lib64/python2.6/site-packages/numpy/lib/_compiled_base.so /lib64/libpthread.so.0 I can't see anything obvious to change here but nor can I find where it picks it up from. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #21 from Jerry James 2011-05-26 21:54:56 EDT --- Don't worry aobut the _compiled_base.so thing. It's a minor detail. I saw your message about the eGenix license, but haven't seen a response. Poke me if I missed it. Once that's approved, it looks to me as though this package is ready. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Jerry James changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #22 from Jerry James 2011-06-07 17:20:46 EDT --- The reply says to call that license "eGenix". I'll trust you to put that into the license field when you import the package. Since everything else is ready to go, I'll approve this package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Steve Traylen changed: What|Removed |Added Flag||fedora-cvs? --- Comment #23 from Steve Traylen 2011-06-08 18:32:19 EDT --- New Package SCM Request === Package Name: python26-numpy Short Description: A fast multi-dimensional array facility for Python Owners: stevetraylen Branches: el5 InitialCC: Thanks a lot for the review and for all the comments from previous folk as well. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #24 from Jon Ciesla 2011-06-08 21:24:23 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #25 from Fedora Update System 2011-06-16 14:21:51 EDT --- python26-numpy-1.5.1-5.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/python26-numpy-1.5.1-5.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #26 from Fedora Update System 2011-06-21 13:08:25 EDT --- python26-numpy-1.5.1-5.el5 has been pushed to the Fedora EPEL 5 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #27 from Fedora Update System 2011-07-06 14:03:01 EDT --- python26-numpy-1.5.1-5.el5 has been pushed to the Fedora EPEL 5 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||python26-numpy-1.5.1-5.el5 Resolution||ERRATA Last Closed||2011-07-06 14:03:10 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Jason Tibbitts changed: What|Removed |Added Status Whiteboard||BuildFails Bug 619355 depends on bug 574531, which changed state. Bug 574531 Summary: Review Request: python26-nose - The "nose" testing package for the python26 EPEL5 package https://bugzilla.redhat.com/show_bug.cgi?id=574531 What|Old Value |New Value Status|NEW |MODIFIED Status|MODIFIED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #1 from Jason Tibbitts 2010-12-03 14:45:24 EST --- This seems to fail to build for me in a Centos5 buildroot in mock. Maybe I'm doing something wrong, but my understanding is that your pyver macro will break srpm parsing because it is empty before python is installed: Executing command: ['bash', '--login', '-c', 'rpmbuild -bs --target x86_64 --nodeps builddir/build/SPECS/python26-numpy.spec'] sh: /usr/bin/python2.6: No such file or directory sh: /usr/bin/python2.6: No such file or directory error: line 26: Version required: Requires: python(abi) = Perhaps it would be better to simply require 2.6 (since it already kind of hardcoded into the name of the package) and dispense with the pyver macro altogether? A scratch build is at http://koji.fedoraproject.org/koji/taskinfo?taskID=2642342 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #2 from Steve Traylen 2010-12-03 15:11:54 EST --- I've seen the message sh: /usr/bin/python2.6: No such file or directory but in the past it has not been fatal. I agree given hard coding of the package name hardcoding the python package name makes sense. Will post new ones soon. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Steve Traylen changed: What|Removed |Added Status Whiteboard|BuildFails | --- Comment #3 from Steve Traylen 2010-12-13 17:39:51 EST --- New packages, these build in mock okay for me. http://cern.ch/straylen/rpms/python26-numpy/python26-numpy-1.5.1-1.el5.src.rpm http://cern.ch/straylen/rpms/python26-numpy/python26-numpy.spec rpmlint shows a lot of devel-file-in-non-devel-package. I'm not sure how far it makes sense to deviate from current packing of numpy as it stands in Fedora. It may be that f2py uses these "dev" files at runtime as well. Steve -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #4 from Kevin Fenzi 2010-12-20 17:28:36 EST --- How does this relate to 601891 ? Is it pretty much decided to ship a different package for this? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Steve Traylen changed: What|Removed |Added Blocks|601891 | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #5 from Steve Traylen 2010-12-22 01:22:11 EST --- I've closed #601891 since that one is obsoleted by this one. That was request to add a python26-numpy subpackage to the existing numpy package in EPEL5 and that proved to be a bad idea. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #6 from Steve Traylen 2011-01-20 03:29:17 EST --- Hi Kevin, if you have time to comment that would be great. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #7 from Jason Tibbitts 2011-01-20 10:48:58 EST --- Actually I wrote up most of a review that I just need to paste in here. Hopefully I'll find sufficient time today to get that done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Jason Tibbitts changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|ti...@math.uh.edu Flag||fedora-review? --- Comment #8 from Jason Tibbitts 2011-01-26 18:20:31 EST --- Turns out that somehow I lost that saved review when I was cleaning up my in-progress reviews directory. Ugh. Rather than try to retype it all now, I'll go over the basic issues: All of those devel-file-in-non-devel-package complaints: These are present in the base numpy package, but this package doesn't get a pass just because the base package may be doing something dumb. I asked the current maintainer of the numpy package but he did not know what those files were actually used for. Probably best if you figure that out, given that you're going to be maintaining this. There are some additional complaints about the f2py subpackage. My understanding is that one actually does call the compiler, but then there's also the question of why stuff in the tests directory is in the final package. There are several private-shared-object-provides warnings that I'd suggest filtering if el5 had the filtering infrastructure. There's one no-soname complaint which I think is inconsequential. Otherwise I can't really complain about the package. I'll whip up a full review if we can get those questions answered. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #9 from Steve Traylen 2011-02-01 13:33:31 EST --- After talking to people and trying a few things out the header files should be in a -devel. They are used for compiling C extensions to numpy. New packages shortly. Steve -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #11 from Steve Traylen 2011-02-05 13:33:39 EST --- Not > http://cern.ch/straylen/rpms/python26-numpy/python26-numpy-1.5.1-1.el5.src.rpm > http://cern.ch/straylen/rpms/python26-numpy/python26-numpy.spec but http://cern.ch/straylen/rpms/python26-numpy/python26-numpy-1.5.1-2.el5.src.rpm http://cern.ch/straylen/rpms/python26-numpy/python26-numpy.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #10 from Steve Traylen 2011-02-05 13:32:54 EST --- New packages , the tests and devel files are both moved out to separate packages. http://cern.ch/straylen/rpms/python26-numpy/python26-numpy-1.5.1-1.el5.src.rpm http://cern.ch/straylen/rpms/python26-numpy/python26-numpy.spec When python2.6 -c "import pkg_resources, numpy ; numpy.test()" is executed without the -tests package installed then it successfully runs no tests which is probably the correct thing to do. With -tests installed they are all executed. Concerning the f2py package then the work flow for this is roughly 1. Write a fortran.f subroutine. 2. Compile to a .so with f2py -c fortran.f -m fortran 3. In python the .so can be loaded with "import fortran". I could split python-numpy-f2py into a runtime and devel package so that only step (3) would work without the devel package, if that is done then /usr/bin/f2py ends up in a -devel package which is possibly odd in itself. I am happy as it is but there may be comment. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #12 from Steve Traylen 2011-03-01 17:51:29 EST --- Hi, any comment? Steve -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 --- Comment #13 from Jason Tibbitts 2011-03-24 15:57:00 EDT --- I have to apologize; by the beginning of February I completely lost all time to pursue package reviews. Hopefully that's coming to an end and I'll have a chance to catch back up with everything soon. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Steve Traylen changed: What|Removed |Added Blocks||601891 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 619355] Review Request: python26-numpy - A fast multidimensional array facility for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=619355 Steve Traylen changed: What|Removed |Added Depends on||574531 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
