[PATCH modular v2 15/15] release.sh: Update jhbuild moduleset with --moduleset

[Dan Nicholson]

On Sat, Mar 12, 2011 at 1:46 AM, Dirk Wallenstein <halsmit at 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.lists at 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