diff options
| author |  Kelvin M. Klann <kmk3.code@protonmail.com> | 2021-11-01 14:10:28 -0300 |
|---|---|---|
| committer | Kelvin M. Klann <kmk3.code@protonmail.com> | 2021-11-25 11:32:08 -0300 |
| commit | 6d42e13a46b970c7197e1b70f6440ada1f8ec410 (patch) | |
| tree | 799170ae63f7a6323cc674cd7025f3ad28f29402 /Makefile.in | |
| parent | configure*: Remove redundant AC_SUBST calls near HAVE_LTS (diff) | |
| download | firejail-6d42e13a46b970c7197e1b70f6440ada1f8ec410.tar.gz firejail-6d42e13a46b970c7197e1b70f6440ada1f8ec410.tar.zst firejail-6d42e13a46b970c7197e1b70f6440ada1f8ec410.zip | |
build: Normalize HAVE_SUID
See commit 15d793838 ("Try to fix #2310 -- Can't create run directory
without suid-root", 2021-05-13) / PR #4273.
It is the only "HAVE_" option whose value is set by if/else on a
makefile. Also, it is set in different places to either "yes", "no",
blank or "-DHAVE_SUID". Set the value only on configure.ac and only to
either blank or to "-DHAVE_SUID".
Misc: The `ifeq ($(HAVE_SUID),-DHAVE_SUID)` comparison that this adds is
based on the existing `ifeq ($(HAVE_APPARMOR),-DHAVE_APPARMOR)`
comparison on Makefile.in.
Diffstat (limited to 'Makefile.in')
| -rw-r--r-- | Makefile.in | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Makefile.in b/Makefile.in index ddc63c1af..1044f1ade 100644 --- a/Makefile.in +++ b/Makefile.in | |||
| @@ -105,7 +105,7 @@ realinstall: | |||
| 105 | # firejail executable | 105 | # firejail executable |
| 106 | install -m 0755 -d $(DESTDIR)$(bindir) | 106 | install -m 0755 -d $(DESTDIR)$(bindir) |
| 107 | install -m 0755 src/firejail/firejail $(DESTDIR)$(bindir) | 107 | install -m 0755 src/firejail/firejail $(DESTDIR)$(bindir) |
| 108 | ifeq ($(HAVE_SUID),yes) | 108 | ifeq ($(HAVE_SUID),-DHAVE_SUID) |
| 109 | chmod u+s $(DESTDIR)$(bindir)/firejail | 109 | chmod u+s $(DESTDIR)$(bindir)/firejail |
| 110 | endif | 110 | endif |
| 111 | # firemon executable | 111 | # firemon executable |