Re: Review Request 126813: Fix build with older polkit
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126813/ --- (Updated Feb. 26, 2016, 3:59 p.m.) Status -- This change has been marked as submitted. Review request for KDE Frameworks. Changes --- Submitted with commit 15c7867e88c5278f61896b5531ac2b544add8220 by Alex Richardson to branch master. Repository: polkit-qt-1 Description --- Seems like the function exists, but the header declaration is missing Diffs - CMakeLists.txt bb91bdedc96b8211eb29a1180c2e451dc60fae18 core/polkitqt1-subject.h 03028f636d7efc154138821419a704b661a7aeac core/polkitqt1-subject.cpp ecb4c0e216d5bacf5dff5cf100611b941d3e8fbd polkitqt1-config.h.cmake PRE-CREATION Diff: https://git.reviewboard.kde.org/r/126813/diff/ Testing --- compiles now Thanks, Alex Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 126813: Fix build with older polkit
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126813/#review92802 --- Ship it! Ship It! - Martin Gräßlin On Jan. 21, 2016, 5:24 p.m., Alex Richardson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126813/ > --- > > (Updated Jan. 21, 2016, 5:24 p.m.) > > > Review request for KDE Frameworks. > > > Repository: polkit-qt-1 > > > Description > --- > > Seems like the function exists, but the header declaration is missing > > > Diffs > - > > CMakeLists.txt bb91bdedc96b8211eb29a1180c2e451dc60fae18 > core/polkitqt1-subject.h 03028f636d7efc154138821419a704b661a7aeac > core/polkitqt1-subject.cpp ecb4c0e216d5bacf5dff5cf100611b941d3e8fbd > polkitqt1-config.h.cmake PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/126813/diff/ > > > Testing > --- > > compiles now > > > Thanks, > > Alex Richardson > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 126813: Fix build with older polkit
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126813/#review91567 --- Ship it! Ship It! - Martin Gräßlin On Jan. 21, 2016, 5:24 p.m., Alex Richardson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126813/ > --- > > (Updated Jan. 21, 2016, 5:24 p.m.) > > > Review request for KDE Frameworks. > > > Repository: polkit-qt-1 > > > Description > --- > > Seems like the function exists, but the header declaration is missing > > > Diffs > - > > CMakeLists.txt bb91bdedc96b8211eb29a1180c2e451dc60fae18 > core/polkitqt1-subject.h 03028f636d7efc154138821419a704b661a7aeac > core/polkitqt1-subject.cpp ecb4c0e216d5bacf5dff5cf100611b941d3e8fbd > polkitqt1-config.h.cmake PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/126813/diff/ > > > Testing > --- > > compiles now > > > Thanks, > > Alex Richardson > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 126813: Fix build with older polkit
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126813/#review91431 --- Looks good. - David Jarvie On Jan. 21, 2016, 4:24 p.m., Alex Richardson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126813/ > --- > > (Updated Jan. 21, 2016, 4:24 p.m.) > > > Review request for KDE Frameworks. > > > Repository: polkit-qt-1 > > > Description > --- > > Seems like the function exists, but the header declaration is missing > > > Diffs > - > > CMakeLists.txt bb91bdedc96b8211eb29a1180c2e451dc60fae18 > core/polkitqt1-subject.h 03028f636d7efc154138821419a704b661a7aeac > core/polkitqt1-subject.cpp ecb4c0e216d5bacf5dff5cf100611b941d3e8fbd > polkitqt1-config.h.cmake PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/126813/diff/ > > > Testing > --- > > compiles now > > > Thanks, > > Alex Richardson > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 126813: Fix build with older polkit
> On Jan. 19, 2016, 5:22 p.m., David Jarvie wrote: > > The symbol `polkit_system_bus_name_get_user_sync` doesn't exist in the > > Debian stable polkit libraries (it doesn't exist in the polkit-1 package > > sources or Debian patches either). The Debian stable package version is > > 0.105-8. > > > > The patch does allow polkit-qt-1 to build, but presumably it might lead to > > link errors if `SystemBusNameSubject::user()` is referenced. Wouldn't it be > > better to deal with the symbol `polkit_system_bus_name_get_user_sync` in > > the same manner as `polkit_authority_get_sync` and > > `polkit_agent_listener_register`, which if they are missing from the > > libraries, CMakeLists.txt sets the conditional define > > `POLKIT_QT_1_COMPATIBILITY_MODE`. That could also be used to conditionally > > compile the code referencing `polkit_system_bus_name_get_user_sync`. we cannot conditionally compile it. Checking is fine, but we should mark an error if it doesn't compile. The whole point of adding the new API was to use it in KAuth. If we allow to conditionally compile we just move the problem down to KAuth. - Martin --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126813/#review91338 --- On Jan. 19, 2016, 2:09 p.m., Alex Richardson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126813/ > --- > > (Updated Jan. 19, 2016, 2:09 p.m.) > > > Review request for KDE Frameworks. > > > Repository: polkit-qt-1 > > > Description > --- > > Seems like the function exists, but the header declaration is missing > > > Diffs > - > > core/polkitqt1-subject.cpp ecb4c0e216d5bacf5dff5cf100611b941d3e8fbd > > Diff: https://git.reviewboard.kde.org/r/126813/diff/ > > > Testing > --- > > compiles now > > > Thanks, > > Alex Richardson > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Review Request 126813: Fix build with older polkit
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126813/ --- Review request for KDE Frameworks. Repository: polkit-qt-1 Description --- Seems like the function exists, but the header declaration is missing Diffs - core/polkitqt1-subject.cpp ecb4c0e216d5bacf5dff5cf100611b941d3e8fbd Diff: https://git.reviewboard.kde.org/r/126813/diff/ Testing --- compiles now Thanks, Alex Richardson ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 126813: Fix build with older polkit
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126813/#review91338 --- The symbol `polkit_system_bus_name_get_user_sync` doesn't exist in the Debian stable polkit libraries (it doesn't exist in the polkit-1 package sources or Debian patches either). The Debian stable package version is 0.105-8. The patch does allow polkit-qt-1 to build, but presumably it might lead to link errors if `SystemBusNameSubject::user()` is referenced. Wouldn't it be better to deal with the symbol `polkit_system_bus_name_get_user_sync` in the same manner as `polkit_authority_get_sync` and `polkit_agent_listener_register`, which if they are missing from the libraries, CMakeLists.txt sets the conditional define `POLKIT_QT_1_COMPATIBILITY_MODE`. That could also be used to conditionally compile the code referencing `polkit_system_bus_name_get_user_sync`. - David Jarvie On Jan. 19, 2016, 1:09 p.m., Alex Richardson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126813/ > --- > > (Updated Jan. 19, 2016, 1:09 p.m.) > > > Review request for KDE Frameworks. > > > Repository: polkit-qt-1 > > > Description > --- > > Seems like the function exists, but the header declaration is missing > > > Diffs > - > > core/polkitqt1-subject.cpp ecb4c0e216d5bacf5dff5cf100611b941d3e8fbd > > Diff: https://git.reviewboard.kde.org/r/126813/diff/ > > > Testing > --- > > compiles now > > > Thanks, > > Alex Richardson > > ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel