Arch Linux User Repository

Please add shellcheck checkdep for test/shellcheck.

This works to install 0.0.24, which is the current version. And it's using the http://deb.debian.org/ URLs.

Side note: Too specific to include in the patch, but make sure you don't have a strict ~/.shellcheckrc file, because the linter runs without a --norc flag.

--- a/PKGBUILD  2024-07-31 20:08:53.132895188 -0500
+++ b/PKGBUILD  2024-07-31 20:10:48.136955913 -0500
@@ -7,7 +7,7 @@
 pkgname=(sensible-utils sensible-browser sensible-editor sensible-pager sensible-terminal sensible-utils-data)
 # This is the above minus the pkgbase name, for use in sensible-utils packaging command.
 _pkgname=(sensible-browser sensible-editor sensible-pager sensible-terminal sensible-utils-data)
-pkgver=0.0.21
+pkgver=0.0.24
 _debianextra='' # Do not remove even if empty as sometimes Debian puts e.g. +1, their version of a pkgrel.
 pkgrel=1
 pkgdesc="Utilities for sensible alternative selection"
@@ -20,21 +20,21 @@
              'findutils')
 checkdepends=('ed')
 _dirname=${pkgbase}_${pkgver}_$pkgrel_${_debianextra}
-source=("$_dirname.tar.xz::http://ftp.debian.org/debian/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.tar.xz"
-        "$_dirname.dsc::http://ftp.debian.org/debian/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.dsc"
+source=("$_dirname.tar.xz::http://deb.debian.org/debian/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.tar.xz"
+        "$_dirname.dsc::http://deb.debian.org/debian/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.dsc"
         sensible-utils.install
         sensible-envvars)
 # (n)vim command to regenerate:
 # :'<,'>! makepkg --geninteg 2>&1 | perl -pe '$_=~s/^[\s]*(==>|->).*//g; $_=~s/^[\s]*$//d'
-b2sums=('3540965529583afe60f7abb9eade22c9f06f7d5dc4d266bfad35c0a33337db341fff2f3e59e25e1c53fb845c9f4e4d1587f0215000fdd3ba9f499ed35d9f7ef9'
-        'da806a63d4fe66f341564d8edaaa7669acf181c57595baf0be9eb33d1c2a0554aa0faa18e0449840d8b5941b592359e937cc0ba79a96a1fa276828a9b05c1181'
+b2sums=('96d8b9b763b81fd5345db1f0f1c795cc615b7dcd26f240202ba1b1f6d45377400e286f7d39733cbb8e1601cc6e45aa695745e60c7db359c69749f54ff3157a46'
+        '6a2483b4aceb07b59eb47e7cb6e5ac3061430bfdfcedf182c01d8bd80e34fa094cf89ac42eea7f6d009acab70b15a1c95e52117c7eae80ac03dff0c1337bf71a'
         'c39a760655833d374dc606894452c997bd86a1645c0b5496c5546364cac07ad4205c8dae05f457bd31a210bfd4d290fbcb71cdcf584df857e93c851b384bba81'
         '8a213b98cc6d432af30ecf58d91ae88f151f2824274702f7e1bde6119b45effc4a0b15907459c9f8f1ee5af94de862e6dec579b4f07d168aaa658443764a1f19')
 # Bastien ROUCARIÈS <rouca@debian.org>
 validpgpkeys=('5D0187B940A245BAD7B0F56A003A1A2DAA41085F')

 build() {
-  cd ${pkgbase}
+  cd ${pkgbase}-${pkgver}
   export -n EDITOR VISUAL PAGER TERMINAL_EMULATOR BROWSER
   ./configure --prefix=/usr
   find .. -type f -and -name '*.patch' -print0 | xargs -0r patch -p1
@@ -44,12 +44,12 @@
 # This trick is needed as the fork-bomb test only works if /bin/sh == /bin/dash,
 # which it isn't on Arch but is on Debian!
 check() {
-  cd ${pkgbase}
+  cd ${pkgbase}-${pkgver}
   env -i make check
 }

 _package_sensible() {
-  cd ${pkgbase}
+  cd ${pkgbase}-${pkgver}
   pkgdesc="$pkgdesc (${pkgname##*-})"
   export -n EDITOR VISUAL PAGER TERMINAL_EMULATOR BROWSER
   make DESTDIR="$pkgdir/" install

This version (0.0.21) is no longer available on Debian's servers. However, it is available in Debian Snapshot. It still doesn't package for other reasons, but....

diff --git a/PKGBUILD b/PKGBUILD
index 532b655..094f55e 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -20,8 +20,8 @@ makedepends=('po4a'
              'findutils')
 checkdepends=('ed')
 _dirname=${pkgbase}_${pkgver}_$pkgrel_${_debianextra}
-source=("$_dirname.tar.xz::http://ftp.debian.org/debian/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.tar.xz"
-        "$_dirname.dsc::http://ftp.debian.org/debian/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.dsc"
+source=("$_dirname.tar.xz::https://snapshot.debian.org/archive/debian/20230705T090048Z/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.tar.xz"
+        "$_dirname.dsc::https://snapshot.debian.org/archive/debian/20230705T090048Z/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.dsc"
         sensible-utils.install
         sensible-envvars)
 # (n)vim command to regenerate:

Debian Trixie (testing) has 0.0.24, FWIW.

Reminder, http://ftp.debian.org still does not work and needs to be replaced with working mirror, such as http://deb.debian.org.

Is anyone else getting this same error?

==> Retrieving sources...
  -> Downloading sensible-utils_0.0.20_.tar.xz...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (7) Failed to connect to ftp.debian.org port 80 after 58 ms: Couldn't connect to server
==> ERROR: Failure while downloading http://ftp.debian.org/debian/pool/main/s/sensible-utils/sensible-utils_0.0.20.tar.xz
    Aborting...

Seems we should switch to the proper CDN, https://deb.debian.org, that even their docker image uses. Below is a full patch updating CDN and bumping package release (works for me):

diff --git a/.SRCINFO b/.SRCINFO
index 3994ed2..9734088 100644
--- a/.SRCINFO
+++ b/.SRCINFO
@@ -1,7 +1,7 @@
pkgbase = sensible-utils
    pkgdesc = Utilities for sensible alternative selection
    pkgver = 0.0.20
-   pkgrel = 6
+   pkgrel = 7
    url = http://packages.debian.org/source/sid/sensible-utils
    arch = any
    license = GPL
@@ -10,8 +10,8 @@ pkgbase = sensible-utils
    makedepends = findutils
    depends = bash
    depends = coreutils
-   source = sensible-utils_0.0.20_.tar.xz::http://ftp.debian.org/debian/pool/main/s/sensible-utils/sensible-utils_0.0.20.tar.xz
-   source = sensible-utils_0.0.20_.dsc::http://ftp.debian.org/debian/pool/main/s/sensible-utils/sensible-utils_0.0.20.dsc
+   source = sensible-utils_0.0.20_.tar.xz::https://deb.debian.org/debian/pool/main/s/sensible-utils/sensible-utils_0.0.20.tar.xz
+   source = sensible-utils_0.0.20_.dsc::https://deb.debian.org/debian/pool/main/s/sensible-utils/sensible-utils_0.0.20.dsc
    source = 0001-no-fork-bomb-test.patch
    source = sensible-utils.install
    source = sensible-envvars
diff --git a/PKGBUILD b/PKGBUILD
index 11cb59a..8c1418c 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -9,7 +9,7 @@ pkgname=(sensible-utils sensible-browser sensible-editor sensible-pager sensible
_pkgname=(sensible-browser sensible-editor sensible-pager sensible-terminal sensible-utils-data)
pkgver=0.0.20
_debianextra='' # Do not remove even if empty as sometimes Debian puts e.g. +1, their version of a pkgrel.
-pkgrel=6
+pkgrel=7
pkgdesc="Utilities for sensible alternative selection"
arch=('any')
url="http://packages.debian.org/source/sid/sensible-utils"
@@ -20,8 +20,8 @@ makedepends=('po4a'
    'findutils')
checkdepends=('ed')
_dirname=${pkgbase}_${pkgver}_$pkgrel_${_debianextra}
-source=("$_dirname.tar.xz::http://ftp.debian.org/debian/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.tar.xz"
-        "$_dirname.dsc::http://ftp.debian.org/debian/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.dsc"
+source=("$_dirname.tar.xz::https://deb.debian.org/debian/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.tar.xz"
+        "$_dirname.dsc::https://deb.debian.org/debian/pool/main/s/$pkgbase/${pkgbase}_${pkgver}${_debianextra}.dsc"
    0001-no-fork-bomb-test.patch
    sensible-utils.install
    sensible-envvars)

Until worked out upstream I've removed fork-bomb test.

Well, it looks like the upstream maintainer rejected your merge request, so can this check please be disabled in the PKGBUILD? It's a real nuisance!

I asked the maintainer to consider making /bin/dash explicit: https://salsa.debian.org/debian/sensible-utils/-/merge_requests/7

Outside Debian, the fork-bomb test may fail if /bin/sh ≠ /bin/dash, because sensible-editor will run under another shell. On Arch, that shell is /bin/bash.

This makes /bin/dash explicit.

Also eagle-eyed users may have noticed the test results report a version of 0.0.15. This wasn't right, my merge request !8 was merged to fix that.

The fork bomb test is annoying and if users agree I'm happy to restore my old patch which did just that, removed it.