* [RESEND PATCH] rootfs-postcommands: Remove spaces before ; in -ROOTFS_..._COMMAND
@ 2020-01-30 4:54 Alex Kiernan
2020-02-08 9:20 ` Alex Kiernan
2020-02-08 13:25 ` Richard Purdie
0 siblings, 2 replies; 4+ messages in thread
From: Alex Kiernan @ 2020-01-30 4:54 UTC (permalink / raw)
To: openembedded-core
With spaces in ROOTFS_POSTPROCESS_COMMAND attempts to remove elements
using ROOTFS_POSTPROCESS_COMMAND_remove causes parse failures of the
form:
bb.pysh.sherrors.ShellSyntaxError: LexToken(COMMA,';',0,0)
followed by:
LexToken(TOKEN,'write_image_test_data',0,0)
LexToken(COMMA,';',0,0)
LexToken(TOKEN,'set_systemd_default_target',0,0)
LexToken(COMMA,';',0,0)
LexToken(TOKEN,'systemd_create_users',0,0)
Removing the spaces between the function name and semicolon allows the
list to parse correctly.
Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
---
meta/classes/rootfs-postcommands.bbclass | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/meta/classes/rootfs-postcommands.bbclass b/meta/classes/rootfs-postcommands.bbclass
index a7f33fbf9e77..ebf96be7e85f 100644
--- a/meta/classes/rootfs-postcommands.bbclass
+++ b/meta/classes/rootfs-postcommands.bbclass
@@ -1,6 +1,6 @@
# Zap the root password if debug-tweaks feature is not enabled
-ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains_any("IMAGE_FEATURES", [ 'debug-tweaks', 'empty-root-password' ], "", "zap_empty_root_password ; ",d)}'
+ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains_any("IMAGE_FEATURES", [ 'debug-tweaks', 'empty-root-password' ], "", "zap_empty_root_password; ",d)}'
# Allow dropbear/openssh to accept logins from accounts with an empty password string if debug-tweaks or allow-empty-password is enabled
ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains_any("IMAGE_FEATURES", [ 'debug-tweaks', 'allow-empty-password' ], "ssh_allow_empty_password; ", "",d)}'
@@ -12,7 +12,7 @@ ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains_any("IMAGE_FEATURES", [ 'deb
ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains_any("IMAGE_FEATURES", [ 'debug-tweaks', 'post-install-logging' ], "postinst_enable_logging; ", "",d)}'
# Create /etc/timestamp during image construction to give a reasonably sane default time setting
-ROOTFS_POSTPROCESS_COMMAND += "rootfs_update_timestamp ; "
+ROOTFS_POSTPROCESS_COMMAND += "rootfs_update_timestamp; "
# Tweak the mount options for rootfs in /etc/fstab if read-only-rootfs is enabled
ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains("IMAGE_FEATURES", "read-only-rootfs", "read_only_rootfs_hook; ", "",d)}'
@@ -26,11 +26,11 @@ ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains("IMAGE_FEATURES", "read-only
APPEND_append = '${@bb.utils.contains("IMAGE_FEATURES", "read-only-rootfs", " ro", "", d)}'
# Generates test data file with data store variables expanded in json format
-ROOTFS_POSTPROCESS_COMMAND += "write_image_test_data ; "
+ROOTFS_POSTPROCESS_COMMAND += "write_image_test_data; "
# Write manifest
IMAGE_MANIFEST = "${IMGDEPLOYDIR}/${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.manifest"
-ROOTFS_POSTUNINSTALL_COMMAND =+ "write_image_manifest ; "
+ROOTFS_POSTUNINSTALL_COMMAND =+ "write_image_manifest; "
# Set default postinst log file
POSTINST_LOGFILE ?= "${localstatedir}/log/postinstall.log"
# Set default target for systemd images
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [RESEND PATCH] rootfs-postcommands: Remove spaces before ; in -ROOTFS_..._COMMAND
2020-01-30 4:54 [RESEND PATCH] rootfs-postcommands: Remove spaces before ; in -ROOTFS_..._COMMAND Alex Kiernan
@ 2020-02-08 9:20 ` Alex Kiernan
2020-02-08 13:25 ` Richard Purdie
1 sibling, 0 replies; 4+ messages in thread
From: Alex Kiernan @ 2020-02-08 9:20 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
On Thu, Jan 30, 2020 at 4:54 AM Alex Kiernan <alex.kiernan@gmail.com> wrote:
>
> With spaces in ROOTFS_POSTPROCESS_COMMAND attempts to remove elements
> using ROOTFS_POSTPROCESS_COMMAND_remove causes parse failures of the
> form:
>
> bb.pysh.sherrors.ShellSyntaxError: LexToken(COMMA,';',0,0)
> followed by:
> LexToken(TOKEN,'write_image_test_data',0,0)
> LexToken(COMMA,';',0,0)
> LexToken(TOKEN,'set_systemd_default_target',0,0)
> LexToken(COMMA,';',0,0)
> LexToken(TOKEN,'systemd_create_users',0,0)
>
> Removing the spaces between the function name and semicolon allows the
> list to parse correctly.
>
> Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
> ---
>
> meta/classes/rootfs-postcommands.bbclass | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/meta/classes/rootfs-postcommands.bbclass b/meta/classes/rootfs-postcommands.bbclass
> index a7f33fbf9e77..ebf96be7e85f 100644
> --- a/meta/classes/rootfs-postcommands.bbclass
> +++ b/meta/classes/rootfs-postcommands.bbclass
> @@ -1,6 +1,6 @@
>
> # Zap the root password if debug-tweaks feature is not enabled
> -ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains_any("IMAGE_FEATURES", [ 'debug-tweaks', 'empty-root-password' ], "", "zap_empty_root_password ; ",d)}'
> +ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains_any("IMAGE_FEATURES", [ 'debug-tweaks', 'empty-root-password' ], "", "zap_empty_root_password; ",d)}'
>
> # Allow dropbear/openssh to accept logins from accounts with an empty password string if debug-tweaks or allow-empty-password is enabled
> ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains_any("IMAGE_FEATURES", [ 'debug-tweaks', 'allow-empty-password' ], "ssh_allow_empty_password; ", "",d)}'
> @@ -12,7 +12,7 @@ ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains_any("IMAGE_FEATURES", [ 'deb
> ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains_any("IMAGE_FEATURES", [ 'debug-tweaks', 'post-install-logging' ], "postinst_enable_logging; ", "",d)}'
>
> # Create /etc/timestamp during image construction to give a reasonably sane default time setting
> -ROOTFS_POSTPROCESS_COMMAND += "rootfs_update_timestamp ; "
> +ROOTFS_POSTPROCESS_COMMAND += "rootfs_update_timestamp; "
>
> # Tweak the mount options for rootfs in /etc/fstab if read-only-rootfs is enabled
> ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains("IMAGE_FEATURES", "read-only-rootfs", "read_only_rootfs_hook; ", "",d)}'
> @@ -26,11 +26,11 @@ ROOTFS_POSTPROCESS_COMMAND += '${@bb.utils.contains("IMAGE_FEATURES", "read-only
> APPEND_append = '${@bb.utils.contains("IMAGE_FEATURES", "read-only-rootfs", " ro", "", d)}'
>
> # Generates test data file with data store variables expanded in json format
> -ROOTFS_POSTPROCESS_COMMAND += "write_image_test_data ; "
> +ROOTFS_POSTPROCESS_COMMAND += "write_image_test_data; "
>
> # Write manifest
> IMAGE_MANIFEST = "${IMGDEPLOYDIR}/${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.manifest"
> -ROOTFS_POSTUNINSTALL_COMMAND =+ "write_image_manifest ; "
> +ROOTFS_POSTUNINSTALL_COMMAND =+ "write_image_manifest; "
> # Set default postinst log file
> POSTINST_LOGFILE ?= "${localstatedir}/log/postinstall.log"
> # Set default target for systemd images
Ping?
--
Alex Kiernan
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RESEND PATCH] rootfs-postcommands: Remove spaces before ; in -ROOTFS_..._COMMAND
2020-01-30 4:54 [RESEND PATCH] rootfs-postcommands: Remove spaces before ; in -ROOTFS_..._COMMAND Alex Kiernan
2020-02-08 9:20 ` Alex Kiernan
@ 2020-02-08 13:25 ` Richard Purdie
2020-02-08 19:49 ` Alex Kiernan
1 sibling, 1 reply; 4+ messages in thread
From: Richard Purdie @ 2020-02-08 13:25 UTC (permalink / raw)
To: Alex Kiernan, openembedded-core
On Thu, 2020-01-30 at 04:54 +0000, Alex Kiernan wrote:
> With spaces in ROOTFS_POSTPROCESS_COMMAND attempts to remove elements
> using ROOTFS_POSTPROCESS_COMMAND_remove causes parse failures of the
> form:
>
> bb.pysh.sherrors.ShellSyntaxError: LexToken(COMMA,';',0,0)
> followed by:
> LexToken(TOKEN,'write_image_test_data',0,0)
> LexToken(COMMA,';',0,0)
> LexToken(TOKEN,'set_systemd_default_target',0,0)
> LexToken(COMMA,';',0,0)
> LexToken(TOKEN,'systemd_create_users',0,0)
>
> Removing the spaces between the function name and semicolon allows
> the list to parse correctly.
>
> Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
> ---
>
> meta/classes/rootfs-postcommands.bbclass | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
This does sound like you're working around a bug in shlex :/
Cheers,
Richard
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RESEND PATCH] rootfs-postcommands: Remove spaces before ; in -ROOTFS_..._COMMAND
2020-02-08 13:25 ` Richard Purdie
@ 2020-02-08 19:49 ` Alex Kiernan
0 siblings, 0 replies; 4+ messages in thread
From: Alex Kiernan @ 2020-02-08 19:49 UTC (permalink / raw)
To: Richard Purdie; +Cc: Patches and discussions about the oe-core layer
On Sat, Feb 8, 2020 at 1:25 PM Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
>
> On Thu, 2020-01-30 at 04:54 +0000, Alex Kiernan wrote:
> > With spaces in ROOTFS_POSTPROCESS_COMMAND attempts to remove elements
> > using ROOTFS_POSTPROCESS_COMMAND_remove causes parse failures of the
> > form:
> >
> > bb.pysh.sherrors.ShellSyntaxError: LexToken(COMMA,';',0,0)
> > followed by:
> > LexToken(TOKEN,'write_image_test_data',0,0)
> > LexToken(COMMA,';',0,0)
> > LexToken(TOKEN,'set_systemd_default_target',0,0)
> > LexToken(COMMA,';',0,0)
> > LexToken(TOKEN,'systemd_create_users',0,0)
> >
> > Removing the spaces between the function name and semicolon allows
> > the list to parse correctly.
> >
> > Signed-off-by: Alex Kiernan <alex.kiernan@gmail.com>
> > ---
> >
> > meta/classes/rootfs-postcommands.bbclass | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
>
> This does sound like you're working around a bug in shlex :/
>
Yeah, sorry...
I'll see if I can go and have a proper look at the underlying problem!
--
Alex Kiernan
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-02-08 19:49 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-30 4:54 [RESEND PATCH] rootfs-postcommands: Remove spaces before ; in -ROOTFS_..._COMMAND Alex Kiernan
2020-02-08 9:20 ` Alex Kiernan
2020-02-08 13:25 ` Richard Purdie
2020-02-08 19:49 ` Alex Kiernan
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.