diff options
author | Lukas Fleischer <archlinux@cryptocrack.de> | 2012-06-10 11:35:39 +0200 |
---|---|---|
committer | Pierre Schmitz <pierre@archlinux.de> | 2012-06-10 12:47:40 +0200 |
commit | fda394f1a054e58f146e942b5971944e562583a0 (patch) | |
tree | b28175467049c0e52dfd085a1196677f682f31ef | |
parent | addea828fef4f96d19752ff409445bff03e13610 (diff) | |
download | devtools-fda394f1a054e58f146e942b5971944e562583a0.tar.xz |
Gracefully handle files containing an "@"
The "@" sign in file names in SVN marks the beginning of a pegged
version number -- from the Subversion book:
Peg revisions are specified to the Subversion command-line client
using at syntax, so called because the syntax involves appending an
“at sign” (@) and the peg revision to the end of the path with which
the revision is associated.
The trivial workaround is to always append an at sign to the end of the
path in the version control checks.
Before:
$ community-stagingpkg 'Add systemd units.'
==> ERROR: exim-submission@.service is not under version control
$ svn status -v | grep 'exim-submission@.service'
A - ? ? exim-submission@.service
After:
$ community-stagingpkg 'Add systemd units.'
==> Committing changes to trunk...done
==> Signing package exim-4.80-2-x86_64.pkg.tar.xz...
[...]
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
Signed-off-by: Pierre Schmitz <pierre@archlinux.de>
-rw-r--r-- | archrelease.in | 3 | ||||
-rw-r--r-- | commitpkg.in | 2 |
2 files changed, 4 insertions, 1 deletions
diff --git a/archrelease.in b/archrelease.in index 5bc2c90..533a547 100644 --- a/archrelease.in +++ b/archrelease.in @@ -54,6 +54,9 @@ for file in "${known_files[@]}"; do fi done +# gracefully handle files containing an "@" character +known_files=("${known_files[@]/%/@}") + for tag in "$@"; do stat_busy "Copying ${trunk} to ${tag}" diff --git a/commitpkg.in b/commitpkg.in index b5cf043..8b87426 100644 --- a/commitpkg.in +++ b/commitpkg.in @@ -57,7 +57,7 @@ esac # check if all local source files are under version control for s in "${source[@]}"; do - if [[ $s != *://* ]] && ! svn status -v "$s" | grep -q '^[ AMRX~]'; then + if [[ $s != *://* ]] && ! svn status -v "$s@" | grep -q '^[ AMRX~]'; then die "$s is not under version control" fi done |