Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py
--- 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 discarded. Review request for Build System, KDE Frameworks and Alex Merry. Repository: kapidox Description --- To fix build failures with kdesrc-build, add a CMakeLists.txt which wraps python setup.py. Diffs - README.md 660e9c3 CMakeLists.txt PRE-CREATION Diff: https://git.reviewboard.kde.org/r/115166/diff/ Testing --- Built and installed with the classic CMake commands. The kapidox tools work fine as long as the installation prefix is either the system one, or defined in $PYTHONPATH. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py
--- 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 1729071b1bd7060aaa1d1b0fe33ad384f25087b9 by Aurélien Gâteau to branch master. - Commit Hook On Jan. 23, 2014, 9:44 a.m., Aurélien Gâteau wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/ --- (Updated Jan. 23, 2014, 9:44 a.m.) Review request for Build System, KDE Frameworks and Alex Merry. Repository: kapidox Description --- To fix build failures with kdesrc-build, add a CMakeLists.txt which wraps python setup.py. Diffs - README.md 660e9c3 CMakeLists.txt PRE-CREATION Diff: https://git.reviewboard.kde.org/r/115166/diff/ Testing --- Built and installed with the classic CMake commands. The kapidox tools work fine as long as the installation prefix is either the system one, or defined in $PYTHONPATH. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py
--- 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 marked as submitted. Review request for Build System, KDE Frameworks and Alex Merry. Repository: kapidox Description --- To fix build failures with kdesrc-build, add a CMakeLists.txt which wraps python setup.py. Diffs - README.md 660e9c3 CMakeLists.txt PRE-CREATION Diff: https://git.reviewboard.kde.org/r/115166/diff/ Testing --- Built and installed with the classic CMake commands. The kapidox tools work fine as long as the installation prefix is either the system one, or defined in $PYTHONPATH. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Review Request 115166: Add a CMakeLists.txt which wraps python setup.py
--- 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. Repository: kapidox Description --- To fix build failures with kdesrc-build, add a CMakeLists.txt which wraps python setup.py. Diffs - README.md 660e9c3 CMakeLists.txt PRE-CREATION Diff: https://git.reviewboard.kde.org/r/115166/diff/ Testing --- Built and installed with the classic CMake commands. The kapidox tools work fine as long as the installation prefix is either the system one, or defined in $PYTHONPATH. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py
--- 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., Aurélien Gâteau wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/ --- (Updated Jan. 21, 2014, 1:49 p.m.) Review request for Build System, KDE Frameworks and Alex Merry. Repository: kapidox Description --- To fix build failures with kdesrc-build, add a CMakeLists.txt which wraps python setup.py. Diffs - README.md 660e9c3 CMakeLists.txt PRE-CREATION Diff: https://git.reviewboard.kde.org/r/115166/diff/ Testing --- Built and installed with the classic CMake commands. The kapidox tools work fine as long as the installation prefix is either the system one, or defined in $PYTHONPATH. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py
--- 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 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 set (which is the case here). - David Faure On Jan. 21, 2014, 1:49 p.m., Aurélien Gâteau wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/ --- (Updated Jan. 21, 2014, 1:49 p.m.) Review request for Build System, KDE Frameworks and Alex Merry. Repository: kapidox Description --- To fix build failures with kdesrc-build, add a CMakeLists.txt which wraps python setup.py. Diffs - README.md 660e9c3 CMakeLists.txt PRE-CREATION Diff: https://git.reviewboard.kde.org/r/115166/diff/ Testing --- Built and installed with the classic CMake commands. The kapidox tools work fine as long as the installation prefix is either the system one, or defined in $PYTHONPATH. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py
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 set (which is the case here). You can't really avoid that with Python packages; you can install them with --user and make sure your PATH is set correctly, or you can install them with a prefix and make sure your PYTHONPATH is set correctly (or there are a couple of other ways of telling python about custom site package dirs). Well, or you can install them in a standard location, such as for distro packages. - Alex --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/#review47881 --- On Jan. 21, 2014, 1:49 p.m., Aurélien Gâteau wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/ --- (Updated Jan. 21, 2014, 1:49 p.m.) Review request for Build System, KDE Frameworks and Alex Merry. Repository: kapidox Description --- To fix build failures with kdesrc-build, add a CMakeLists.txt which wraps python setup.py. Diffs - README.md 660e9c3 CMakeLists.txt PRE-CREATION Diff: https://git.reviewboard.kde.org/r/115166/diff/ Testing --- Built and installed with the classic CMake commands. The kapidox tools work fine as long as the installation prefix is either the system one, or defined in $PYTHONPATH. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py
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 set (which is the case here). Alex Merry wrote: You can't really avoid that with Python packages; you can install them with --user and make sure your PATH is set correctly, or you can install them with a prefix and make sure your PYTHONPATH is set correctly (or there are a couple of other ways of telling python about custom site package dirs). Well, or you can install them in a standard location, such as for distro packages. Note that the breakage is at runtime, not install time. But since kdesrc-build now ignores kapidox, maybe it's simpler to discard this patch? That's one less piece to maintain. - Aurélien --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/#review47881 --- On Jan. 21, 2014, 2:49 p.m., Aurélien Gâteau wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/ --- (Updated Jan. 21, 2014, 2:49 p.m.) Review request for Build System, KDE Frameworks and Alex Merry. Repository: kapidox Description --- To fix build failures with kdesrc-build, add a CMakeLists.txt which wraps python setup.py. Diffs - README.md 660e9c3 CMakeLists.txt PRE-CREATION Diff: https://git.reviewboard.kde.org/r/115166/diff/ Testing --- Built and installed with the classic CMake commands. The kapidox tools work fine as long as the installation prefix is either the system one, or defined in $PYTHONPATH. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py
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 set (which is the case here). Alex Merry wrote: You can't really avoid that with Python packages; you can install them with --user and make sure your PATH is set correctly, or you can install them with a prefix and make sure your PYTHONPATH is set correctly (or there are a couple of other ways of telling python about custom site package dirs). Well, or you can install them in a standard location, such as for distro packages. Aurélien Gâteau wrote: Note that the breakage is at runtime, not install time. But since kdesrc-build now ignores kapidox, maybe it's simpler to discard this patch? That's one less piece to maintain. Sure; I have no opinion on it either way, TBH. - Alex --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/#review47881 --- On Jan. 21, 2014, 1:49 p.m., Aurélien Gâteau wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/ --- (Updated Jan. 21, 2014, 1:49 p.m.) Review request for Build System, KDE Frameworks and Alex Merry. Repository: kapidox Description --- To fix build failures with kdesrc-build, add a CMakeLists.txt which wraps python setup.py. Diffs - README.md 660e9c3 CMakeLists.txt PRE-CREATION Diff: https://git.reviewboard.kde.org/r/115166/diff/ Testing --- Built and installed with the classic CMake commands. The kapidox tools work fine as long as the installation prefix is either the system one, or defined in $PYTHONPATH. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 115166: Add a CMakeLists.txt which wraps python setup.py
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 set (which is the case here). Alex Merry wrote: You can't really avoid that with Python packages; you can install them with --user and make sure your PATH is set correctly, or you can install them with a prefix and make sure your PYTHONPATH is set correctly (or there are a couple of other ways of telling python about custom site package dirs). Well, or you can install them in a standard location, such as for distro packages. Aurélien Gâteau wrote: Note that the breakage is at runtime, not install time. But since kdesrc-build now ignores kapidox, maybe it's simpler to discard this patch? That's one less piece to maintain. Alex Merry wrote: Sure; I have no opinion on it either way, TBH. Ah right, it won't break make install. Then I withdraw my objection, go ahead. - David --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/#review47881 --- On Jan. 21, 2014, 1:49 p.m., Aurélien Gâteau wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115166/ --- (Updated Jan. 21, 2014, 1:49 p.m.) Review request for Build System, KDE Frameworks and Alex Merry. Repository: kapidox Description --- To fix build failures with kdesrc-build, add a CMakeLists.txt which wraps python setup.py. Diffs - README.md 660e9c3 CMakeLists.txt PRE-CREATION Diff: https://git.reviewboard.kde.org/r/115166/diff/ Testing --- Built and installed with the classic CMake commands. The kapidox tools work fine as long as the installation prefix is either the system one, or defined in $PYTHONPATH. Thanks, Aurélien Gâteau ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
