summaryrefslogtreecommitdiffstats
path: root/scripts/libmakepkg/lint_pkgbuild/epoch.sh.in
diff options
context:
space:
mode:
authorLuke Shumaker <lukeshu@parabola.nu>2018-08-09 19:41:12 +0200
committerAllan McRae <allan@archlinux.org>2019-01-10 08:00:03 +0100
commitf7efa6a93d5361af610827d41045d87c7a72f2b5 (patch)
tree1d3d0bb731e5af544fa732b342fb95adecaee67e /scripts/libmakepkg/lint_pkgbuild/epoch.sh.in
parent30e3e21e873b01b98ab91dc1252d4112d6a9fd2a (diff)
downloadpacman-f7efa6a93d5361af610827d41045d87c7a72f2b5.tar.gz
pacman-f7efa6a93d5361af610827d41045d87c7a72f2b5.tar.xz
makepkg: Better error messages for versions in (check, make, opt)depends/provides/conflicts
Given the depends depends=('foo>=1.2-1.par2') and the error message ==> ERROR: pkgver in depends is not allowed to contain colons, forward slashes, hyphens or whitespace. One would be lead to believe that the problem is that they gave a pkgrel in depends at all, not that the pkgrel contains letters. Each of the (check,make,opt)depends, conflicts, and provides linters use a glob to trim off properly formed epoch an rel from the full version string, and pass the remainder to check_pkgver(). This does a good job of accepting/rejecting full versions, but doesn't do a good job of generating good error messages when rejecting if it's because of the epoch or rel. 1. Factor out check_epoch() and check_pkgrel() from lint_epoch() and lint_pkgrel(), similarly to check_pkgver(). 2. Add a check_fullpkgver() that takes a full [epoch:]ver[-rel] string and splits it in to epoch/ver/rel, and calls the appropriate check_ function on each. 3. Use check_fullpkgver() in the {,check,make,opt}depends, conflicts, and provides linters. Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'scripts/libmakepkg/lint_pkgbuild/epoch.sh.in')
-rw-r--r--scripts/libmakepkg/lint_pkgbuild/epoch.sh.in10
1 files changed, 8 insertions, 2 deletions
diff --git a/scripts/libmakepkg/lint_pkgbuild/epoch.sh.in b/scripts/libmakepkg/lint_pkgbuild/epoch.sh.in
index 93bd05c1..c98f91b0 100644
--- a/scripts/libmakepkg/lint_pkgbuild/epoch.sh.in
+++ b/scripts/libmakepkg/lint_pkgbuild/epoch.sh.in
@@ -29,9 +29,15 @@ source "$LIBRARY/util/message.sh"
lint_pkgbuild_functions+=('lint_epoch')
-lint_epoch() {
+check_epoch() {
+ local epoch=$1 type=$2
+
if [[ $epoch != *([[:digit:]]) ]]; then
- error "$(gettext "%s must be an integer, not %s.")" "epoch" "$epoch"
+ error "$(gettext "%s must be an integer, not %s.")" "epoch${type:+ in $type}" "$epoch"
return 1
fi
}
+
+lint_epoch() {
+ check_epoch "$epoch"
+}