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

Fix usb-devices to work under busybox #181

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fancsali
Copy link

The previous implementation relied on the -printf option for find, which is not implemented in BusyBox.

Replaced it with sed to find the basenae of files reliably.

@fancsali fancsali changed the title Fix usb-utils to work under busybox Fix usb-devices to work under busybox Dec 12, 2023
The previous implementation relied on the -printf option for find,
which is not implemented in BusyBox.

Replaced it with sed to find the basenae of files reliably.
@gregkh
Copy link
Owner

gregkh commented Jan 4, 2024

As the bot says, you need to sign off on your change :(

Copy link
Owner

@gregkh gregkh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needs a signed-off-by

@fancsali
Copy link
Author

fancsali commented Jan 6, 2024

@gregkh, thanks for reminding me.

Just to confirm, what exact meaning do we attach to a commit being signed-off in this project?

@fabioestevam
Copy link

@gregkh, thanks for reminding me.

Just to confirm, what exact meaning do we attach to a commit being signed-off in this project?

You need to add the following line to your path:

Signed-off-by: Your Name <youremail.com>

See the other commits as references.

@gregkh
Copy link
Owner

gregkh commented Jan 18, 2024 via email

@fancsali
Copy link
Author

@gregkh, did you, by any chance, get the opportunkty to update the documentation on this, please?

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.

None yet

3 participants