Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190247
Summary: Review Request: PyX Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: bugzilla-sink@leemhuis.info ReportedBy: mpeters@mac.com QAContact: fedora-package-review@redhat.com
Spec URL: http://mpeters.us/PyX/PyX.spec SRPM URL: http://mpeters.us/PyX/PyX-0.8.1-1.fc5.src.rpm Description: PyX is a Python package for the creation of PostScript and PDF files. It combines an abstraction of the PostScript drawing model with a TeX/LaTeX interface. Complex tasks like 2d and 3d plots in publication-ready quality are built out of these primitives.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: PyX - Python graphics package
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190247
mpeters@mac.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: PyX |Review Request: PyX - Python | |graphics package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: PyX - Python graphics package
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190247
------- Additional Comments From mpeters@mac.com 2006-04-29 04:27 EST ------- Known bug (blocker) - installing in a buildroot results in broken pyx/siteconfig.py and pyx/siteconfig.pyc (includes the buildroot)
I'll see if I can fix this.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: PyX - Python graphics package
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190247
------- Additional Comments From mpeters@mac.com 2006-04-29 07:40 EST ------- Problem solved:
http://mpeters.us/PyX/PyX-0.8.1-3.fc5.src.rpm http://mpeters.us/PyX/PyX.spec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: PyX - Python graphics package
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190247
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|bugzilla-sink@leemhuis.info |tibbs@math.uh.edu OtherBugsDependingO|163776 |163778 nThis| |
------- Additional Comments From tibbs@math.uh.edu 2006-04-30 00:52 EST ------- FYI, theres an open ticket for the mkhowto issue: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=177349
PyChart is working around it by including a copy of Python's Doc directory. I see no problems with simply shipping the formatted manual from upstream, but you should probably treat it as any other upstream source and provide a full URL.
Issues: The build fails on x86_64 (both FC5 and development): gcc -pthread -shared -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic build/temp.linux-x86_64-2.4/pyx/pykpathsea/pykpathsea.o -lkpathsea -o build/lib.linux-x86_64-2.4/pyx/pykpathsea/_pykpathsea.so /usr/bin/ld: /usr/lib/gcc/x86_64-redhat-linux/4.1.0/../../../../lib64/libkpathsea.a(tex-file.o): relocation R_X86_64_32S against `kpse_format_info' can not be used when making a shared object; recompile with -fPIC /usr/lib/gcc/x86_64-redhat-linux/4.1.0/../../../../lib64/libkpathsea.a: could not read symbols: Bad value collect2: ld returned 1 exit status error: command 'gcc' failed with exit status 1 error: Bad exit status from /var/tmp/rpm-tmp.12911 (%build)
I'm afraid I have no idea at all what that means, except for "recompile with -fPIC", and -fPIC already on all of the gcc command lines so I assume it's complaining about /usr/lib644/libkpathsea.a. A quick search leads to https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=150085.
Anyway, it builds fine on i386, so you'll need to ExcludeArch: x86_64 at least (no way to test PPC, sorry) and then open a bug on the failed x86_64 build and have it block FE-ExcludeArch-x64 (and have 150085 block it).
rpmlint complains: E: PyX non-executable-script /usr/lib/python2.4/site-packages/pyx/graph/axis/tick.py 0644 (and 41 additional complaints about other files)
These files all contain #!/blah/python lines. Those scripts aren't intended to be executable, so those shebang lines need to be removed. Note that they aren't consistent; graph/axis/timeaxis.py is the only .py file not to have such a line. This is the only blocker that I can see.
Review: * package meets naming and packaging guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * license field matches the actual license. * license is open source-compatible and included in the package. * source files match upstream: bfab7bbc4f3442c946f489133df515f0 manual.pdf bfab7bbc4f3442c946f489133df515f0 manual.pdf-srpm 5e751cef8d62774a6fc659cc9a03c231 PyX-0.8.1.tar.gz 5e751cef8d62774a6fc659cc9a03c231 PyX-0.8.1.tar.gz-srpm * latest version is being packaged. * BuildRequires are proper. O package builds in mock (development, i386) but fails on x86_64. X rpmlint complains about errant shebang lines in all but one .py file. * final provides and requires are sane. * shared libraries are present, but internal to python so no need to call ldconfig. * package is not relocatable. * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * .pyo files are properly %ghosted. * %clean is present. * %check is not present; no test suite. * code, not content. * documentation is not exactly small, but not large enough that a -docs subpackage is necessary. * %docs are not necessary for the proper functioning of the package. * no headers. * no pkgconfig files. * no libtool .la droppings. * not a GUI app.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: PyX - Python graphics package
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190247
------- Additional Comments From mpeters@mac.com 2006-04-30 04:32 EST ------- Note on documentation - full path url is in comment about Source1 I changed the name of the source for the src.rpm because the upstream filename was way to generic - manual.pdf
I have fixed the rpmlint error - the way I fixed it was simply to run sed on all non executable .py scripts and change ^#! to ##
Is that acceptable or would refer the enture shebang lines just be removed?
I think I have temporarily fixed it for x86_64 by simply not building the kpathsea C extension for x86_64 - the kpathsea part of the package should still work on x86_64, just with a performance hit.
New src.rpm: http://mpeters.us/PyX/PyX-0.8.1-4.fc5.src.rpm New specfile: http://mpeters.us/PyX/PyX.spec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: PyX - Python graphics package
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190247
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 |163779 nThis| |
------- Additional Comments From tibbs@math.uh.edu 2006-05-01 22:59 EST ------- This is much better. The package builds OK on i386 and x86_64 and rpmlint is silent.
I understand about the manual; generally we name things the way they come from upstream, but in this case upstream isn't even versioned so you might not get what you expect when you fetch the file. Hopefully 177349 will be fixed one of these days and this issue will go away.
I don't have a preference for fixing the shebang lines; If I were doing it I'd probably delete them, but the main goal here is to quiet rpmlint.
I'm glad you were able to work around 150085 without sacrificing functionality.
APPROVED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: PyX - Python graphics package
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190247
mpeters@mac.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From mpeters@mac.com 2006-05-04 07:37 EST ------- Been through build system for FC-4/FC-5
fails on devel due to broken devel build system http://buildsys.fedoraproject.org/logs/fedora-development-extras/8816-PyX-0.... http://buildsys.fedoraproject.org/logs/fedora-development-extras/8816-PyX-0....
closing as next-release, I'll recue for devel when I've seen other compiled packages get through devel build system.
package-review@lists.fedoraproject.org