Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-19 Thread Martin Gräßlin
> On Nov. 18, 2015, 3:40 p.m., David Edmundson wrote: > > won't make a difference, SDDM sources a tonne before we get to you. > > David Edmundson wrote: > more specifically: > > https://github.com/sddm/sddm/blob/master/data/scripts/wayland-session > > we also have anything

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-19 Thread David Edmundson
> On Nov. 18, 2015, 2:40 p.m., David Edmundson wrote: > > won't make a difference, SDDM sources a tonne before we get to you. > > David Edmundson wrote: > more specifically: > > https://github.com/sddm/sddm/blob/master/data/scripts/wayland-session > > we also have anything

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread David Edmundson
> On Nov. 18, 2015, 2:40 p.m., David Edmundson wrote: > > won't make a difference, SDDM sources a tonne before we get to you. > > David Edmundson wrote: > more specifically: > > https://github.com/sddm/sddm/blob/master/data/scripts/wayland-session > > we also have anything

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread Matthias Klumpp
> On Nov. 18, 2015, 2:57 nachm., Matthias Klumpp wrote: > > It just wanted to write what David wrote ;-) > > Maybe a way to resolve this is to filter environment variables in KWin or > > before starting KWin, so anything pointing to directories in $HOME gets > > stripped away (unsetting LD_*

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread Martin Gräßlin
> On Nov. 18, 2015, 3:40 p.m., David Edmundson wrote: > > won't make a difference, SDDM sources a tonne before we get to you. > > David Edmundson wrote: > more specifically: > > https://github.com/sddm/sddm/blob/master/data/scripts/wayland-session > > we also have anything

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread Martin Gräßlin
> On Nov. 18, 2015, 3:57 p.m., Matthias Klumpp wrote: > > It just wanted to write what David wrote ;-) > > Maybe a way to resolve this is to filter environment variables in KWin or > > before starting KWin, so anything pointing to directories in $HOME gets > > stripped away (unsetting LD_*

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread Martin Gräßlin
> On Nov. 18, 2015, 3:57 p.m., Matthias Klumpp wrote: > > It just wanted to write what David wrote ;-) > > Maybe a way to resolve this is to filter environment variables in KWin or > > before starting KWin, so anything pointing to directories in $HOME gets > > stripped away (unsetting LD_*

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126102/ --- (Updated Nov. 18, 2015, 4:36 p.m.) Status -- This change has been

Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126102/ --- Review request for Plasma. Repository: plasma-workspace Description

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126102/#review88511 --- startkde/startplasma.cmake (line 20)

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread Martin Gräßlin
> On Nov. 18, 2015, 3:40 p.m., David Edmundson wrote: > > won't make a difference, SDDM sources a tonne before we get to you. > > David Edmundson wrote: > more specifically: > > https://github.com/sddm/sddm/blob/master/data/scripts/wayland-session > > we also have anything

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread Matthias Klumpp
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126102/#review88528 --- It just wanted to write what David wrote ;-) Maybe a way to

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread David Edmundson
> On Nov. 18, 2015, 2:40 p.m., David Edmundson wrote: > > won't make a difference, SDDM sources a tonne before we get to you. more specifically: https://github.com/sddm/sddm/blob/master/data/scripts/wayland-session we also have anything loaded from pam_env which can include ~/.pam_environment

Re: Review Request 126102: [startkde] Move sourceing of env scripts to startplasma

2015-11-18 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126102/#review88525 --- won't make a difference, SDDM sources a tonne before we get