All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: Karthik Ramanan <a0393906@ti.com>
Cc: meta-arago@arago-project.org
Subject: Re: weston: Add custom profile into targetfs
Date: Thu, 9 Apr 2015 17:16:14 -0400	[thread overview]
Message-ID: <20150409211614.GB6414@edge> (raw)
In-Reply-To: <1428582873-438-1-git-send-email-a0393906@ti.com>

Karthik,

Looks better. Now, can you just merge the existing wayland_env and the new 
profile you are adding? Both of them are installed in /etc/profile.d and get 
sourced on every login. Just add the mkdir and chmod part to the end of 
existing wayland_env.sh and that's it. Also, since you are changing wayldn_env 
file anyway, set XDG_RUNTIME_DIR to the same value of /tmp/${UID}-runtime-dir?

-- 
Denys


On Thu, Apr 09, 2015 at 06:04:33PM +0530, Karthik Ramanan wrote:
> There are different set of requirements on the bootup behavior
> across different platforms. To name a few:
>  * J6, the required behavior is console/command prompt
>  * AM57xx, it will be the matrix-gui (QT5 on Wayland)
> 
> weston-init recipe addresses the needs of AM57xx.
> This patch is needed for J6/J6-Eco etc.
> 
> Note that if there are customers on AM57xx who need the default behavior
> to be console/command prompt, this patch will address those needs.
> It does not conflict with weston-init.
> 
> Signed-off-by: Karthik Ramanan <a0393906@ti.com>
> ---
>  .../recipes-graphics/wayland/weston/profile        |    8 ++++++++
>  .../recipes-graphics/wayland/weston/wayland_env.sh |    2 +-
>  .../recipes-graphics/wayland/weston_1.6.0.bbappend |    4 +++-
>  3 files changed, 12 insertions(+), 2 deletions(-)
>  create mode 100644 meta-arago-distro/recipes-graphics/wayland/weston/profile
> 
> diff --git a/meta-arago-distro/recipes-graphics/wayland/weston/profile b/meta-arago-distro/recipes-graphics/wayland/weston/profile
> new file mode 100644
> index 0000000..0848c18
> --- /dev/null
> +++ b/meta-arago-distro/recipes-graphics/wayland/weston/profile
> @@ -0,0 +1,8 @@
> +if test -z "${XDG_RUNTIME_DIR}"; then
> +       export XDG_RUNTIME_DIR=/tmp/${UID}-runtime-dir
> +fi
> +
> +if ! test -d "${XDG_RUNTIME_DIR}"; then
> +       mkdir -p "${XDG_RUNTIME_DIR}"
> +       chmod 0700 "${XDG_RUNTIME_DIR}"
> +fi
> diff --git a/meta-arago-distro/recipes-graphics/wayland/weston/wayland_env.sh b/meta-arago-distro/recipes-graphics/wayland/weston/wayland_env.sh
> index a0e3b90..70a7d17 100644
> --- a/meta-arago-distro/recipes-graphics/wayland/weston/wayland_env.sh
> +++ b/meta-arago-distro/recipes-graphics/wayland/weston/wayland_env.sh
> @@ -3,6 +3,6 @@
>  ### Wayland Environment Variables ###
>  
>  export XDG_CONFIG_HOME=/etc/
> -export XDG_RUNTIME_DIR=/run/user/root
> +export XDG_RUNTIME_DIR=/tmp
>  export WAYLAND_DISPLAY=wayland-0
>  export WS_CALUDEV_FILE=/etc/udev/rules.d/ws-calibrate.rules
> diff --git a/meta-arago-distro/recipes-graphics/wayland/weston_1.6.0.bbappend b/meta-arago-distro/recipes-graphics/wayland/weston_1.6.0.bbappend
> index b0e8985..e3a9089 100644
> --- a/meta-arago-distro/recipes-graphics/wayland/weston_1.6.0.bbappend
> +++ b/meta-arago-distro/recipes-graphics/wayland/weston_1.6.0.bbappend
> @@ -2,13 +2,14 @@
>  PACKAGECONFIG[fbdev] = "--enable-fbdev-compositor WESTON_NATIVE_BACKEND="fbdev-backend.so",--disable-fbdev-compositor,udev mtdev"
>  PACKAGECONFIG[kms] = "--enable-drm-compositor,--disable-drm-compositor,drm udev libgbm mtdev"
>  
> -PR_append = "-arago3"
> +PR_append = "-arago4"
>  
>  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>  
>  SRC_URI += "file://wayland_env.sh \
>              file://weston.ini \
>              file://0001-weston-Enabling-DRM-backend-with-multiple-displays.patch \
> +            file://profile \
>  "
>  
>  # Add custom Arago Wayland Environment script file
> @@ -16,6 +17,7 @@ do_install_append () {
>      install -d ${D}${sysconfdir}/profile.d
>      install -m 0644 ${WORKDIR}/weston.ini ${D}${sysconfdir}
>      install -m 0644 ${WORKDIR}/wayland_env.sh ${D}${sysconfdir}/profile.d/
> +    install -m 0644 ${WORKDIR}/profile ${D}${sysconfdir}/profile.d/weston_profile
>  }
>  
>  PACKAGES += "${PN}-conf"
> -- 
> 1.7.9.5
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


  reply	other threads:[~2015-04-09 21:16 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-09 12:34 weston: Add custom profile into targetfs Karthik Ramanan
2015-04-09 21:16 ` Denys Dmytriyenko [this message]
2015-04-10 12:52   ` Karthik Ramanan
2015-04-10 17:44     ` Denys Dmytriyenko

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=20150409211614.GB6414@edge \
    --to=denys@ti.com \
    --cc=a0393906@ti.com \
    --cc=meta-arago@arago-project.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.