On 14 Jun 2022, at 16:11, Paulo Neves via lists.openembedded.org
<ptsneves=gmail....@lists.openembedded.org> wrote:
> +create_cmdline_shebang_wrapper () {
The indentation in this function is a mix of tabs and spaces, please unify.
> + argument="$(basename "$(head -n1 $cmd | sed -e 's|#![ ]*||g' )”)”
[ ]* seems overly, complex, no need for the brackets. I’d anchor to the
beginning of the line, and remove the g as there’s only one expected.
Might also be wise to assert that you found something here, so running this on
an ELF fails with an obvious error.
Ross
IMPORTANT NOTICE: The contents of this email and any attachments are
confidential and may also be privileged. If you are not the intended recipient,
please notify the sender immediately and do not disclose the contents to any
other person, use it for any purpose, or store or copy the information in any
medium. Thank you.
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#166929):
https://lists.openembedded.org/g/openembedded-core/message/166929
Mute This Topic: https://lists.openembedded.org/mt/91751270/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
