From 4908658e9b07ba1beb1684e260e3523207ee1b16 Mon Sep 17 00:00:00 2001
From: "James D. Forrester" <jforrester@wikimedia.org>
Date: Fri, 21 Nov 2014 18:17:31 -0800
Subject: [PATCH] Maintainance: Update OOjs UI pull-through script to use
 release versions

Fetch from npm repository instead of from local git clone.
Matching the update-oojs.sh script.

Change-Id: I9e441064be2e28daaf55435e783a0318358bb565
---
 maintenance/resources/update-oojs-ui.sh | 99 ++++++++-----------------
 maintenance/resources/update-oojs.sh    |  2 +
 2 files changed, 32 insertions(+), 69 deletions(-)

diff --git a/maintenance/resources/update-oojs-ui.sh b/maintenance/resources/update-oojs-ui.sh
index 651f211e5b..bfa2eb021b 100755
--- a/maintenance/resources/update-oojs-ui.sh
+++ b/maintenance/resources/update-oojs-ui.sh
@@ -1,95 +1,56 @@
 #!/usr/bin/env bash
 
-# This script generates a commit that updates our distribution copy of OOjs UI
+# This script generates a commit that updates our copy of OOjs UI
 
-if [ -z "$1" ]
+if [ -n "$2" ]
 then
-	# Missing required parameter
-	echo >&2 "Usage: $0 path/to/repo/for/oojs-ui"
+	# Too many parameters
+	echo >&2 "Usage: $0 [<version>]"
 	exit 1
 fi
 
-TARGET_REPO=$(cd "$(dirname $0)/../.."; pwd)
-TARGET_DIR=resources/lib/oojs-ui
-UI_REPO=$1
-
-function oojsuihash() {
-	grep "OOjs UI v" "$TARGET_REPO/$TARGET_DIR/oojs-ui.js" \
-		| head -n 1 \
-		| grep -Eo '\([a-z0-9]+\)' \
-		| sed 's/^(//' \
-		| sed 's/)$//'
-}
-
-function oojsuitag() {
-	grep "OOjs UI v" "$TARGET_REPO/$TARGET_DIR/oojs-ui.js" \
-		| head -n 1 \
-		| grep -Eo '\bv[0-9a-z.-]+\b'
-}
-
-function oojsuiversion() {
-	grep "OOjs UI v" "$TARGET_REPO/$TARGET_DIR/oojs-ui.js" \
-		| head -n 1 \
-		| grep -Eo '\bv[0-9a-z.-]+\b.*$'
-}
+REPO_DIR=$(cd "$(dirname $0)/../.."; pwd) # Root dir of the git repo working tree
+TARGET_DIR="resources/lib/oojs-ui" # Destination relative to the root of the repo
+NPM_DIR=$(mktemp -d 2>/dev/null || mktemp -d -t 'update-oojs-ui') # e.g. /tmp/update-oojs-ui.rI0I5Vir
 
 # Prepare working tree
-cd "$TARGET_REPO" &&
+cd "$REPO_DIR" &&
 git reset $TARGET_DIR && git checkout $TARGET_DIR && git fetch origin &&
-git checkout -B upstream-oojsui origin/master || exit 1
-
-cd $UI_REPO || exit 1
+git checkout -B upstream-oojs-ui origin/master || exit 1
 
-# Read the old version and check for changes
-OLDHASH=$(oojsuihash)
-if [ -z "$OLDHASH" ]
+# Fetch upstream version
+cd $NPM_DIR
+if [ -n "$1" ]
 then
-	OLDTAG=$(oojsuitag)
+	npm install "oojs-ui@$1" || exit 1
+else
+	npm install oojs-ui || exit 1
 fi
-if [ "$OLDHASH" == "" ]
-then
-	OLDHASH=$(git rev-parse "$OLDTAG")
-	if [ $? != 0 ]
-	then
-		echo "Could not find OOjs UI version"
-		cd -
-		exit 1
-	fi
-fi
-if [ "$(git rev-parse $OLDHASH)" == "$(git rev-parse HEAD)" ]
+
+OOJSUI_VERSION=$(node -e 'console.log(JSON.parse(require("fs").readFileSync("./node_modules/oojs-ui/package.json")).version);')
+if [ "$OOJSUI_VERSION" == "" ]
 then
-	echo "No changes (already at $OLDHASH)"
-	cd -
-	exit 0
+	echo 'Could not find OOjs UI version'
+	exit 1
 fi
 
-# Build the distribution
-npm install && grunt git-build || exit 1
-
-# Get the list of changes
-NEWCHANGES=$(git log $OLDHASH.. --oneline --no-merges --reverse --color=never)
-NEWCHANGESDISPLAY=$(git log $OLDHASH.. --oneline --no-merges --reverse --color=always)
-
 # Copy files
 # - Exclude the minimised distribution files and RTL sheets for non-CSSJanus environments
-rsync --recursive --delete --force --exclude 'oojs-ui*.min.*' --exclude 'oojs-ui*.rtl.css' ./dist/ "$TARGET_REPO/$TARGET_DIR" || exit 1
+rsync --force --recursive --delete --exclude 'oojs-ui*.min.*' --exclude 'oojs-ui*.rtl.css' ./node_modules/oojs-ui/dist/ "$REPO_DIR/$TARGET_DIR" || exit 1
 
-# Read the new version
-NEWVERSION=$(oojsuiversion)
+# Clean up temporary area
+rm -rf "$NPM_DIR"
 
 # Generate commit
-cd "$TARGET_REPO"
+cd $REPO_DIR || exit 1
+
 COMMITMSG=$(cat <<END
-Update OOjs UI to $NEWVERSION
+Update OOjs UI to v$OOJSUI_VERSION
 
-New changes:
-$NEWCHANGES
+Release notes:
+ https://git.wikimedia.org/blob/oojs%2Fui.git/v$OOJSUI_VERSION/History.md
 END
 )
-git add -u $TARGET_DIR && git add $TARGET_DIR && git commit -m "$COMMITMSG"
-cat >&2 <<END
-
 
-Created commit with changes:
-$NEWCHANGESDISPLAY
-END
+# Stage deletion, modification and creation of files. Then commit.
+git add --update $TARGET_DIR && git add $TARGET_DIR && git commit -m "$COMMITMSG" || exit 1
diff --git a/maintenance/resources/update-oojs.sh b/maintenance/resources/update-oojs.sh
index d9e6fb9d92..8627c7199c 100755
--- a/maintenance/resources/update-oojs.sh
+++ b/maintenance/resources/update-oojs.sh
@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+# This script generates a commit that updates our copy of OOjs
+
 if [ -n "$2" ]
 then
 	# Too many parameters
-- 
2.20.1