Review Request 117400: kdm: read DesktopNames from session file and export XDG_CURRENT_DESKTOP from it Add DesktopNames key to kdm sessions files.

[Oswald Buddenhagen]

-----------------------------------------------------------
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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140406/067111bc/attachment.htm>