Skip to content

post_install.sh: Use uaccess tag in udev rules - #711

Open
tormodvolden wants to merge 1 commit into
arduino:mainfrom
tormodvolden:udev_rules
Open

tormodvolden wants to merge 1 commit into
arduino:mainfrom
tormodvolden:udev_rules

Conversation

@tormodvolden

@tormodvolden tormodvolden commented Aug 17, 2023

Copy link
Copy Markdown

Do not make device node world-writeable, instead give access to desktop user.

Also use one "here" document instead of mixing in echo statements. Customarily keep "then" on same line as "if".

@CLAassistant

CLAassistant commented Aug 17, 2023

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@tormodvolden
tormodvolden marked this pull request as draft August 18, 2023 06:06
Do not make the device node world-writeable, instead give access to the
current desktop user.

Also use one "here" document instead of mixing in echo statements, and
customarily keep "then" on same line as "if".

Signed-off-by: Tormod Volden <debian.tormod@gmail.com>
@tormodvolden

Copy link
Copy Markdown
Author

(fixed up commit message)

@tormodvolden
tormodvolden marked this pull request as ready for review August 18, 2023 19:48
@tormodvolden

Copy link
Copy Markdown
Author

Signed the CLA with the help of this: cla-assistant/cla-assistant#730 (comment)

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.

2 participants