Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py

2014-01-23 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/ --- (Updated Jan. 23, 2014, 9:47 a.m.) Status -- This change has been ma

Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py

2014-01-23 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/#review48095 --- This review has been submitted with commit 1729071b1bd7060aaa

Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py

2014-01-23 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/ --- (Updated Jan. 23, 2014, 10:44 a.m.) Status -- This change has been d

Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py

2014-01-21 Thread David Faure
> On Jan. 21, 2014, 2:57 p.m., David Faure wrote: > > Note that I fixed the "build failure with kdesrc-build" differently - I > > added a "ignore-modules kapidox" in kdesrc-build's include files. > > > > And I don't like that this will break when using a custom prefix and > > PYTHONPATH is not

Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py

2014-01-21 Thread Alex Merry
> On Jan. 21, 2014, 2:57 p.m., David Faure wrote: > > Note that I fixed the "build failure with kdesrc-build" differently - I > > added a "ignore-modules kapidox" in kdesrc-build's include files. > > > > And I don't like that this will break when using a custom prefix and > > PYTHONPATH is not

Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py

2014-01-21 Thread Aurélien Gâteau
> On Jan. 21, 2014, 3:57 p.m., David Faure wrote: > > Note that I fixed the "build failure with kdesrc-build" differently - I > > added a "ignore-modules kapidox" in kdesrc-build's include files. > > > > And I don't like that this will break when using a custom prefix and > > PYTHONPATH is not

Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py

2014-01-21 Thread Alex Merry
> On Jan. 21, 2014, 2:57 p.m., David Faure wrote: > > Note that I fixed the "build failure with kdesrc-build" differently - I > > added a "ignore-modules kapidox" in kdesrc-build's include files. > > > > And I don't like that this will break when using a custom prefix and > > PYTHONPATH is not

Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py

2014-01-21 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/#review47881 --- Note that I fixed the "build failure with kdesrc-build" differ

Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py

2014-01-21 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/#review47874 --- Ship it! Ship It! - Alex Merry On Jan. 21, 2014, 1:49 p.m.

Review Request 115166: Add a CMakeLists.txt which wraps python setup.py

2014-01-21 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/ --- Review request for Build System, KDE Frameworks and Alex Merry. Repositor