Skip to content

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

Open
tormodvolden wants to merge 1 commit intoarduino:mainfrom
tormodvolden:udev_rules
Open

post_install.sh: Use uaccess tag in udev rules#711
tormodvolden wants to merge 1 commit intoarduino:mainfrom
tormodvolden:udev_rules

Conversation

@tormodvolden
Copy link
Copy Markdown

@tormodvolden tormodvolden commented Aug 17, 2023

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
Copy link
Copy Markdown

CLAassistant commented Aug 17, 2023

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