[Bug 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
https://bugzilla.redhat.com/show_bug.cgi?id=663956 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
https://bugzilla.redhat.com/show_bug.cgi?id=663956 --- Comment #18 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
https://bugzilla.redhat.com/show_bug.cgi?id=663956 Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl changed: What|Removed |Added CC||zbys...@in.waw.pl Flags|fedora-cvs+ |fedora-cvs? --- Comment #17 from Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl --- Package Change Request == Package Name: python-numexpr New Branches: epel7 Owners: tnorth zbyszek InitialCC: -- You are receiving this mail because: 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
https://bugzilla.redhat.com/show_bug.cgi?id=663956 Orion Poplawski or...@cora.nwra.com changed: What|Removed |Added CC||or...@cora.nwra.com --- Comment #16 from Orion Poplawski or...@cora.nwra.com --- Thibault - would you be willing to maintain this in EPEL7? -- You are receiving this mail because: 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956 --- Comment #14 from Jon Ciesla l...@jcomserv.net 2011-10-31 08:11:07 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956 Thibault North thibault.no...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Last Closed||2011-10-31 10:38:01 --- Comment #15 from Thibault North thibault.no...@gmail.com 2011-10-31 10:38:01 EDT --- Builds done, updates pending. Thanks. -- 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956 --- Comment #11 from Thibault North thibault.no...@gmail.com 2011-10-30 15:17:57 EDT --- - Fixed permissions for .py and .so files - Add %check section after %build - Use %global instead of %define - Filed a bug upstream about licence boilerplates - Removed %clean section, defattr and buildroot definition Thank you for that 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956 --- Comment #12 from Thibault North thibault.no...@gmail.com 2011-10-30 15:24:24 EDT --- And new SRPM... http://tnorth.fedorapeople.org/python-numexpr-1.4.2-2.fc15.src.rpm -- 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956 Thibault North thibault.no...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #13 from Thibault North thibault.no...@gmail.com 2011-10-30 15:26:52 EDT --- New Package SCM Request === Package Name: python-numexpr Short Description: Fast numerical array expression evaluator for Python and NumPy Owners: tnorth Branches: f14 f15 f16 el6 -- 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956
Jussi Lehtola jussi.leht...@iki.fi changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|jussi.leht...@iki.fi
Flag||fedora-review+
--- Comment #10 from Jussi Lehtola jussi.leht...@iki.fi 2011-10-29 07:37:31
EDT ---
rpmlint output:
python-numexpr.src: W: spelling-error Summary(en_US) evaluator - evaluate,
elevator
python-numexpr.src: W: spelling-error %description -l en_US runtime - run
time, run-time, rudiment
python-numexpr.src: W: invalid-url Source0:
http://numexpr.googlecode.com/files/numexpr-1.4.2.tar.gz HTTP Error 404: Not
Found
python-numexpr.x86_64: W: spelling-error Summary(en_US) evaluator - evaluate,
elevator
python-numexpr.x86_64: W: spelling-error %description -l en_US runtime - run
time, run-time, rudiment
3 packages and 0 specfiles checked; 0 errors, 5 warnings.
These are OK (the URL does work).
**
About the chmod - files in %{python_sitelib} and %{python_sitearch} should have
644 permissions (755 for .so libraries), since they're normally not meant for
execution, but instead are imported by other programs.
**
Since there are benchmarks included, you must run it in the %check phase to
verify correct functionality, e.g.
%check
libdir=`ls build/|grep lib`
export PYTHONPATH=`pwd`/build/$libdir
python bench/timing.py
**
MUST: The package does not yet exist in Fedora. The Review Request is not a
duplicate. OK
MUST: The spec file for the package is legible and macros are used
consistently. NEEDSWORK
- Please use %global instead of %define.
http://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define
MUST: The package must be named according to the Package Naming Guidelines. OK
MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines. OK
MUST: The License field in the package spec file must match the actual license.
OK
- LICENSE.txt defines a MIT license.
- License boilerplates are missing from the source code files. Please ask
upstream to add them.
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. OK
139115cc196dc57a66b2eb30cd3e80a0 numexpr-1.4.2.tar.gz
139115cc196dc57a66b2eb30cd3e80a0 ../SOURCES/numexpr-1.4.2.tar.gz
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. N/A
MUST: Optflags are used and time stamps preserved. OK
MUST: Packages containing shared library files must call ldconfig. N/A
MUST: A package must own all directories that it creates or require the package
that owns the directory. OK
MUST: Files only listed once in %files listings. OK
MUST: Debuginfo package is complete. OK
MUST: Permissions on files must be set properly. OK
- %defattr(-,root,root) is somewhat obsolete, usually %defattr(-,root,root,-)
is used.
- You can also drop this altogether along with the cleaning stuff and BuildRoot
tag, since they're defaulted in current versions of rpm.
MUST: Large documentation files must go in a -doc subpackage. N/A
MUST: All relevant items are included in %doc. Items in %doc do not affect
runtime of application. OK
MUST: Header files must be in a -devel package. N/A
MUST: Static libraries must be in a -static package. N/A
MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. N/A
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned, architecture dependent dependency. N/A
MUST: Packages does not contain any .la libtool archives. N/A
MUST: Desktop files are installed properly. N/A
MUST: No file conflicts with other packages and no general names. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. OK
SHOULD: The package builds in mock. OK
EPEL: Clean section exists. OK
EPEL: Buildroot cleaned before install. OK
EPEL: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. N/A
**
Add the check phase and switch the define to global before git import. This
package has been
APPROVED
Please also remember to ask upstream to add license boilerplates to the source
files.
--
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
[Bug 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956 --- Comment #9 from Thibault North thibault.no...@gmail.com 2011-10-20 21:11:19 EDT --- Let's try and finish with that review now. New release, new spec: http://tnorth.fedorapeople.org/python-numexpr.spec SRPM: http://tnorth.fedorapeople.org/ rpmlint -v output: python-numexpr.src: I: checking python-numexpr.src: W: spelling-error %description -l en_US runtime - run time, run-time, runtish python-numexpr.src: I: checking-url http://numexpr.googlecode.com/ (timeout 10 seconds) python-numexpr.src: I: checking-url http://numexpr.googlecode.com/files/numexpr-1.4.2.tar.gz (timeout 10 seconds) python-numexpr.x86_64: I: checking python-numexpr.x86_64: W: spelling-error %description -l en_US runtime - run time, run-time, runtish python-numexpr.x86_64: I: checking-url http://numexpr.googlecode.com/ (timeout 10 seconds) python-numexpr-debuginfo.x86_64: I: checking python-numexpr-debuginfo.x86_64: I: checking-url http://numexpr.googlecode.com/ (timeout 10 seconds) 3 packages and 0 specfiles checked; 0 errors, 2 warnings. -- 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956
Mario Blättermann mari...@freenet.de changed:
What|Removed |Added
CC||mari...@freenet.de
--- Comment #7 from Mario Blättermann mari...@freenet.de 2011-04-29 06:56:29
EDT ---
$ rpmlint -v *
python-numexpr.src: I: checking
python-numexpr.src: W: spelling-error %description -l en_US runtime - run
time, run-time, runtish
python-numexpr.src: I: checking-url http://numexpr.googlecode.com/ (timeout 10
seconds)
python-numexpr.src:34: W: rpm-buildroot-usage %build python setup.py install
-O1 --skip-build --root %{buildroot}
python-numexpr.src: I: checking-url
http://numexpr.googlecode.com/files/numexpr-1.4.1.tar.gz (timeout 10 seconds)
python-numexpr.src: W: invalid-url Source0:
http://numexpr.googlecode.com/files/numexpr-1.4.1.tar.gz HTTP Error 404: Not
Found
python-numexpr.i686: I: checking
python-numexpr.i686: W: spelling-error %description -l en_US runtime - run
time, run-time, runtish
python-numexpr.i686: I: checking-url http://numexpr.googlecode.com/ (timeout 10
seconds)
python-numexpr-debuginfo.i686: I: checking
python-numexpr-debuginfo.i686: I: checking-url http://numexpr.googlecode.com/
(timeout 10 seconds)
python-numexpr.spec:34: W: rpm-buildroot-usage %build python setup.py install
-O1 --skip-build --root %{buildroot}
python-numexpr.spec: I: checking-url
http://numexpr.googlecode.com/files/numexpr-1.4.1.tar.gz (timeout 10 seconds)
python-numexpr.spec: W: invalid-url Source0:
http://numexpr.googlecode.com/files/numexpr-1.4.1.tar.gz HTTP Error 404: Not
Found
3 packages and 1 specfiles checked; 0 errors, 6 warnings.
The HTTP errors are due to problems with Google, I know about that.
Regarding the rpm-buildroot-usage error, please read the following:
http://lists.fedoraproject.org/pipermail/devel/2011-January/148045.html
--
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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956 --- Comment #8 from Thibault North thibault.no...@gmail.com 2011-04-29 10:16:01 EDT --- Old spec: http://tnorth.fedorapeople.org/python-numexpr.old.spec Updated spec: Spec URL: http://tnorth.fedorapeople.org/python-numexpr.spec SRPM URL: http://tnorth.fedorapeople.org/python-numexpr-1.4.1-3.fc14.src.rpm rpmlint -v : python-numexpr.src: W: spelling-error %description -l en_US runtime - run time, run-time, runtish python-numexpr.src: W: invalid-url Source0: http://numexpr.googlecode.com/files/numexpr-1.4.1.tar.gz HTTP Error 404: Not Found python-numexpr.x86_64: W: spelling-error %description -l en_US runtime - run time, run-time, runtish 3 packages and 0 specfiles checked; 0 errors, 3 warnings. [tnorth@grouchy ~]$ rpmlint -v /home/tnorth/rpmbuild/SRPMS/python-numexpr-1.4.1-3.fc14.src.rpm /home/tnorth/rpmbuild/RPMS/x86_64/python-numexpr-1.4.1-3.fc14.x86_64.rpm /home/tnorth/rpmbuild/RPMS/x86_64/python-numexpr-debuginfo-1.4.1-3.fc14.x86_64.rpm python-numexpr.src: I: checking python-numexpr.src: W: spelling-error %description -l en_US runtime - run time, run-time, runtish python-numexpr.src: I: checking-url http://numexpr.googlecode.com/ (timeout 10 seconds) python-numexpr.src: I: checking-url http://numexpr.googlecode.com/files/numexpr-1.4.1.tar.gz (timeout 10 seconds) python-numexpr.src: W: invalid-url Source0: http://numexpr.googlecode.com/files/numexpr-1.4.1.tar.gz HTTP Error 404: Not Found python-numexpr.x86_64: I: checking python-numexpr.x86_64: W: spelling-error %description -l en_US runtime - run time, run-time, runtish python-numexpr.x86_64: I: checking-url http://numexpr.googlecode.com/ (timeout 10 seconds) python-numexpr-debuginfo.x86_64: I: checking python-numexpr-debuginfo.x86_64: I: checking-url http://numexpr.googlecode.com/ (timeout 10 seconds) 3 packages and 0 specfiles checked; 0 errors, 3 warnings. -- 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956
Jussi Lehtola jussi.leht...@iki.fi changed:
What|Removed |Added
CC||jussi.leht...@iki.fi
--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2010-12-20 10:56:59
EST ---
Please get rid of the Mandriva stuff:
%define name python-%{module}
Name: %{name}
is just silly. If you want to use the %{module} definition for something, then
just do
Name: python-%{module}
***
Drop
%if ! (0%{?fedora} 12 || 0%{?rhel} 5)
%{!?python_sitelib: %global python_sitelib %(%{__python} -c from
distutils.sysconfig import get_python_lib; print(get_python_lib()))}
%endif
as you don't use it anywhere.
***
The build system looks a bit odd, since it doesn't print out the command used
to compile the object file:
compiling C sources
C compiler: gcc -pthread -fno-strict-aliasing -O2 -g -pipe -Wall
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector
--param=ssp-buffer-size=4 -m64 -mtune=generic -D_GNU_SOURCE -fPIC -fwrapv
-DNDEBUG -O2
-g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector
--param=ssp-buffer-size=4 -m64 -mtune=generic -D_GNU_SOURCE -fPIC -fwrapv -fPIC
creating build/temp.linux-x86_64-2.7
creating build/temp.linux-x86_64-2.7/numexpr
compile options: '-I/usr/lib64/python2.7/site-packages/numpy/core/include
-I/usr/include/python2.7 -c'
extra options: '-funroll-all-loops'
gcc: numexpr/interpreter.c
gcc -pthread -shared build/temp.linux-x86_64-2.7/numexpr/interpreter.o
-L/usr/lib64 -lpython2.7 -o build/lib.linux-x86_64-2.7/numexpr/interpreter.so
running scons
+ exit 0
***
There's no need to use macros for standard commands, although this is not
disallowed in the guidelines.
**
Please use the standard install command from the Python guidelines
python setup.py install -O1 --skip-build --root %{buildroot}
**
rpmlint output is not clean:
python-numexpr.src: W: spelling-error %description -l en_US runtime - run
time, run-time, runtish
python-numexpr.src: W: invalid-url Source0:
http://numexpr.googlecode.com/files/numexpr-1.4.1.tar.gz HTTP Error 404: Not
Found
python-numexpr.x86_64: W: spelling-error %description -l en_US runtime - run
time, run-time, runtish
python-numexpr.x86_64: E: non-standard-executable-perm
/usr/lib64/python2.7/site-packages/numexpr/interpreter.so 0775L
3 packages and 0 specfiles checked; 1 errors, 3 warnings.
**
I'm not fond of using wildcards when they are not needed, as they can lead you
to trouble such as owning something you're not supposed to, or not finding out
if there are files missing from the release. Please be more explicit and change
%{python_sitearch}/*
to
%{python_sitearch}/numexpr/
%{python_sitearch}/numexpr-%{version}-py*.egg-info/
--
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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956 --- Comment #2 from Thibault North thibault.no...@gmail.com 2010-12-20 13:37:16 EST --- Hi Jussy, Thanks for your comments. I have indeed requested a review a bit quickly, because I have been working on that package for my own use and wanted to commit it at some point. Please bear with me. Here are some changes, and the spec + srpm have been updated. I don't get the warning: python-numexpr.x86_64: E: non-standard-executable-perm /usr/lib64/python2.7/site-packages/numexpr/interpreter.so 0775L And the URL is valid (there may be a redirection at some point which perturbs rpmlint). Thanks, Thibault -- 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956 --- Comment #3 from Jussi Lehtola jussi.leht...@iki.fi 2010-12-20 13:40:39 EST --- Please bump the release and make a changelog entry whenever you make changes to the spec file. Otherwise it's impossible for others to see what has been 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 663956] Review Request: python-numexpr - Fast numerical array expression evaluator for Python and NumPy.
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=663956 --- Comment #4 from Thibault North thibault.no...@gmail.com 2010-12-20 13:55:48 EST --- Please bump the release and make a changelog entry whenever you make changes to the spec file. Otherwise it's impossible for others to see what has been done. Ok... well anyway the spec is not versionned yet, and the spec will certainly be removed soon after the review. Plus, you gave the changes in your comment. I'll do that next time though. -- 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
