Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2215)

Unified Diff: remoting/host/installer/mac/do_signing.sh

Issue 10243015: [Chromoting] Sign uninstaller and fixup user-visible package and dmg names. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | remoting/remoting.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/installer/mac/do_signing.sh
===================================================================
--- remoting/host/installer/mac/do_signing.sh (revision 134295)
+++ remoting/host/installer/mac/do_signing.sh (working copy)
@@ -15,7 +15,8 @@
set -e -u
# Binaries to sign.
-ME2ME_HOST=PrivilegedHelperTools/org.chromium.chromoting.me2me_host
+ME2ME_HOST='PrivilegedHelperTools/org.chromium.chromoting.me2me_host'
+UNINSTALLER='Applications/@@HOST_UNINSTALLER_NAME@@.app'
# Iceberg creates this directory to write its output.
PKG_DIR=build
@@ -29,13 +30,13 @@
PKGPROJ_HOST_SERVICE='ChromotingHostService.packproj'
PKGPROJ_HOST_UNINSTALLER='ChromotingHostUninstaller.packproj'
-# Final mpkg name (for Official builds).
-PKG_FINAL='ChromeRemoteDesktopHost.mpkg'
+# Final (user-visible) mpkg name.
+PKG_FINAL='@@HOST_PKG@@.mpkg'
DMG_TEMP=dmg_tmp
-DMG_NAME='Chrome Remote Desktop'
+DMG_NAME='@@DMG_NAME@@'
DMG_DIR="${DMG_TEMP}/${DMG_NAME}"
-DMG_FILENAME='Chrome Remote Desktop.dmg'
+DMG_FILENAME='@@DMG_NAME@@.dmg'
ME="$(basename "${0}")"
readonly ME
@@ -72,26 +73,32 @@
shopt -u nullglob dotglob
if [[ ${#dir_contents[@]} -ne 0 ]]; then
- err "output directory must be empty"
+ err "Output directory must be empty"
exit 1
fi
}
-sign_binaries() {
- local input_dir="${1}"
+sign() {
+ local name="${1}"
local keychain="${2}"
local id="${3}"
- me2me_host="${input_dir}/${ME2ME_HOST}"
- if [[ ! -f "${me2me_host}" ]]; then
- err_exit "Input file doesn't exist: ${me2me_host}"
+ if [[ ! -e "${name}" ]]; then
+ err_exit "Input file doesn't exist: ${name}"
fi
- echo Signing "${me2me_host}"
- codesign -vv -s "${id}" --keychain "${keychain}" "${me2me_host}"
+ echo Signing "${name}"
+ codesign -vv -s "${id}" --keychain "${keychain}" "${name}"
+ codesign -v "${name}"
+}
- # Verify signing.
- codesign -v "${me2me_host}"
+sign_binaries() {
+ local input_dir="${1}"
+ local keychain="${2}"
+ local id="${3}"
+
+ sign "${input_dir}/${ME2ME_HOST}" "${keychain}" "${id}"
+ sign "${input_dir}/${UNINSTALLER}" "${keychain}" "${id}"
}
build_package() {
@@ -117,6 +124,7 @@
echo "Building .dmg..."
mkdir -p "${input_dir}/${DMG_DIR}/${PKG_FINAL}"
# Copy .mpkg installer.
+ echo "Copying ${input_dir}/${PKG_DIR}/${PKG_FINAL}"
ditto "${input_dir}/${PKG_DIR}/${PKG_FINAL}" \
"${input_dir}/${DMG_DIR}/${PKG_FINAL}"
# Copy .keystone_install script to top level of .dmg.
« no previous file with comments | « no previous file | remoting/remoting.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698