----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117400/#review55089 -----------------------------------------------------------
kdm/backend/client.c <https://git.reviewboard.kde.org/r/117400/#comment38413> you can use buf for that. kdm/backend/client.c <https://git.reviewboard.kde.org/r/117400/#comment38414> i'd actually prefer it if you used buf2 to iterate via pointer. kdm/backend/client.c <https://git.reviewboard.kde.org/r/117400/#comment38416> excess braces (yup, qt style in kdm). kdm/backend/client.c <https://git.reviewboard.kde.org/r/117400/#comment38415> for pedantry, you actually should free() desktopNames (to be buf). - Oswald Buddenhagen On April 6, 2014, 11:10 a.m., David Faure wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117400/ > ----------------------------------------------------------- > > (Updated April 6, 2014, 11:10 a.m.) > > > Review request for kde-workspace and Oswald Buddenhagen. > > > Repository: kde-workspace > > > Description > ------- > > kdm: read DesktopNames from session file and export XDG_CURRENT_DESKTOP from > it > > [after converting ';' (XDG list separator) to ':' (path separator)] > > As discussed at the freedesktop summit. > > XDG_CURRENT_DESKTOP is useful for OnlyShowIn and for the new mimeapps.list > spec. > > + Add DesktopNames key to kdm sessions files (separate commit). > > > Diffs > ----- > > kdm/backend/client.c 26bb0b4d56ebc87a0e8900b0936a90601a862603 > kdm/kfrontend/sessions/gnome.desktop > f4263dd7e75ed90e85ac0002f463c4e594a70f91 > kdm/kfrontend/sessions/kde-plasma-safe.desktop.cmake > 722091e3693e3fb86bd8983c1e0423a93a2b482d > kdm/kfrontend/sessions/kde-plasma.desktop.cmake > 37413b6beea83b33880858502b451eda370cf8f4 > kdm/kfrontend/sessions/lxde.desktop > b794fb24e349627df35b82ee4cc722e5fad0090b > kdm/kfrontend/sessions/xfce.desktop > c3362304edea90fd40d19e6c3f4a434ba780b490 > kdm/kfrontend/sessions/xfce4.desktop > 75c88a13eaf007c0149744c3f437d06532a398b6 > > Diff: https://git.reviewboard.kde.org/r/117400/diff/ > > > Testing > ------- > > Rebuilt OpenSuSE's kdm with the patch applied, commented out the setting of > XDG_CURRENT_DESKTOP in startkde, logged into a session with DesktopNames=TEST > --> echo $XDG_CURRENT_DESKTOP says TEST. > > > Thanks, > > David Faure > >