Skip to content

configure.ac syntax updates#109

Closed
jengelh wants to merge 4 commits intointel:mainfrom
jengelh:master
Closed

configure.ac syntax updates#109
jengelh wants to merge 4 commits intointel:mainfrom
jengelh:master

Conversation

@jengelh
Copy link
Copy Markdown
Contributor

@jengelh jengelh commented Jan 16, 2026

No description provided.

The manual says: "This macro should be used instead of plain ‘if’ in
code outside of an ‘AC_DEFUN’ macro, when the contents of the ‘if’
use ‘AC_REQUIRE’ directly or indirectly". Since one cannot know
whether something is going to be AC_REQUIRED indirectly, it is safer
to just always use AS_IF.

Signed-off-by: Jan Engelhardt <jengelh@inai.de>
The use of `-a` is non-portable, and it has been actively removed
from POSIX.1-2024
<https://pubs.opengroup.org/onlinepubs/9799919799/utilities/test.html>.

Signed-off-by: Jan Engelhardt <jengelh@inai.de>
Signed-off-by: Jan Engelhardt <jengelh@inai.de>
Signed-off-by: Jan Engelhardt <jengelh@inai.de>
@spandruvada
Copy link
Copy Markdown
Contributor

Merged to intel-lpmd-next branch.
Thanks!

@ptr1337
Copy link
Copy Markdown
Contributor

ptr1337 commented Apr 11, 2026

This seems to break the packaging of the systemd service.

Fix can be found here:
CachyOS@f25403f

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.

3 participants