diff options
| author | Thomas Petazzoni <thomas.petazzoni@free-electrons.com> | 2013-08-20 13:03:03 +0200 |
|---|---|---|
| committer | Peter Korsgaard <jacmet@sunsite.dk> | 2013-09-02 23:06:19 +0200 |
| commit | 52e7073a58409ac09b79865a3ae4332cf01edb58 (patch) | |
| tree | 190654437bb74d0775249a2e8fd5198388efcb3a /Makefile | |
| parent | 4ce8af8632894db3e4ed25d9d3230359a2cc640c (diff) | |
| download | buildroot-52e7073a58409ac09b79865a3ae4332cf01edb58.tar.gz buildroot-52e7073a58409ac09b79865a3ae4332cf01edb58.zip | |
Makefile: do not try to strip inexisting file when stripping libpthread
libpthread.so has a special stripping condition to preserve parts of
it that are needed for debugging. However, due to the usage of 'xargs'
instead of 'xargs -r', the strip command is executed regardless of
whether a libpthread.so file is found or not. This leads to a big
error message being displayed in static-only builds, because strip is
executed without a file argument. Thanks to the '|| true', the build
continues, but still shows a big error message in the middle, which is
not nice.
By using 'xargs -r', we avoid the strip command from being executed
when 'find' doesn't find any match. We also remove the '|| true' to
catch other real errors.
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
Diffstat (limited to 'Makefile')
| -rw-r--r-- | Makefile | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -494,7 +494,7 @@ endif # I set a breakpoint" ifeq ($(BR2_TOOLCHAIN_HAS_THREADS),y) find $(TARGET_DIR)/lib -type f -name 'libpthread*.so*' | \ - xargs $(STRIPCMD) $(STRIP_STRIP_DEBUG) || true + xargs -r $(STRIPCMD) $(STRIP_STRIP_DEBUG) endif mkdir -p $(TARGET_DIR)/etc |
