On Sat, Mar 12, 2011 at 1:46 AM, Dirk Wallenstein <hals...@t-online.de> wrote: > On Fri, Mar 11, 2011 at 06:31:30AM -0800, Dan Nicholson wrote: >> In order to keep the katamari moduleset up to date with current tarball >> information, have release.sh update the file when new releases are made. >> This is enabled by passing a module file with --moduleset. >> >> Signed-off-by: Dan Nicholson <dbn.li...@gmail.com> >> --- >> V2: Split the sha1sum output so the hash is separate from $targz >> >> release.sh | 22 ++++++++++++++++++++++ >> 1 files changed, 22 insertions(+), 0 deletions(-) >> >> diff --git a/release.sh b/release.sh >> index ea4d9c9..5228a30 100755 >> --- a/release.sh >> +++ b/release.sh >> @@ -12,6 +12,7 @@ host_xorg=xorg.freedesktop.org >> host_dri=dri.freedesktop.org >> user= >> remote=origin >> +moduleset= >> >> usage() >> { >> @@ -24,6 +25,7 @@ Options: >> --help this help message >> --ignore-local-changes don't abort on uncommitted local changes >> --remote git remote where the change should be pushed (default >> "origin") >> + --moduleset jhbuild moduleset to update with relase info >> HELP >> } >> >> @@ -107,6 +109,11 @@ while [ $# != 0 ]; do >> remote=$1 >> shift >> ;; >> + --moduleset) >> + shift >> + moduleset=$1 >> + shift >> + ;; >> --*) >> echo "error: unknown option" >> usage >> @@ -198,6 +205,14 @@ if [ -z "$tag_previous" ] || >> exit 1 >> fi >> >> +if [ -n "$moduleset" ]; then >> + echo "checking for moduleset" >> + if ! [ -w "$moduleset" ]; then >> + echo "moduleset $moduleset does not exist or is not writable" >> + exit 1 >> + fi >> +fi >> + >> if [ "$section" = "libdrm" ]; then >> section_path="libdrm" >> srv_path="/srv/$host_dri/www/$section_path" >> @@ -236,6 +251,13 @@ echo "generating announce mail template, remember to >> sign it" >> gen_announce_mail >$announce >> echo " at: $announce" >> >> +if [ -n "$moduleset" ]; then >> + echo "updating moduleset $moduleset" >> + modulardir=`dirname "$0"` > > When testing it, I thought that finding the modulardir could > optionally be improved by adding a `readlink` here.
Hmm, running readlink on non-links returns an error, and this script is running with set -e in places. Care to suggest a patch on top of this one? I'm about to push all mine. -- Dan _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel