[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-09-13 Thread Sigurd F
> The user's .profile is to be read by the shell, on interactive shell > (i.e. terminal) logins ONLY This is incorrect, .profile is read by the shell also when the shell is non-interactive. Here is the quote from the bash manual: > When Bash is invoked as an interactive login shell, or as a >

Re: [Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-26 Thread Gunnar Hjalmarsson
On 06/26/2015 04:22 AM, Anders Kaseorg wrote: ~/.xsessionrc is sourced using /bin/sh. Actually it's sourced using /bin/bash since a few months. A default Ubuntu session does not invoke ~/.xsession at all. Creating one that works correctly with Unity or GNOME or whatever, as well as

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Gunnar Hjalmarsson
@Derek: I don't know what's right and wrong here, but what does it really matter in practice? Have you considered the mess which would be the result if lightdm suddenly stopped to source ~/.profile? -- You received this bug notification because you are a member of Desktop Packages, which is

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Anders Kaseorg
.xsession is sourced within /bin/sh, not /bin/bash. It wouldn’t make sense to source .bashrc from it (unless you happened to write a .bashrc that doesn’t use any bash features). Also, .bashrc is sourced by _every_ interactive shell, so if you were to add things to environment variables from

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Derek Martin
See also Bug #1468834. -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to lightdm in Ubuntu. https://bugs.launchpad.net/bugs/1468832 Title: lightdm sources .profile Status in lightdm package in Ubuntu: New Bug description: It is a

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Derek Martin
@Myself: in my description I repeatedly used interactive shell when I meant login shell instead. The distinction is important, (and thus the shell makes the distinction)--the user's .profile is only sourced in the latter case. On all other interactive shells it sources .bashrc instead. This is

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Anders Kaseorg
No, you misunderstand my comment. It doesn’t matter whether or not your shell is bash. ~/.xsession file is sourced using /bin/sh, not using your shell. You cannot expect to be able to source ~/.bashrc from ~/.xsession, no matter what your preferred shell is. -- You received this bug notification

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Derek Martin
Also slight correction: BASH does NOT automatically source .bashrc on login shells; so .profile would need to source the environment file as well. -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to lightdm in Ubuntu.

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Gunnar Hjalmarsson
On 06/26/2015 04:22 AM, Anders Kaseorg wrote: ~/.xsessionrc is sourced using /bin/sh. Actually it's sourced using /bin/bash since a few months (/bin/bash is in the shebang line of /etc/sbin/lightdm-session). A default Ubuntu session does not invoke ~/.xsession at all. Creating one that works

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Anders Kaseorg
Aha, I was confusing ~/.xsessionrc with ~/.xsession. ~/.xsessionrc is sourced using /bin/sh. A default Ubuntu session does not invoke ~/.xsession at all. Creating one that works correctly with Unity or GNOME or whatever, as well as configuring LightDM to invoke it it, requires a very pointy hat.

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Derek Martin
Which is a good point actually... what if the user is a C shell user? How does sourcing .profile help them? The classic way of setting this up is clearly far superior. -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to lightdm in Ubuntu.

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Derek Martin
In point of fact .xsession is supposed to be an executable shell script with a shebang line, it's not supposed to be sourced at all, or at least historically that has always been the case. Theoretically the .xsession file could be a C shell script so as long as it is executed instead of sourced,

[Desktop-packages] [Bug 1468832] Re: lightdm sources .profile

2015-06-25 Thread Gunnar Hjalmarsson
That error dialog is supposed to be helpful. ;) It was implemented (by me) as a fix of bug #678421, which was originally filed as a bug about GDM and Xsession, and one idea is to *prevent* that errors are silently ignored. On 06/26/2015 12:58 AM, Derek Martin wrote: if you don't care to fix