Re: [OpenBabel-Devel] CMake Python Changes

2013-11-05 Thread Daniel Leidert
Hi,   Reinis Danne wrote: >> Doesn't the use of DESTDIR cover this very case? >It does. But you can set it only when running make install. And that is exactly how it should work: http://www.gnu.org/prep/standards/html_node/DESTDIR.html >I >would prefer to set it during configure with the rest o

Re: [OpenBabel-Devel] CMake Python Changes

2013-11-05 Thread Reinis Danne
On Thu, Oct 24, 2013 at 04:08:59PM +0100, Noel O'Boyle wrote: > Doesn't the use of DESTDIR cover this very case? It does. But you can set it only when running make install. I would prefer to set it during configure with the rest of the parameters, there is no reason for this one to be special. R

Re: [OpenBabel-Devel] CMake Python Changes

2013-10-24 Thread Noel O'Boyle
Doesn't the use of DESTDIR cover this very case? - Noel On 24 October 2013 15:18, Reinis Danne wrote: > On Tue, Oct 22, 2013 at 11:52:11AM -0400, David Hall wrote: >> Let me know if you have any questions or see any problems with these >> changes. > > Hi! > > PYTHON_INSTDIR should be made user o

Re: [OpenBabel-Devel] CMake Python Changes

2013-10-24 Thread Reinis Danne
On Tue, Oct 22, 2013 at 11:52:11AM -0400, David Hall wrote: > Let me know if you have any questions or see any problems with these > changes. Hi! PYTHON_INSTDIR should be made user overridable and prefferably in the way that it is seen in ccmake when Python bindings are enabled. One usecase for

Re: [OpenBabel-Devel] CMake Python Changes

2013-10-23 Thread Geoffrey Hutchison
Thanks to both of you. I definitely appreciate finding site-packages, since it means I don’t need to do manual moves on my Mac. Thanks again! -Geoff -- October Webinars: Code for Performance Free Intel webinars can help y

Re: [OpenBabel-Devel] CMake Python Changes

2013-10-23 Thread Noel O'Boyle
Ok - looking back to a thread in Dec 2011 ultimately the problem was that I could not figure out how to ask distutils for the appropriate location of the site-packages folder, so I had to move away from this approach. You seem to have figured it out - nice work. Merge away, Geoff, if you're happy.

Re: [OpenBabel-Devel] CMake Python Changes

2013-10-22 Thread Noel O'Boyle
That looks good - I was just a bit worried that one of these combinations would not work. Let me go back through my old emails to the list about this problem and see whether this covers all the cases. I think it does though. - Noel On 22 October 2013 16:52, David Hall wrote: > Noel and Geoff ask

[OpenBabel-Devel] CMake Python Changes

2013-10-22 Thread David Hall
Noel and Geoff asked that I discuss my changes at https://github.com/openbabel/openbabel/pull/33 on the mailing list. There are two changes. The first is to ask distutils where to install the python bindings for a given executable on non-windows systems. Currently, they are installed in LIBDIR, w