Skip to content

Improve spaces handling in driver names#69

Open
elliotberman wants to merge 2 commits intolinux-msm:masterfrom
elliotberman:master
Open

Improve spaces handling in driver names#69
elliotberman wants to merge 2 commits intolinux-msm:masterfrom
elliotberman:master

Conversation

@elliotberman
Copy link

@elliotberman elliotberman commented Jun 11, 2025

I was running bootrr for a board that has spaces in the driver name, and I get the following complaints:

/libexec/bootrr/helpers/timeout: line 13: [: /sys/bus/foo/drivers/bar: binary operator expected

The driver name in this case is "bar baz".

Further, the rule generator is unhappy trying to cd /sys/bus/foo/drivers/bar baz/qux.

Remove the intermediate "cmd" argument so that spaces in passed
arguments are preserved.

Signed-off-by: Elliot Berman <eberman@anduril.com>
`cd ${dev}` prevents drivers (or devices) with spaces in their names
from being cd'd to. Add quotes to allow spaces in the driver names.

Signed-off-by: Elliot Berman <eberman@anduril.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant