diff options
| author |  Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-06-18 10:24:19 -0300 |
|---|---|---|
| committer | Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-06-25 10:04:47 -0300 |
| commit | cddf20ff1e69c85715e65394388f4ce912c01789 (patch) | |
| tree | bf99451d4f33afce9d71be136355807407b892f9 /src/so.mk | |
| parent | build: split misc flags from MANFLAGS (diff) | |
| download | firejail-cddf20ff1e69c85715e65394388f4ce912c01789.tar.gz firejail-cddf20ff1e69c85715e65394388f4ce912c01789.tar.zst firejail-cddf20ff1e69c85715e65394388f4ce912c01789.zip | |
build: remove redundant LDFLAGS in so.mk
Changes:
* Remove -fPIE, as it is mutually exclusive with -fPIC
* Remove -pie, as it is intended for executables (with -fPIE / -fpie)
* Remove duplicated `-z relro`
Note: The files built by the affected recipe are identical with and
without these changes when using gcc 13.1.1-1 on Artix Linux.
Diffstat (limited to 'src/so.mk')
| -rw-r--r-- | src/so.mk | 4 |
1 files changed, 2 insertions, 2 deletions
| @@ -16,7 +16,7 @@ SO_CFLAGS = \ | |||
| 16 | -fstack-protector-all -D_FORTIFY_SOURCE=2 \ | 16 | -fstack-protector-all -D_FORTIFY_SOURCE=2 \ |
| 17 | -fPIC | 17 | -fPIC |
| 18 | 18 | ||
| 19 | SO_LDFLAGS = -pie -fPIE -Wl,-z,relro -Wl,-z,now | 19 | SO_LDFLAGS = -fPIC -Wl,-z,relro -Wl,-z,now |
| 20 | 20 | ||
| 21 | .PHONY: all | 21 | .PHONY: all |
| 22 | all: $(TARGET) | 22 | all: $(TARGET) |
| @@ -25,7 +25,7 @@ all: $(TARGET) | |||
| 25 | $(CC) $(SO_CFLAGS) $(CFLAGS) $(INCLUDE) -c $< -o $@ | 25 | $(CC) $(SO_CFLAGS) $(CFLAGS) $(INCLUDE) -c $< -o $@ |
| 26 | 26 | ||
| 27 | $(SO): $(OBJS) $(ROOT)/config.mk | 27 | $(SO): $(OBJS) $(ROOT)/config.mk |
| 28 | $(CC) $(SO_LDFLAGS) -shared -fPIC -z relro $(LDFLAGS) -o $@ $(OBJS) -ldl | 28 | $(CC) $(SO_LDFLAGS) -shared $(LDFLAGS) -o $@ $(OBJS) -ldl |
| 29 | 29 | ||
| 30 | .PHONY: clean | 30 | .PHONY: clean |
| 31 | clean:; rm -fr $(SO) $(CLEANFILES) | 31 | clean:; rm -fr $(SO) $(CLEANFILES) |