Skip to content

data: fix meson.build - #540

Merged
trigg merged 1 commit into
WayfireWM:masterfrom
hustlerone:master
Aug 9, 2026
Merged

data: fix meson.build#540
trigg merged 1 commit into
WayfireWM:masterfrom
hustlerone:master

Conversation

@hustlerone

Copy link
Copy Markdown
Contributor

hardcoded paths are le bad

@trigg

trigg commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

In the case of XDPW I think this will work, but for PAM this would only break it.

We would need it to be in /etc/pam.d or /usr/lib/pam.d but not at the whims of prefix locations.

@hustlerone

Copy link
Copy Markdown
Contributor Author

So, what do we do then? This is what Meson says:
https://mesonbuild.com/Builtin-options.html

and normally it would be /etc/pam.d/

@trigg

trigg commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

Fair play, that's on me, I misread it as another variable. Well it's worth a look at. Do we know any use-cases we can test where sysconfdir is set to something else and PAM sees it the same way?

@hustlerone

Copy link
Copy Markdown
Contributor Author

You can override where sysconfdir points to when running iirc. By default it's /etc/ or whatever the distro chooses for sysconf directory to be

@trigg

trigg commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

Yes, the more I read into it the more I'm sure you're correct.

Thanks for being patient with me!

@trigg
trigg merged commit bcb6521 into WayfireWM:master Aug 9, 2026
2 checks passed
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