* [PATCH 1/3] x11-common: Fix XDG_RUNTIME_DIR typo
@ 2016-06-09 19:27 Tom Hochstein
2016-06-09 19:27 ` [PATCH 2/3] weston-init: Use bitbake path variable Tom Hochstein
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Tom Hochstein @ 2016-06-09 19:27 UTC (permalink / raw)
To: openembedded-core
Fix the if-statement that had misspelled XDG_RUNTIME_DIR.
Also fix whitespace.
Signed-off-by: Tom Hochstein <tom.hochstein@nxp.com>
---
.../x11-common/x11-common/etc/X11/Xsession.d/13xdgbasedirs.sh | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-graphics/x11-common/x11-common/etc/X11/Xsession.d/13xdgbasedirs.sh b/meta/recipes-graphics/x11-common/x11-common/etc/X11/Xsession.d/13xdgbasedirs.sh
index 91594b9..6bd40b2 100644
--- a/meta/recipes-graphics/x11-common/x11-common/etc/X11/Xsession.d/13xdgbasedirs.sh
+++ b/meta/recipes-graphics/x11-common/x11-common/etc/X11/Xsession.d/13xdgbasedirs.sh
@@ -7,7 +7,7 @@
# In the ideal case where SystemD is booting and userspace is initiated by a
# SystemD user session this will have been set already, so don't overwrite it.
-if [ -z "$XGD_RUNTIME_DIR" ]; then
- XDG_RUNTIME_DIR="/tmp"
- export XDG_RUNTIME_DIR
+if [ -z "$XDG_RUNTIME_DIR" ]; then
+ XDG_RUNTIME_DIR="/tmp"
+ export XDG_RUNTIME_DIR
fi
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/3] weston-init: Use bitbake path variable
2016-06-09 19:27 [PATCH 1/3] x11-common: Fix XDG_RUNTIME_DIR typo Tom Hochstein
@ 2016-06-09 19:27 ` Tom Hochstein
2016-06-10 13:48 ` Otavio Salvador
2016-06-09 19:27 ` [PATCH 3/3] weston-init: Don't change XDG_RUNTIME_DIR if it already exists Tom Hochstein
2016-06-10 13:48 ` [PATCH 1/3] x11-common: Fix XDG_RUNTIME_DIR typo Otavio Salvador
2 siblings, 1 reply; 6+ messages in thread
From: Tom Hochstein @ 2016-06-09 19:27 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Tom Hochstein <tom.hochstein@nxp.com>
---
meta/recipes-graphics/wayland/weston-init.bb | 1 +
meta/recipes-graphics/wayland/weston-init/weston-start | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/recipes-graphics/wayland/weston-init.bb b/meta/recipes-graphics/wayland/weston-init.bb
index e4e2701..291cd16 100644
--- a/meta/recipes-graphics/wayland/weston-init.bb
+++ b/meta/recipes-graphics/wayland/weston-init.bb
@@ -15,6 +15,7 @@ do_install() {
# Install weston-start script
install -Dm755 ${WORKDIR}/weston-start ${D}${bindir}/weston-start
sed -i 's,@DATADIR@,${datadir},g' ${D}${bindir}/weston-start
+ sed -i 's,@LOCALSTATEDIR@,${localstatedir},g' ${D}${bindir}/weston-start
}
inherit allarch update-rc.d distro_features_check systemd
diff --git a/meta/recipes-graphics/wayland/weston-init/weston-start b/meta/recipes-graphics/wayland/weston-init/weston-start
index 72ba4b7..4b41576 100755
--- a/meta/recipes-graphics/wayland/weston-init/weston-start
+++ b/meta/recipes-graphics/wayland/weston-init/weston-start
@@ -69,4 +69,4 @@ if test -z "$XDG_RUNTIME_DIR"; then
chmod 0700 $XDG_RUNTIME_DIR
fi
-exec openvt $openvt_args -- $launcher $weston_args --log=/var/log/weston.log
+exec openvt $openvt_args -- $launcher $weston_args --log=@LOCALSTATEDIR@/log/weston.log
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 3/3] weston-init: Don't change XDG_RUNTIME_DIR if it already exists
2016-06-09 19:27 [PATCH 1/3] x11-common: Fix XDG_RUNTIME_DIR typo Tom Hochstein
2016-06-09 19:27 ` [PATCH 2/3] weston-init: Use bitbake path variable Tom Hochstein
@ 2016-06-09 19:27 ` Tom Hochstein
2016-06-10 13:49 ` Otavio Salvador
2016-06-10 13:48 ` [PATCH 1/3] x11-common: Fix XDG_RUNTIME_DIR typo Otavio Salvador
2 siblings, 1 reply; 6+ messages in thread
From: Tom Hochstein @ 2016-06-09 19:27 UTC (permalink / raw)
To: openembedded-core
In the case that XDG_RUNTIME_DIR already exists, the mkdir --parents
at least does nothing, but the chmod could overwrite the desired
file mode settings.
Signed-off-by: Tom Hochstein <tom.hochstein@nxp.com>
---
meta/recipes-graphics/wayland/weston-init/weston-start | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-graphics/wayland/weston-init/weston-start b/meta/recipes-graphics/wayland/weston-init/weston-start
index 4b41576..5b7604f 100755
--- a/meta/recipes-graphics/wayland/weston-init/weston-start
+++ b/meta/recipes-graphics/wayland/weston-init/weston-start
@@ -65,8 +65,10 @@ fi
if test -z "$XDG_RUNTIME_DIR"; then
export XDG_RUNTIME_DIR=/run/user/`id -u`
- mkdir --parents $XDG_RUNTIME_DIR
- chmod 0700 $XDG_RUNTIME_DIR
+ if ! test -d "$XDG_RUNTIME_DIR"; then
+ mkdir --parents $XDG_RUNTIME_DIR
+ chmod 0700 $XDG_RUNTIME_DIR
+ fi
fi
exec openvt $openvt_args -- $launcher $weston_args --log=@LOCALSTATEDIR@/log/weston.log
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 1/3] x11-common: Fix XDG_RUNTIME_DIR typo
2016-06-09 19:27 [PATCH 1/3] x11-common: Fix XDG_RUNTIME_DIR typo Tom Hochstein
2016-06-09 19:27 ` [PATCH 2/3] weston-init: Use bitbake path variable Tom Hochstein
2016-06-09 19:27 ` [PATCH 3/3] weston-init: Don't change XDG_RUNTIME_DIR if it already exists Tom Hochstein
@ 2016-06-10 13:48 ` Otavio Salvador
2 siblings, 0 replies; 6+ messages in thread
From: Otavio Salvador @ 2016-06-10 13:48 UTC (permalink / raw)
To: Tom Hochstein; +Cc: Patches and discussions about the oe-core layer
On Thu, Jun 9, 2016 at 4:27 PM, Tom Hochstein <tom.hochstein@nxp.com> wrote:
> Fix the if-statement that had misspelled XDG_RUNTIME_DIR.
> Also fix whitespace.
>
> Signed-off-by: Tom Hochstein <tom.hochstein@nxp.com>
Acked-by: Otavio Salvador <otavio@ossystems.com.br>
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-06-10 13:49 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-09 19:27 [PATCH 1/3] x11-common: Fix XDG_RUNTIME_DIR typo Tom Hochstein
2016-06-09 19:27 ` [PATCH 2/3] weston-init: Use bitbake path variable Tom Hochstein
2016-06-10 13:48 ` Otavio Salvador
2016-06-09 19:27 ` [PATCH 3/3] weston-init: Don't change XDG_RUNTIME_DIR if it already exists Tom Hochstein
2016-06-10 13:49 ` Otavio Salvador
2016-06-10 13:48 ` [PATCH 1/3] x11-common: Fix XDG_RUNTIME_DIR typo Otavio Salvador
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.