Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PW_SID:797481] [v2] shared/shell: Fix --init-script commandline option #425

Closed
wants to merge 2 commits into from

Conversation

tedd-an
Copy link
Owner

@tedd-an tedd-an commented Oct 30, 2023

The newly added option -i/--init-script introduced a short option
namespace collision with btmgmt's --index, both of which use '-i'.

As a result, a provided --index is treated as a file name:

$ sudo btmgmt --index 0 info
Unable to open 0: No such file or directory (2)

Fix this by using '-s' for --init-script.

Fixes: bluez/bluez#639
Signed-off-by: Juerg Haefliger juerg.haefliger@canonical.com


v2:

  • Replace reference to broken commit with reference to github issue.

src/shared/shell.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

tedd-an and others added 2 commits October 24, 2023 18:47
The newly added option -i/--init-script introduced a short option
namespace collision with btmgmt's --index, both of which use '-i'.

As a result, a provided --index is treated as a file name:

$ sudo btmgmt --index 0 info
Unable to open 0: No such file or directory (2)

Fix this by using '-s' for --init-script.

Fixes: bluez/bluez#639
Signed-off-by: Juerg Haefliger <juerg.haefliger@canonical.com>
@github-actions github-actions bot force-pushed the workflow branch 3 times, most recently from 3a0d959 to f0acca1 Compare November 13, 2023 19:17
@tedd-an tedd-an closed this Nov 13, 2023
@tedd-an tedd-an deleted the 797481 branch November 13, 2023 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants