Arch Linux User Repository

PSA: Like most of the PKGBUILDs that I (co-)maintain, I host prebuilt packages for this in my user repository and all its dependencies for those who wish to install it using pacman without messing around with building from the AUR. Issue reports or contributions are welcome either in comments below or via this GitHub repository.

Upstream has released 0.119.0. I ran into some issues building it though: even after setting CXX=clang++ it still has issues with the C++ build toolchain. If anybody figures out how to build it I'm open to submissions here.

@dansecob To get email notifications on your packages you have to hit the "enable notifications" link. Response to packaging issues by about the 2 week mark is pretty much expected of AUR packages, especially when an orphan request is submitted. An orphan request can be 2 weeks from an initial comment, and then takes at least 2 weeks to clear. Basically that means if you're unresponsive to a broken package for a month then it can get orphaned for the next person to try. That's what happened here, although the timeline was much extended from the minimum.

I have no problem having you on as a co-maintainer if you like, just let me know if you want to be back on.

I’m sorry for not noticing the patch here: the AUR evidently doesn’t send emails to a linked address (by default, at least) and I have not been checking this page. Seeing as you fixed up the package and are updating it (I initially did a chroot build and subsequently only did regular system builds for all subsequent updates, not recognizing the new Python etc. deps I needed to add) I went ahead and dropped ownership so you are the sole maintainer. Feel free to delete my maintainer line on the next update.

PSA: Like most of the PKGBUILDs that I (co-)maintain, I host prebuilt packages for this in my user repository and all its dependencies for those who wish to install it using pacman without messing around with building from the AUR. Issue reports or contributions are welcome either in comments below or via this GitHub repository.

Not only did you bump this a couple times since my last comment without fixing the broken dependency situation, but it has gotten worse because upstream introduced a new one you're missing.

I rebased and update my patch to cover the new dependencies, again copy (with the trailing blank line) into a file and apply with git am < file.patch:

From 9fb9f922f53993630d82a543523be7156b95e674 Mon Sep 17 00:00:00 2001
From: Caleb Maclennan <caleb@alerque.com>
Date: Wed, 27 Sep 2023 15:39:55 +0300
Subject: [PATCH] Fixup dependencies so build in clean chroot works

* Build requires clang
* Build requires luau
* Correct LTO flags
* New dependency on xlswriter
* Version restriction not necessary
  (Arch's only supported python is new enough)

Signed-off-by: Caleb Maclennan <caleb@alerque.com>
---
 .SRCINFO | 11 ++++++-----
 PKGBUILD | 23 ++++++++++++-----------
 2 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/.SRCINFO b/.SRCINFO
index 1d7cc56..41b44e8 100644
--- a/.SRCINFO
+++ b/.SRCINFO
@@ -1,16 +1,17 @@
 pkgbase = qsv
    pkgdesc = A command line program for CSV files. Fork of xsv.
    pkgver = 0.117.0
-   pkgrel = 1
+   pkgrel = 2
    url = https://github.com/jqnatividad/qsv
    arch = any
    license = MIT
    license = Unlicense
    makedepends = cargo
-   depends = python>=3.8
-   optdepends = bash-completion: tab completion for bash
+   makedepends = clang
+   makedepends = luau
+   depends = python
+   depends = python-xlsxwriter
    source = qsv-0.117.0.tar.gz::https://github.com/jqnatividad/qsv/archive/refs/tags/0.117.0.tar.gz
-   md5sums = fb6bb78cd5e05757b17311411ad503c5
-   sha512sums = 89107394baa468bf12bfcce874047773d26cd2c258c9808164a588b231cc07ca0f033073f45183f07292a43da78bbd349df77979b951bbb1aef1e55e0b6fdf0f
+   sha256sums = 5f74e4380a1fea1b2fb8ae41c852f3fcde3021780e2352351ab20869913ac578

 pkgname = qsv
diff --git a/PKGBUILD b/PKGBUILD
index 4dae8f4..5527426 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -1,19 +1,19 @@
 # Maintainer: Bennett Petzold <dansecob.aur gmail com>
+# Contributor: Caleb Maclennan <caleb@alerque.com>

-pkgname='qsv'
-pkgver='0.117.0'
-pkgrel='1'
+BUILDENV+=(!check)
+
+pkgname=qsv
+pkgver=0.117.0
+pkgrel=2
 pkgdesc='A command line program for CSV files. Fork of xsv.'
-arch=('any')
+arch=(any)
 url='https://github.com/jqnatividad/qsv'
-license=('MIT' 'Unlicense')
-depends=('python>=3.8')
-makedepends=('cargo')
-optdepends=('bash-completion: tab completion for bash')
+license=(MIT Unlicense)
+depends=(python python-xlsxwriter)
+makedepends=(cargo clang luau)
 source=("$pkgname-$pkgver.tar.gz::$url/archive/refs/tags/$pkgver.tar.gz")
-
-md5sums=('fb6bb78cd5e05757b17311411ad503c5')
-sha512sums=('89107394baa468bf12bfcce874047773d26cd2c258c9808164a588b231cc07ca0f033073f45183f07292a43da78bbd349df77979b951bbb1aef1e55e0b6fdf0f')
+sha256sums=('5f74e4380a1fea1b2fb8ae41c852f3fcde3021780e2352351ab20869913ac578')

 _features='feature_capable,apply,fetch,foreach,generate,luau,polars,python,to,geocode'

@@ -26,6 +26,7 @@ build() {
     cd $pkgname-$pkgver
     export RUSTUP_TOOLCHAIN=stable
     export CARGO_TARGET_DIR=target
+    CFLAGS+=" -ffat-lto-objects"
     cargo build --frozen --release --features "$_features"
 }

-- 
2.42.0

Thanks for packaging this. At first blush this looked pretty good, but then I tried to build it using Arch official tooling in a chroot and it wouldn't build. I worked on it a while and found some missing build time dependencies and options that needed fixing relative to Arch default build time flags in order for this to build cleanly in a chroot (as recommended for all packages!).

Here is a patch you can copy to a file and apply using git am < file.patch that fixes the build issues. Note that it currently skips tests because they don't work yet. If I get a change to look into why the tests are failing I will get back to you on that.

From 6481fb32b29154b58b935740c7c745e1d79de1ea Mon Sep 17 00:00:00 2001
From: Caleb Maclennan <caleb@alerque.com>
Date: Wed, 27 Sep 2023 15:39:55 +0300
Subject: [PATCH] Fix dependencies and remove unnecessary quoting

Signed-off-by: Caleb Maclennan <caleb@alerque.com>
---
 .SRCINFO | 10 +++++-----
 PKGBUILD | 23 ++++++++++++-----------
 2 files changed, 17 insertions(+), 16 deletions(-)

diff --git a/.SRCINFO b/.SRCINFO
index 0493101..d575c09 100644
--- a/.SRCINFO
+++ b/.SRCINFO
@@ -1,16 +1,16 @@
 pkgbase = qsv
    pkgdesc = A command line program for CSV files. Fork of xsv.
    pkgver = 0.114.0
-   pkgrel = 1
+   pkgrel = 2
    url = https://github.com/jqnatividad/qsv
    arch = any
    license = MIT
    license = Unlicense
    makedepends = cargo
-   depends = python>=3.8
-   optdepends = bash-completion: tab completion for bash
+   makedepends = clang
+   makedepends = luau
+   depends = python
    source = qsv-0.114.0.tar.gz::https://github.com/jqnatividad/qsv/archive/refs/tags/0.114.0.tar.gz
-   md5sums = 3a847da3cef349b141ecfd3ffd61acf0
-   sha512sums = ebb4a25bff302f4321ce885972ebe00375f05acb959b16177aef263cfffde956b500361e5547a6a9ac936700a59a5e358a639772b4c812d9e85c2044fd1b7631
+   sha256sums = 12fa3b54ff805ef9b6e72b29d8355b5b4085a8cda4042a1be790ddd1dca7945b

 pkgname = qsv
diff --git a/PKGBUILD b/PKGBUILD
index 8bee187..7ddf485 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -1,19 +1,19 @@
 # Maintainer: Bennett Petzold <dansecob.aur gmail com>
+# Contributor: Caleb Maclennan <caleb@alerque.com>

-pkgname='qsv'
-pkgver='0.114.0'
-pkgrel='1'
+BUILDENV+=(!check)
+
+pkgname=qsv
+pkgver=0.114.0
+pkgrel=2
 pkgdesc='A command line program for CSV files. Fork of xsv.'
-arch=('any')
+arch=(any)
 url='https://github.com/jqnatividad/qsv'
-license=('MIT' 'Unlicense')
-depends=('python>=3.8')
-makedepends=('cargo')
-optdepends=('bash-completion: tab completion for bash')
+license=(MIT Unlicense)
+depends=(python)
+makedepends=(cargo clang luau)
 source=("$pkgname-$pkgver.tar.gz::$url/archive/refs/tags/$pkgver.tar.gz")
-
-md5sums=('3a847da3cef349b141ecfd3ffd61acf0')
-sha512sums=('ebb4a25bff302f4321ce885972ebe00375f05acb959b16177aef263cfffde956b500361e5547a6a9ac936700a59a5e358a639772b4c812d9e85c2044fd1b7631')
+sha256sums=('12fa3b54ff805ef9b6e72b29d8355b5b4085a8cda4042a1be790ddd1dca7945b')

 _features='feature_capable,apply,fetch,foreach,generate,luau,polars,python,to,geocode'

@@ -26,6 +26,7 @@ build() {
     cd $pkgname-$pkgver
     export RUSTUP_TOOLCHAIN=stable
     export CARGO_TARGET_DIR=target
+    CFLAGS+=" -ffat-lto-objects"
     cargo build --frozen --release --features "$_features"
 }

-- 
2.42.0