Skip to content

wpa_supplicant: readd a patch, create user - #62014

Open
dkwo wants to merge 1 commit into
void-linux:masterfrom
dkwo:wpas-212-fix
Open

wpa_supplicant: readd a patch, create user#62014
dkwo wants to merge 1 commit into
void-linux:masterfrom
dkwo:wpas-212-fix

Conversation

@dkwo

@dkwo dkwo commented Aug 14, 2026

Copy link
Copy Markdown
Contributor
  • I tested the changes in this PR: YES
  • I built this PR locally for my native architecture, (aarch64-gnu)

Without this patch, wifi connection fails in certain cases (e.g. pixel hotspot to macbook air).
Whille at it, also add an unpriviledge user _wpas: this does nothing per se, but it makes life easier if you want to run the service unprivileged (otherwise permissions on conf files are rewritten at every update).

Comment thread srcpkgs/wpa_supplicant/INSTALL Outdated
chmod 750 /etc/wpa_supplicant

chown -R root:_wpas /etc/wpa_supplicant/
chmod -R u=rwX,g=rX,o= /etc/wpa_supplicant/

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

relative paths should be used in INSTALL scripts.

I'm also not the biggest fan of changing permissions of config files in INSTALL scripts.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done. But is there a better way?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

File permissions are part of the packaged files and applied when extracted.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think that setup needs a bit more consideration and precedence from other distributions, chowing *.conf also doesn't seem too nice.

I think its better to not mix this into a bug fix PR.

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