| Message ID | cover.1372861799.git.laurentiu.palcu@intel.com |
|---|---|
| State | New |
| Headers | show |
Saul spotted an issue with the 4/4 regarding the usage of /sbin directly in the .desktop file and I'm going to send a v2 to address that. Laurentiu On 07/03/2013 05:34 PM, Laurentiu Palcu wrote: > The following changes since commit 8a186a6b3853fc1a7dcf342d421c8926c38949c9: > > bitbake: hob: save button from settings called a nonexisting method (2013-07-03 08:13:35 +0100) > > are available in the git repository at: > > git://git.yoctoproject.org/poky-contrib lpalcu/b4345_shutdown_icon_rootless_x > http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=lpalcu/b4345_shutdown_icon_rootless_x > > Laurentiu Palcu (4): > sysvinit: allow users in shutdown group to perform halt/reboot > connman: make xuser a member of 'shutdown' group > xserver-nodm-init: make xuser a member of 'shutdown' group > shutdown-desktop: give entire path in Exec field > > meta/recipes-connectivity/connman/connman.inc | 2 +- > meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 12 +++++++++--- > .../x11-common/xserver-nodm-init.bb | 2 +- > .../shutdown-desktop/shutdown-desktop.bb | 2 +- > .../shutdown-desktop/shutdown.desktop | 4 ++-- > 5 files changed, 14 insertions(+), 8 deletions(-) >
On Wed, Jul 03, 2013 at 05:34:20PM +0300, Laurentiu Palcu wrote: > The following changes since commit 8a186a6b3853fc1a7dcf342d421c8926c38949c9: > > bitbake: hob: save button from settings called a nonexisting method (2013-07-03 08:13:35 +0100) > > are available in the git repository at: > > git://git.yoctoproject.org/poky-contrib lpalcu/b4345_shutdown_icon_rootless_x > http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=lpalcu/b4345_shutdown_icon_rootless_x > > Laurentiu Palcu (4): > sysvinit: allow users in shutdown group to perform halt/reboot > connman: make xuser a member of 'shutdown' group > xserver-nodm-init: make xuser a member of 'shutdown' group > shutdown-desktop: give entire path in Exec field Shouldn't we create separate recipe for xuser or any other default user and add it to connman and xserver-nodm-init RDEPENDS? I know there is some overhead from having one more package installed on image, but it would be a bit easier if we later decide that xuser also needs some file to be installed in his home or something like that. > meta/recipes-connectivity/connman/connman.inc | 2 +- > meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 12 +++++++++--- > .../x11-common/xserver-nodm-init.bb | 2 +- > .../shutdown-desktop/shutdown-desktop.bb | 2 +- > .../shutdown-desktop/shutdown.desktop | 4 ++-- > 5 files changed, 14 insertions(+), 8 deletions(-) > > -- > 1.7.9.5 > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core
On 07/03/2013 07:19 PM, Martin Jansa wrote: > On Wed, Jul 03, 2013 at 05:34:20PM +0300, Laurentiu Palcu wrote: >> The following changes since commit 8a186a6b3853fc1a7dcf342d421c8926c38949c9: >> >> bitbake: hob: save button from settings called a nonexisting method (2013-07-03 08:13:35 +0100) >> >> are available in the git repository at: >> >> git://git.yoctoproject.org/poky-contrib lpalcu/b4345_shutdown_icon_rootless_x >> http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=lpalcu/b4345_shutdown_icon_rootless_x >> >> Laurentiu Palcu (4): >> sysvinit: allow users in shutdown group to perform halt/reboot >> connman: make xuser a member of 'shutdown' group >> xserver-nodm-init: make xuser a member of 'shutdown' group >> shutdown-desktop: give entire path in Exec field > > Shouldn't we create separate recipe for xuser or any other default user > and add it to connman and xserver-nodm-init RDEPENDS? > > I know there is some overhead from having one more package installed on > image, but it would be a bit easier if we later decide that xuser also > needs some file to be installed in his home or something like that. I can see one advantage of having a separate recipe for xuser: the user creation would be done in one place and we souldn't have to change in two places if we want to add the user to another group (for example). It sounds a good change to me. Laurentiu > >> meta/recipes-connectivity/connman/connman.inc | 2 +- >> meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 12 +++++++++--- >> .../x11-common/xserver-nodm-init.bb | 2 +- >> .../shutdown-desktop/shutdown-desktop.bb | 2 +- >> .../shutdown-desktop/shutdown.desktop | 4 ++-- >> 5 files changed, 14 insertions(+), 8 deletions(-) >> >> -- >> 1.7.9.5 >> >> _______________________________________________ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.openembedded.org/mailman/listinfo/openembedded-core >
On 03/07/13 17:19, Martin Jansa wrote: > Shouldn't we create separate recipe for xuser or any other default user > and add it to connman and xserver-nodm-init RDEPENDS? > > I know there is some overhead from having one more package installed on > image, but it would be a bit easier if we later decide that xuser also > needs some file to be installed in his home or something like that. Yes, please; the current situation makes it rather awkward if you need to tweak some of the xuser parameters in a distro specific way. But perhaps it should be called something other than xuser? Tomas
On 4 July 2013 09:54, Tomas Frydrych <tf+lists.yocto@r-finger.com> wrote: >> I know there is some overhead from having one more package installed on >> image, but it would be a bit easier if we later decide that xuser also >> needs some file to be installed in his home or something like that. > > Yes, please; the current situation makes it rather awkward if you need > to tweak some of the xuser parameters in a distro specific way. But > perhaps it should be called something other than xuser? Wayland likes to have a non-root user start it up, so make that three locations and another reason for it not being called "xuser". Ross