Patchwork [0/4] Allow xuser to shutdown

login
register
mail settings
Submitter Laurentiu Palcu
Date July 3, 2013, 2:34 p.m.
Message ID <cover.1372861799.git.laurentiu.palcu@intel.com>
Download mbox
Permalink /patch/52901/
State New
Headers show

Pull-request

git://git.yoctoproject.org/poky-contrib lpalcu/b4345_shutdown_icon_rootless_x

Comments

Laurentiu Palcu - July 3, 2013, 2:34 p.m.
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(-)
Laurentiu Palcu - July 3, 2013, 3:40 p.m.
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(-)
>
Martin Jansa - July 3, 2013, 4:19 p.m.
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
Laurentiu Palcu - July 4, 2013, 7:24 a.m.
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
>
Tomas Frydrych - July 4, 2013, 8:54 a.m.
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
Ross Burton - July 4, 2013, 11:03 a.m.
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