Skip to content

Commit

Permalink
build: move HAVE_SANDBOX_CHECK out of MANFLAGS
Browse files Browse the repository at this point in the history 
It is not used in the man pages.

This amends commit 108327c ("feature: build: add
--disable-sandbox-check configure flag", 2024-12-27) / PR #6592.
  • Loading branch information
@kmk3
kmk3 committed Jan 13, 2025
1 parent e6de48c commit 5c6fa6a
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions config.mk.in
View file
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ HAVE_CONTRIB_INSTALL=@HAVE_CONTRIB_INSTALL@
HAVE_FATAL_WARNINGS=@HAVE_FATAL_WARNINGS@
HAVE_GCOV=@HAVE_GCOV@
HAVE_MAN=@HAVE_MAN@
HAVE_SANDBOX_CHECK=@HAVE_SANDBOX_CHECK@

# MANFLAGS
HAVE_APPARMOR=@HAVE_APPARMOR@
Expand All @@ -44,7 +45,6 @@ HAVE_OUTPUT=@HAVE_OUTPUT@
HAVE_OVERLAYFS=@HAVE_OVERLAYFS@
HAVE_PRIVATE_HOME=@HAVE_PRIVATE_HOME@
HAVE_PRIVATE_LIB=@HAVE_PRIVATE_LIB@
HAVE_SANDBOX_CHECK=@HAVE_SANDBOX_CHECK@
HAVE_SELINUX=@HAVE_SELINUX@
HAVE_SUID=@HAVE_SUID@
HAVE_USERNS=@HAVE_USERNS@
Expand All @@ -66,7 +66,6 @@ MANFLAGS = \
$(HAVE_OVERLAYFS) \
$(HAVE_PRIVATE_HOME) \
$(HAVE_PRIVATE_LIB) \
$(HAVE_SANDBOX_CHECK) \
$(HAVE_SELINUX) \
$(HAVE_SUID) \
$(HAVE_USERNS) \
Expand Down

2 comments on commit 5c6fa6a

@powerjungle
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh sorry, I wasn't aware that "MANFLAGS" was regarding "man pages" and I didn't notice the "Misc flags" above it.

@powerjungle
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you think it would make sense to move it as a separate flag in PROG_CFLAGS, like how HAVE_GCOV acts?

Please sign in to comment.