Arch Linux User Repository

paraxor-st: /usr/share/terminfo/s/st è già presente nel filesystem (appartiene a ncurses) paraxor-st: /usr/share/terminfo/s/st-256color è già presente nel filesystem (appartiene a ncurses)

Thanks @waschtl for your clear explanations! I must admit I don't read the doc since a long time, shame on me.

@milouse, thank you for your suggestions. I had completely missed the fact that the URLs were not using HTTPS.

I incorporated several of your changes, but not all of them. I don't want to seem like I'm ignoring your suggestions, though, so I would like to explain my decisions:

• Using variables in the source array is preferred over static definitions according to man pkgbuild. See "OPTIONS AND DIRECTIVES"->source.
• Using $srcdir is not necessary because all PKGBUILD functions are executed within that directory. See man pkgbuild->"PACKAGING FUNCTIONS"->srcdir.
• I have a personal preference not to use cd in favor of --directory options to make and co. Therefore, I have removed what cd statements were still around.

Hi,

I would like to suggest the following changes:

• Use https URL to download st
• Remove unnecessary variable resolution
• Fix sed line to remove terminfo files (no more @ prefix in the current release).

diff --git a/PKGBUILD b/PKGBUILD
index 8a0056e..31c5ac1 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -12,11 +12,14 @@ license=('MIT')
 depends=(libxft)
 makedepends=(libxext ncurses)
 url="http://st.suckless.org"
-source=(http://dl.suckless.org/st/$pkgname-$pkgver.tar.gz)
+source=(https://dl.suckless.org/st/st-$pkgver.tar.gz)
 sha256sums=('aeb74e10aa11ed364e1bcc635a81a523119093e63befd2f231f8b0705b15bf35')

 prepare() {
-  sed '/@tic/d' -i $pkgname-$pkgver/Makefile
+  cd $srcdir/$pkgname-$pkgver
+  # skip terminfo which conflicts with ncurses
+  sed '/tic /d' -i Makefile
+  sed -i '/@echo .* terminfo/d' Makefile

   # This package provides a mechanism to provide a custom config.h. Multiple
   # configuration states are determined by the presence of two files in
@@ -39,7 +42,7 @@ prepare() {
   abort=
   if [ -e "$BUILDDIR/config.h" ]
   then
-    cp "$BUILDDIR/config.h" "$pkgname-$pkgver"
+    cp "$BUILDDIR/config.h" "$srcdir/$pkgname-$pkgver"
   elif [ ! -e "$BUILDDIR/config.def.h" ]
   then
     abort=1
@@ -49,7 +52,7 @@ prepare() {
     msg+='values. Then restart the build process.'
     error "$msg"
   fi
-  cp "$pkgname-$pkgver/config.def.h" "$BUILDDIR"
+  cp config.def.h "$BUILDDIR"
   test -z "$abort"
 }

@simona: The purpose of this is to alert the user to the unusual situation that configuration happens at build time. Build is successful on the second run, thereby giving you the chance to look at the configuration file if you want to.

I will change "Then rerun makepkg." to "Then restart the build process." to account for builds by AUR helpers.

==> ERRORE: This package can be configured in config.h. Copy provided config.def.h to config.h and modify to change configuration. Leave config.def.h as it is to use default values. Then rerun makepkg. ==> ERRORE: Si è verificato un errore in prepare().

Thanks @famar for the pointer. I have replaced it with $BUILDDIR, which is not mentioned in that document, but seems appropriate.

Hi, according to PKGBUILD manual, the use of the packaging variable startdir "is deprecated and strongly discouraged".

Can you please update?

The line

# remove to avoid conflict with ncurses
rm "${pkgdir}/usr/share/terminfo/s/st" "${pkgdir}/usr/share/terminfo/s/st-256color"

doesn't quite do the trick. The two files

st-meta
st-meta-256color

still get installed. And they do not play well with the terminfo files from ncurses. It would be safer to disable terminfo generation in prepare() as follows (taken from st-git):

sed '/@tic/d' -i Makefile

adopted and finally updated