diff options
| author | Xavier Chantry <shiningxc@gmail.com> | 2009-08-30 22:05:46 +0200 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2009-09-06 16:52:47 -0500 |
| commit | 0845b2f13cef6db23a4a281976ec1374e93b7c4e (patch) | |
| tree | 3f230e9d3771463ad7fa49647ffba918cdee4ac5 /scripts | |
| parent | 5dbd00faf7a27866ffc2a7197c5cc704425a7af8 (diff) | |
| download | pacman-0845b2f13cef6db23a4a281976ec1374e93b7c4e.tar.xz | |
sanity check for optdepends syntax
only allow optdepends like:
pkgname: description
some (real) examples of invalid optdepends:
'tcl, python and/or ruby: to use corresponding binding'
'xorg-fonts-75dpi : X bitmap fonts needed for the interface'
'ruby-htmlentities (AUR): for one provider named Deastore'
'xpdf - for pdf'
Signed-off-by: Xavier Chantry <shiningxc@gmail.com>
[Allan: rebase off de39a1f6 and adjust man page]
Signed-off-by: Allan McRae <allan@archlinux.org>
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/makepkg.sh.in | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 558f0938..3e918e69 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1180,6 +1180,14 @@ check_sanity() { fi done + local optdepend + for optdepend in "${optdepends[@]}"; do + pkg=${optdepend%%:*} + if [[ ! $pkg =~ ^[[:alnum:]\>\<\=\.\+\_\-]*$ ]]; then + error "$(gettext "Invalid syntax for optdepend : '%s'")" "$optdepend" + fi + done + if [ "$install" -a ! -f "$install" ]; then error "$(gettext "Install scriptlet (%s) does not exist.")" "$install" return 1 |