diff options
| author | Vladimir Kotal <Vladimir.Kotal@Sun.COM> | 2009-05-07 16:42:22 +0200 |
|---|---|---|
| committer | Vladimir Kotal <Vladimir.Kotal@Sun.COM> | 2009-05-07 16:42:22 +0200 |
| commit | 25cc4e45d2ae4f199cec8b3a58f01500f2db7815 (patch) | |
| tree | ed1c3bc04034e4071fec371b744c34a9b434666f /usr/src/tools/scripts/webrev.sh | |
| parent | 87d06e46cdea545c3c673120b7211158dcfd35cc (diff) | |
| download | illumos-joyent-25cc4e45d2ae4f199cec8b3a58f01500f2db7815.tar.gz | |
6820408 webrev doesn't work with /usr/gnu/bin/sed
6835735 webrev -t can no longer be used for remote target specification after 6820421
6837938 cdm.py needs to be updated with options from 6820421
Diffstat (limited to 'usr/src/tools/scripts/webrev.sh')
| -rw-r--r-- | usr/src/tools/scripts/webrev.sh | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/usr/src/tools/scripts/webrev.sh b/usr/src/tools/scripts/webrev.sh index dfdeca4cac..d20b9607ac 100644 --- a/usr/src/tools/scripts/webrev.sh +++ b/usr/src/tools/scripts/webrev.sh @@ -489,13 +489,16 @@ function upload_webrev # so encode it first to avoid recursive transformation, and skip '/' which is # a path delimiter. # +# The quotation character is deliberately not escaped in order to make +# the substitution work with GNU sed. +# function url_encode { $SED -e "s|%|%25|g" -e "s|:|%3A|g" -e "s|\&|%26|g" \ -e "s|?|%3F|g" -e "s|#|%23|g" -e "s|\[|%5B|g" \ -e "s|*|%2A|g" -e "s|@|%40|g" -e "s|\!|%21|g" \ -e "s|=|%3D|g" -e "s|;|%3B|g" -e "s|\]|%5D|g" \ - -e "s|(|%28|g" -e "s|)|%29|g" -e "s|\'|%27|g" \ + -e "s|(|%28|g" -e "s|)|%29|g" -e "s|'|%27|g" \ -e "s|+|%2B|g" -e "s|\,|%2C|g" -e "s|\\\$|%24|g" } @@ -2191,7 +2194,7 @@ remote_target= # NOTE: when adding/removing options it is necessary to sync the list # with usr/src/tools/onbld/hgext/cdm.py # -while getopts "C:tDi:I:lnNo:Op::Uw" opt +while getopts "C:Di:I:lnNo:Op:t:Uw" opt do case $opt in C) Cflag=1 |