Re: Review Request 117023: Use the full path for kconf_update when running it
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117023/ --- (Updated March 25, 2014, 10:23 a.m.) Status -- This change has been marked as submitted. Review request for KDE Frameworks. Repository: kconfig Description --- Use the full path for kconf_update when running it It is not on $PATH, so we cannot just expect QProcess to find it. Export the location of kconf_update in CMake This is for the benefit of kded, which needs to run this binary. Diffs - src/core/CMakeLists.txt 4e13ee5ac129353feaa4ad6053226574f805b73c src/core/kconfig.cpp 5a4a083d0c220cd28e60c84cdaff8a014fe5acf5 src/kconf_update/CMakeLists.txt e4e37ba963aac186d5c8495f8e1ed8e961429669 Diff: https://git.reviewboard.kde.org/r/117023/diff/ Testing --- Builds, installs. Checked the define passed to the compiler by eye. Not sure how to test the actual code. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117023: Use the full path for kconf_update when running it
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117023/#review54084 --- This review has been submitted with commit d53cf39a20d3f3f23883ca9a6b154041ee094a8d by Alex Merry to branch master. - Commit Hook On March 24, 2014, 4:19 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117023/ > --- > > (Updated March 24, 2014, 4:19 p.m.) > > > Review request for KDE Frameworks. > > > Repository: kconfig > > > Description > --- > > Use the full path for kconf_update when running it > > It is not on $PATH, so we cannot just expect QProcess to find it. > > Export the location of kconf_update in CMake > > This is for the benefit of kded, which needs to run this binary. > > > Diffs > - > > src/core/CMakeLists.txt 4e13ee5ac129353feaa4ad6053226574f805b73c > src/core/kconfig.cpp 5a4a083d0c220cd28e60c84cdaff8a014fe5acf5 > src/kconf_update/CMakeLists.txt e4e37ba963aac186d5c8495f8e1ed8e961429669 > > Diff: https://git.reviewboard.kde.org/r/117023/diff/ > > > Testing > --- > > Builds, installs. Checked the define passed to the compiler by eye. Not > sure how to test the actual code. > > > Thanks, > > Alex Merry > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117023: Use the full path for kconf_update when running it
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117023/ --- (Updated March 25, 2014, 10:23 a.m.) Status -- This change has been marked as submitted. Review request for KDE Frameworks. Repository: kconfig Description --- Use the full path for kconf_update when running it It is not on $PATH, so we cannot just expect QProcess to find it. Export the location of kconf_update in CMake This is for the benefit of kded, which needs to run this binary. Diffs - src/core/CMakeLists.txt 4e13ee5ac129353feaa4ad6053226574f805b73c src/core/kconfig.cpp 5a4a083d0c220cd28e60c84cdaff8a014fe5acf5 src/kconf_update/CMakeLists.txt e4e37ba963aac186d5c8495f8e1ed8e961429669 Diff: https://git.reviewboard.kde.org/r/117023/diff/ Testing --- Builds, installs. Checked the define passed to the compiler by eye. Not sure how to test the actual code. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117023: Use the full path for kconf_update when running it
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117023/#review54085 --- This review has been submitted with commit e1ad9a6192d1e2be6fcb17c8a1e961104d921752 by Alex Merry to branch master. - Commit Hook On March 24, 2014, 4:19 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117023/ > --- > > (Updated March 24, 2014, 4:19 p.m.) > > > Review request for KDE Frameworks. > > > Repository: kconfig > > > Description > --- > > Use the full path for kconf_update when running it > > It is not on $PATH, so we cannot just expect QProcess to find it. > > Export the location of kconf_update in CMake > > This is for the benefit of kded, which needs to run this binary. > > > Diffs > - > > src/core/CMakeLists.txt 4e13ee5ac129353feaa4ad6053226574f805b73c > src/core/kconfig.cpp 5a4a083d0c220cd28e60c84cdaff8a014fe5acf5 > src/kconf_update/CMakeLists.txt e4e37ba963aac186d5c8495f8e1ed8e961429669 > > Diff: https://git.reviewboard.kde.org/r/117023/diff/ > > > Testing > --- > > Builds, installs. Checked the define passed to the compiler by eye. Not > sure how to test the actual code. > > > Thanks, > > Alex Merry > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 117023: Use the full path for kconf_update when running it
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117023/#review54021 --- Ship it! Ship It! - Kevin Ottens On March 24, 2014, 4:19 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117023/ > --- > > (Updated March 24, 2014, 4:19 p.m.) > > > Review request for KDE Frameworks. > > > Repository: kconfig > > > Description > --- > > Use the full path for kconf_update when running it > > It is not on $PATH, so we cannot just expect QProcess to find it. > > Export the location of kconf_update in CMake > > This is for the benefit of kded, which needs to run this binary. > > > Diffs > - > > src/core/CMakeLists.txt 4e13ee5ac129353feaa4ad6053226574f805b73c > src/core/kconfig.cpp 5a4a083d0c220cd28e60c84cdaff8a014fe5acf5 > src/kconf_update/CMakeLists.txt e4e37ba963aac186d5c8495f8e1ed8e961429669 > > Diff: https://git.reviewboard.kde.org/r/117023/diff/ > > > Testing > --- > > Builds, installs. Checked the define passed to the compiler by eye. Not > sure how to test the actual code. > > > Thanks, > > Alex Merry > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Review Request 117023: Use the full path for kconf_update when running it
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117023/ --- Review request for KDE Frameworks. Repository: kconfig Description --- Use the full path for kconf_update when running it It is not on $PATH, so we cannot just expect QProcess to find it. Export the location of kconf_update in CMake This is for the benefit of kded, which needs to run this binary. Diffs - src/core/CMakeLists.txt 4e13ee5ac129353feaa4ad6053226574f805b73c src/core/kconfig.cpp 5a4a083d0c220cd28e60c84cdaff8a014fe5acf5 src/kconf_update/CMakeLists.txt e4e37ba963aac186d5c8495f8e1ed8e961429669 Diff: https://git.reviewboard.kde.org/r/117023/diff/ Testing --- Builds, installs. Checked the define passed to the compiler by eye. Not sure how to test the actual code. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel