public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: Randolph Sapp <rs@ti.com>
To: <raj.khem@gmail.com>, <rs@ti.com>
Cc: <richard.purdie@linuxfoundation.org>,
	<mathieu.dubois-briand@bootlin.com>, <alex@linutronix.de>,
	<otavio@ossystems.com.br>, <kexin.hao@windriver.com>,
	<afd@ti.com>, <detheridge@ti.com>, <denis@denix.org>,
	<reatmon@ti.com>, <openembedded-core@lists.openembedded.org>,
	<vijayp@ti.com>
Subject: Re: [oe-core][PATCHv15 5/6] xuser-account: merge with xserver-nodm-init
Date: Fri, 6 Feb 2026 14:00:03 -0600	[thread overview]
Message-ID: <DG859CXX3LWP.2BXUQMRVSRAV@ti.com> (raw)
In-Reply-To: <CAMKF1sqEMoKd2U7sDVQNyPZH9Xd=MKx62c=o-iussWnVDk+zUg@mail.gmail.com>

On Fri Feb 6, 2026 at 12:15 PM CST, Khem Raj via lists.openembedded.org wrote:
> this also needs adjustments for fvwm and xscreensaver recipes in meta-oe
>
> On Mon, Feb 2, 2026 at 5:20 PM Randolph Sapp via lists.openembedded.org <rs=
> ti.com@lists.openembedded.org> wrote:

If everyone's alright with this version of the series I'd be more than happy to
submit the necessary patches.

- Randolph

>> From: Randolph Sapp <rs@ti.com>
>>
>> Merge the xuser-account recipe with xserver-nodm-init, since:
>>
>>  - This is the only recipe dependent on it
>>  - This matches the current behavior in weston-init
>>  - Running x11 as root is no longer a supported path in
>>    xserver-nodm-init
>>  - The display manager configuration and user account name are heavily
>>    connected
>>  - The existing method of integration ignored this connection
>>
>> Signed-off-by: Randolph Sapp <rs@ti.com>
>> ---
>>  meta/conf/distro/include/maintainers.inc      |  1 -
>>  .../xserver-nodm-init}/system-xuser.conf      |  0
>>  .../x11-common/xserver-nodm-init_3.0.bb       | 20 +++++++++---
>>  .../user-creation/xuser-account_0.1.bb        | 31 -------------------
>>  4 files changed, 15 insertions(+), 37 deletions(-)
>>  rename meta/{recipes-support/user-creation/files =>
>> recipes-graphics/x11-common/xserver-nodm-init}/system-xuser.conf (100%)
>>  delete mode 100644 meta/recipes-support/user-creation/
>> xuser-account_0.1.bb
>>
>> diff --git a/meta/conf/distro/include/maintainers.inc
>> b/meta/conf/distro/include/maintainers.inc
>> index a7425f7020..ec0a128901 100644
>> --- a/meta/conf/distro/include/maintainers.inc
>> +++ b/meta/conf/distro/include/maintainers.inc
>> @@ -935,7 +935,6 @@ RECIPE_MAINTAINER:pn-xserver-xf86-config = "Unassigned
>> <unassigned@yoctoproject.
>>  RECIPE_MAINTAINER:pn-xserver-xorg = "Unassigned <
>> unassigned@yoctoproject.org>"
>>  RECIPE_MAINTAINER:pn-xset = "Unassigned <unassigned@yoctoproject.org>"
>>  RECIPE_MAINTAINER:pn-xtrans = "Unassigned <unassigned@yoctoproject.org>"
>> -RECIPE_MAINTAINER:pn-xuser-account = "Randolph Sapp <rs@ti.com>"
>>  RECIPE_MAINTAINER:pn-xvinfo = "Unassigned <unassigned@yoctoproject.org>"
>>  RECIPE_MAINTAINER:pn-xwayland = "Unassigned <unassigned@yoctoproject.org
>> >"
>>  RECIPE_MAINTAINER:pn-xwininfo = "Unassigned <unassigned@yoctoproject.org
>> >"
>> diff --git a/meta/recipes-support/user-creation/files/system-xuser.conf
>> b/meta/recipes-graphics/x11-common/xserver-nodm-init/system-xuser.conf
>> similarity index 100%
>> rename from meta/recipes-support/user-creation/files/system-xuser.conf
>> rename to
>> meta/recipes-graphics/x11-common/xserver-nodm-init/system-xuser.conf
>> diff --git a/meta/recipes-graphics/x11-common/xserver-nodm-init_3.0.bb
>> b/meta/recipes-graphics/x11-common/xserver-nodm-init_3.0.bb
>> index ed056df72f..0e9fe1e65a 100644
>> --- a/meta/recipes-graphics/x11-common/xserver-nodm-init_3.0.bb
>> +++ b/meta/recipes-graphics/x11-common/xserver-nodm-init_3.0.bb
>> @@ -1,14 +1,15 @@
>> -SUMMARY = "Simple Xserver Init Script (no dm)"
>> +SUMMARY = "Simple Xserver Init Script and user account"
>>  LICENSE = "MIT"
>>  LIC_FILES_CHKSUM =
>> "file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
>>  SECTION = "x11"
>>
>>  SRC_URI = "file://emptty.conf.in \
>> -           file://default.desktop"
>> +           file://default.desktop \
>> +           file://system-xuser.conf"
>>
>>  S = "${UNPACKDIR}"
>>
>> -inherit features_check
>> +inherit features_check useradd
>>
>>  REQUIRED_DISTRO_FEATURES = "x11"
>>
>> @@ -26,10 +27,19 @@ do_install() {
>>
>>      sed -i "s:@NO_CURSOR_ARG@:${NO_CURSOR_ARG}:"
>> ${D}${sysconfdir}/emptty/conf
>>      sed -i "s:@BLANK_ARGS@:${BLANK_ARGS}:" ${D}${sysconfdir}/emptty/conf
>> +
>> +    install -D -m 0644 ${S}/system-xuser.conf
>> ${D}${sysconfdir}/dbus-1/system.d/system-xuser.conf
>>  }
>>
>>  FILES:${PN} = "${sysconfdir}/emptty/conf \
>> -               ${datadir}/xsessions/default.desktop"
>> +               ${datadir}/xsessions/default.desktop \
>> +               ${sysconfdir}/dbus-1/system.d/system-xuser.conf"
>> +
>> +USERADD_PACKAGES = "${PN}"
>> +USERADD_PARAM:${PN} = "--create-home \
>> +                       --groups
>> video,tty,audio,input,shutdown,disk,nopasswdlogin \
>> +                       --user-group xuser"
>> +GROUPADD_PARAM:${PN} = "-r nopasswdlogin"
>>
>> -RDEPENDS:${PN} = "emptty xuser-account"
>> +RDEPENDS:${PN} = "emptty"
>>  RPROVIDES:${PN} += "virtual-emptty-conf"
>> diff --git a/meta/recipes-support/user-creation/xuser-account_0.1.bb
>> b/meta/recipes-support/user-creation/xuser-account_0.1.bb
>> deleted file mode 100644
>> index f1e4cb674f..0000000000
>> --- a/meta/recipes-support/user-creation/xuser-account_0.1.bb
>> +++ /dev/null
>> @@ -1,31 +0,0 @@
>> -SUMMARY = "Creates an 'xuser' account used for running X11"
>> -LICENSE = "MIT"
>> -LIC_FILES_CHKSUM =
>> "file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
>> -
>> -SRC_URI = "file://system-xuser.conf"
>> -
>> -inherit allarch useradd
>> -
>> -S = "${UNPACKDIR}"
>> -
>> -do_configure() {
>> -    :
>> -}
>> -
>> -do_compile() {
>> -    :
>> -}
>> -
>> -do_install() {
>> -    install -D -m 0644 ${UNPACKDIR}/system-xuser.conf
>> ${D}${sysconfdir}/dbus-1/system.d/system-xuser.conf
>> -}
>> -
>> -FILES:${PN} = "${sysconfdir}/dbus-1/system.d/system-xuser.conf"
>> -
>> -USERADD_PACKAGES = "${PN}"
>> -USERADD_PARAM:${PN} = "--create-home \
>> -                       --groups
>> video,tty,audio,input,shutdown,disk,nopasswdlogin \
>> -                       --user-group xuser"
>> -GROUPADD_PARAM:${PN} = "-r nopasswdlogin"
>> -
>> -ALLOW_EMPTY:${PN} = "1"
>> --
>> 2.52.0
>>
>>
>> 
>>
>>



  reply	other threads:[~2026-02-06 20:00 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-03  1:20 [oe-core][PATCHv15 0/6] Display manager proposal for x11 and wayland rs
2026-02-03  1:20 ` [oe-core][PATCHv15 1/6] emptty: add version 0.15.0 rs
2026-02-12 16:45   ` Mathieu Dubois-Briand
2026-02-12 22:52     ` Randolph Sapp
2026-02-12 23:00       ` Richard Purdie
     [not found]     ` <1893A19D18D5A1FB.653184@lists.openembedded.org>
2026-02-12 23:48       ` Randolph Sapp
2026-02-26 18:09     ` Randolph Sapp
2026-02-26 18:14       ` Mathieu Dubois-Briand
2026-02-27  7:44         ` Mathieu Dubois-Briand
2026-02-27 17:59           ` Randolph Sapp
2026-02-03  1:20 ` [oe-core][PATCHv15 2/6] weston-init: convert to virtual-emptty-conf rs
2026-02-03  1:20 ` [oe-core][PATCHv15 3/6] weston: remove deprecated weston-start scripts rs
2026-02-03  1:20 ` [oe-core][PATCHv15 4/6] xserver-nodm-init: convert to virtual-emptty-conf rs
2026-02-03  1:20 ` [oe-core][PATCHv15 5/6] xuser-account: merge with xserver-nodm-init rs
2026-02-06 18:15   ` Khem Raj
2026-02-06 20:00     ` Randolph Sapp [this message]
     [not found]     ` <1891C0B4F3A2B8B7.419336@lists.openembedded.org>
2026-02-06 23:28       ` Randolph Sapp
     [not found]       ` <1891CC14A53C0CE5.426509@lists.openembedded.org>
2026-02-06 23:42         ` Randolph Sapp
2026-02-03  1:20 ` [oe-core][PATCHv15 6/6] xsessions: add unique desktop entries rs

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DG859CXX3LWP.2BXUQMRVSRAV@ti.com \
    --to=rs@ti.com \
    --cc=afd@ti.com \
    --cc=alex@linutronix.de \
    --cc=denis@denix.org \
    --cc=detheridge@ti.com \
    --cc=kexin.hao@windriver.com \
    --cc=mathieu.dubois-briand@bootlin.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=otavio@ossystems.com.br \
    --cc=raj.khem@gmail.com \
    --cc=reatmon@ti.com \
    --cc=richard.purdie@linuxfoundation.org \
    --cc=vijayp@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox