[Bug 458678] Review Request: picviz - Parallel coordinates plotter

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=458678





--- Comment #4 from Steve Grubb <sgrubb@xxxxxxxxxx>  2008-08-12 15:02:04 EDT ---
The new package still has the same problem. 

The problem is that /lib is hardcoded in the makefiles. See
src/libpicviz/picviz.pc.cmake and look for Libs: -L@CMAKE_INSTALL_PREFIX@/lib
-lpicviz. That needs to be changed to a prefix passed in from the spec file. I
can confirm changing that to lib64 makes all the /usr/lib/*.so issues go away
leaving just python going to the wrong place. It just needs to be done
automatically instead of hardcoding it.

WRT python, I'd say that there needs to be a passing of %{python_sitearch} into
the setup.py files so that it understands what rpm wants it to do. 

picviz-0.2.3/src/libpicviz/bindings/python/setup.py --help install

has some hints. Maybe, something like:

python ./setup.py install --root=%{buildroot}
--install-lib=%{buildroot}%{python_sitearch}

or maybe --install-platlib=%{buildroot}%{python_sitearch}

but then some specfile locations need adjusting. You do not need to make a -3
srpm since when its imported it will be a -1 file. Just let me know when you
have a new file posted to try out.

Also, you need to try this on F9 or rawhide because python there makes egginfo
files and you are not packaging them or deleting them as the case may warrant.

-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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]