Commit 3b5581d4 authored by Lambros Lambrou's avatar Lambros Lambrou Committed by Commit Bot

[remoting] Do eu-strip on the copied binaries

This fixes a warning on the official Linux builders. When building the
remoting Debian package, don't strip output binaries in-place, as this
confuses the build system into thinking that .stamp files are outdated.

Instead, copy the binaries into the package staging area before running
eu-strip on the copied binaries.

Bug: 828970
Change-Id: I47f5ee43ecb3bf29bb7bc27844e2dd2dc171a190
Reviewed-on: https://chromium-review.googlesource.com/1054827Reviewed-by: default avatarJamie Walch <jamiewalch@chromium.org>
Commit-Queue: Lambros Lambrou <lambroslambrou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557754}
parent dc1996d3
......@@ -48,23 +48,26 @@ install:
"$(BUILD_DIR)/remoting/com.google.chrome.remote_assistance.json" \
"$(NATIVE_MESSAGING_DIR)/com.google.chrome.remote_assistance.json"
eu-strip -f "$(ME2ME_DEBUGFILE)" "$(ME2ME_PROGNAME)"
install "$(ME2ME_PROGNAME)" \
"$(INSTALL_DIR)/chrome-remote-desktop-host"
eu-strip -f "$(ME2ME_DEBUGFILE)" \
"$(INSTALL_DIR)/chrome-remote-desktop-host"
eu-strip -f "$(START_DEBUGFILE)" "$(START_PROGNAME)"
install "$(START_PROGNAME)" "$(INSTALL_DIR)/start-host"
eu-strip -f "$(START_DEBUGFILE)" "$(INSTALL_DIR)/start-host"
eu-strip -f "$(SESSION_WRAPPER_DEBUGFILE)" "$(SESSION_WRAPPER_PROGNAME)"
install "$(SESSION_WRAPPER_PROGNAME)" "$(INSTALL_DIR)/user-session"
eu-strip -f "$(SESSION_WRAPPER_DEBUGFILE)" \
"$(INSTALL_DIR)/user-session"
eu-strip -f "$(ME2ME_NM_DEBUGFILE)" "$(ME2ME_NM_PROGNAME)"
install "$(ME2ME_NM_PROGNAME)" "$(INSTALL_DIR)/native-messaging-host"
eu-strip -f "$(ME2ME_NM_DEBUGFILE)" \
"$(INSTALL_DIR)/native-messaging-host"
eu-strip -f "$(REMOTE_ASSISTANCE_DEBUGFILE)" \
"$(REMOTE_ASSISTANCE_PROGNAME)"
install "$(REMOTE_ASSISTANCE_PROGNAME)" \
"$(INSTALL_DIR)/remote-assistance-host"
eu-strip -f "$(REMOTE_ASSISTANCE_DEBUGFILE)" \
"$(INSTALL_DIR)/remote-assistance-host"
install -m 0644 \
"$(BUILD_DIR)/icudtl.dat" "$(INSTALL_DIR)/icudtl.dat"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment