cros_workon: print out short summary of what has been done for start/stop

cros_workon

 #!/bin/bash
 
 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # This script moves ebuilds between 'stable' and 'live' states.
 # By default 'stable' ebuilds point at and build from source at the
 # last known good commit. Moving an ebuild to 'live' (via cros_workon start)
 # is intended to support development. The current source tip is fetched,
@@ skipping 41 matching lines @@
 # there's no way to override.
 [ -n "${FLAGS_board}" ] && [ "${FLAGS_host}" = ${FLAGS_TRUE} ] && \
   FLAGS_board="" # kill board
 [ -z "${FLAGS_board}" ] && [ "${FLAGS_host}" = ${FLAGS_FALSE} ] && \
   die "You must specify either --host or --board="
 
 if [ -n "${FLAGS_board}" ]; then
   BOARD_DIR=/build/"${FLAGS_board}" # --board specified
   EQUERYCMD=equery-"${FLAGS_board}"
   EBUILDCMD=ebuild-"${FLAGS_board}"
+  BOARD_STR="${FLAGS_board}"
 else
   BOARD_DIR="" # --host specified
   EQUERYCMD=equery
   EBUILDCMD=ebuild
+  BOARD_STR="host"
 fi
 
 WORKON_DIR=${CHROOT_TRUNK_DIR}/.config/cros_workon
 KEYWORDS_DIR=${BOARD_DIR}/etc/portage/package.keywords
 UNMASK_DIR=${BOARD_DIR}/etc/portage/package.unmask
 WORKON_FILE=${WORKON_DIR}/${FLAGS_board:-host}
 KEYWORDS_FILE=${KEYWORDS_DIR}/cros-workon
 UNMASK_FILE=${UNMASK_DIR}/cros-workon
 
 # TODO(msb): remove the backward compatibility after 09/01/2010
@@ skipping 117 matching lines @@
       | sort | uniq >> "${local_manifest}"
     echo "</manifest>" >> "${local_manifest}"
     echo "Please run \"repo sync\" now."
   fi
 }
 
 # Move a stable ebuild to the live development catgeory.  The ebuild
 # src_unpack step fetches the package source for local development.
 ebuild_to_live () {
   local atoms=$1
+  local atoms_success=""
 
   for atom in ${atoms}; do
     if ! grep -qx "[~=]${atom}-9999" "${WORKON_FILE}" ; then
-      sudo bash -c "echo \"=${atom}-9999\" >> \"${WORKON_FILE}\""
+      if sudo bash -c "echo \"=${atom}-9999\" >> \"${WORKON_FILE}\""; then
+        atoms_success="${atoms_success} ${atom}"
+      fi
     else
       warn "Already working on ${atom}"
     fi
   done
+  [ -n "${atoms_success}" ] && regen_manifest_and_sync && \
+    info "Started working on '${atoms_success/ /}' for '${BOARD_STR}'"
 }
 
 # Move a live development ebuild back to stable.
 ebuild_to_stable () {
   local atoms=$1
+  local atoms_success=""
 
   for atom in ${atoms}; do
     if grep -qx "[~=]${atom}-9999" "${WORKON_FILE}" ; then
-      sudo bash -c "grep -v '^[~=]${atom}-9999\$' \"${WORKON_FILE}\" > \
-          \"${WORKON_FILE}+\""
-      sudo mv "${WORKON_FILE}+" "${WORKON_FILE}"
+      if sudo sed -e "/^[~=]${atom/\//\\/}-9999\$/d" -i "${WORKON_FILE}"; then
+        atoms_success="${atoms_success} ${atom}"
+      fi
     else
       warn "Not working on ${atom}"
     fi
   done
+  [ -n "${atoms_success}" ] && \
+    info "Stopped working on '${atoms_success/ /}' for '${BOARD_STR}'"
 }
 
 # Run a command on all or a set of repos.
 ebuild_iterate() {
   local atoms=$1
 
   for atom in ${atoms}; do
     info "Running \"${FLAGS_command}\" on ${atom}"
     eval $(${EBUILDCMD} $(${EQUERYCMD} which ${atom}) info)
     (cd "${CROS_WORKON_SRCDIR}" && bash -c "${FLAGS_command}")
@@ skipping 15 matching lines @@
       if [ -z "${ATOM_LIST}" ]; then
         die "${WORKON_CMD}: No packages specified"
       elif ! ATOM_LIST=$(canonicalize_names "${ATOM_LIST}"); then
         die "Error parsing package list"
       fi;;
     *) ;;
   esac
 fi
 
 case ${WORKON_CMD} in
-  start)  ebuild_to_live "${ATOM_LIST}"; regen_manifest_and_sync ;;
+  start)  ebuild_to_live "${ATOM_LIST}" ;;
   stop)   ebuild_to_stable "${ATOM_LIST}" ;;
   list)   [ ${FLAGS_all} = "${FLAGS_FALSE}" ] && show_live_ebuilds || show_workon_ebuilds ;;
   iterate)ebuild_iterate "${ATOM_LIST}" ;;
   *)      die "$(basename $0): command '${WORKON_CMD}' not recognized" ;;
 esac