Arch Linux User Repository

@severach I'm using yay to handle the dependencies, I can reproduce the issue in a fresh image of Arch in docker.

I'm building fine, in chroot and out of chroot. python-geojson is not installed and no complaints about it missing. Do you have all your AUR python dependencies updated? pacman -Qo /usr/lib/python3.8

I'm getting the following error:

======================================================================
ERROR: test_to_sql_create_statement_with_dialects (tests.test_agatesql.TestSQL)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/husam/.cache/yay/python-agate-sql/src/agate-sql-0.5.5/tests/test_agatesql.py", line 153, in test_to_sql_create_statement_with_dialects
    self.table.to_sql_create_statement('test_table', dialect=dialect)
  File "/home/husam/.cache/yay/python-agate-sql/src/agate-sql-0.5.5/agatesql/table.py", line 323, in to_sql_create_statement
    sql_dialect = dialects.registry.load(dialect)()
  File "/usr/lib/python3.9/site-packages/sqlalchemy/util/langhelpers.py", line 266, in load
    return impl.load()
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 2447, in load
    return self.resolve()
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 2453, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/lib/python3.9/site-packages/crate/client/sqlalchemy/__init__.py", line 22, in <module>
    from .dialect import CrateDialect
  File "/usr/lib/python3.9/site-packages/crate/client/sqlalchemy/dialect.py", line 29, in <module>
    from .compiler import (
  File "/usr/lib/python3.9/site-packages/crate/client/sqlalchemy/compiler.py", line 28, in <module>
    from .types import MutableDict
  File "/usr/lib/python3.9/site-packages/crate/client/sqlalchemy/types.py", line 27, in <module>
    import geojson
ModuleNotFoundError: No module named 'geojson'

----------------------------------------------------------------------
Ran 18 tests in 0.329s

FAILED (errors=1)
Test failed: <unittest.runner.TextTestResult run=18 errors=1 failures=0>

Hey @severach I'm one of the co-maintainers on csvkit and just gave this PKGBUILD a bit of an overhaul. This -git version has some obsolete dependencies and is missing some current ones. Also the actual build process could use some improvement. I've synced it up with the stable one. Here is a patch you can apply with git am (or add me as a co-maintainer and I'll be happy to post it and help keep this updated as well):

From 2d6eb3635f4fe5efb10c47a611617cc0b845d245 Mon Sep 17 00:00:00 2001
From: Caleb Maclennan <caleb@alerque.com>
Date: Wed, 4 Mar 2020 08:39:25 +0300
Subject: [PATCH] Update dependencies per upstream, sync with stable packaging

Signed-off-by: Caleb Maclennan <caleb@alerque.com>
---
 .SRCINFO | 29 ++++++++++---------
 PKGBUILD | 86 +++++++++++++++++++++++++++-----------------------------
 2 files changed, 56 insertions(+), 59 deletions(-)

diff --git a/.SRCINFO b/.SRCINFO
index fe8fdc7..a3560a4 100644
--- a/.SRCINFO
+++ b/.SRCINFO
@@ -1,27 +1,28 @@
-# Generated by mksrcinfo v8
-# Thu Sep 27 18:43:36 UTC 2018
 pkgbase = csvkit-git
-   pkgdesc = A suite of utilities for converting to and working with CSV.
-   pkgver = 1.0.3.r35.g6d280f5
+   pkgdesc = A suite of utilities for converting to and working with CSV
+   pkgver = 1.0.4.r29.g9b76055
    pkgrel = 1
    url = http://csvkit.readthedocs.org
    arch = any
    license = MIT
-   makedepends = git
    makedepends = python-setuptools
-   makedepends = python-pip
+   makedepends = python-sphinx>=1.2.2
    depends = python
-   depends = python-agate
-   depends = python-agate-dbf
-   depends = python-agate-excel
-   depends = python-agate-sql
+   depends = python-agate-dbf>=0.2.0
+   depends = python-agate-excel>=0.2.2
+   depends = python-agate-sql>=0.5.3
+   depends = python-agate>=1.6.1
+   depends = python-babel
    depends = python-dateutil
-   depends = python-six
+   depends = python-openpyxl
+   depends = python-six>=1.6.1
+   depends = python-sphinx_rtd_theme>=0.1.6
    depends = python-sqlalchemy
-   provides = csvkit=1.0.3
+   depends = python-xlrd
+   optdepends = ipython: nicer command-line for csvpy utility
+   provides = csvkit
    conflicts = csvkit
-   source = csvkit::git://github.com/wireservice/csvkit.git
-   md5sums = SKIP
+   source = csvkit-git::git://github.com/wireservice/csvkit.git
    sha256sums = SKIP

 pkgname = csvkit-git
diff --git a/PKGBUILD b/PKGBUILD
index 1a4c480..8ce1623 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -3,65 +3,61 @@
 # Contributor: Mariusz Szczepańczyk <mszczepanczyk@gmail.com>
 # Contributor: Thor K. H. <thor alfakrøll roht dott no>

-set -u
-_pkgname='csvkit'
-pkgname="${_pkgname}-git"
-pkgver=1.0.3.r35.g6d280f5
+pkgname=csvkit-git
+pkgver=1.0.4.r29.g9b76055
 pkgrel=1
-pkgdesc="A suite of utilities for converting to and working with CSV."
+pkgdesc='A suite of utilities for converting to and working with CSV'
 arch=('any')
 url='http://csvkit.readthedocs.org'
 license=('MIT')
 depends=(
-  'python'
-  'python-agate' # >= 1.2.2
-  'python-agate-dbf' # >= 0.1.0
-  'python-agate-excel' # >= 0.1.0
-  'python-agate-sql'
-  'python-dateutil' # >=2.2
-  'python-six' # >=1.6.1
-  'python-sqlalchemy' # >=0.9.3
-)
+    'python'
+    'python-agate-dbf>=0.2.0'
+    'python-agate-excel>=0.2.2'
+    'python-agate-sql>=0.5.3'
+    'python-agate>=1.6.1'
+    'python-babel'
+    'python-dateutil'
+    'python-openpyxl'
+    'python-six>=1.6.1'
+    'python-sphinx_rtd_theme>=0.1.6'
+    'python-sqlalchemy'
+    'python-xlrd'
+  )
+optdepends=(
+    'ipython: nicer command-line for csvpy utility'
+  )
 makedepends=(
-  'git'
-  'python-setuptools'
-  'python-pip'
-)
-provides=("${_pkgname}=${pkgver%%.r*}")
-conflicts=("${_pkgname}")
-_srcdir='csvkit'
-source=('csvkit::git://github.com/wireservice/csvkit.git')
-md5sums=('SKIP')
+    'python-setuptools'
+    'python-sphinx>=1.2.2'
+  )
+provides=("${pkgname%-git}")
+conflicts=("${pkgname%-git}")
+source=("$pkgname::git://github.com/wireservice/${pkgname/-/.}")
 sha256sums=('SKIP')

 pkgver() {
-  set -u
-  cd "${_srcdir}"
-  git describe --long | sed -r 's/([^-]*-g)/r\1/;s/-/./g'
-  set +u
+    cd "$pkgname"
+    git describe --long | sed -r 's/([^-]*-g)/r\1/;s/-/./g'
 }

-prepare() {
-  set -u
-  cd "${_srcdir}"
-  # Quick and dirty fix until the author officially supports dateutil>=2.2
-  # (see https://github.com/onyxfish/csvkit/issues/370)
-  #sed -i 's/python-dateutil==2.2/python-dateutil>=2.2/' setup.py
-  #sed -i 's/openpyxl==2.2.0-b1/openpyxl>=2.2.0-b1/' setup.py
-  set +u
+build() {
+    cd "$pkgname"
+    python setup.py build
+    python setup.py build_sphinx
+    _rtd_theme_path="$(python -c 'import sphinx_rtd_theme; print(sphinx_rtd_theme.get_html_theme_path())')"
+    rm -rvf "build/sphinx/html/_static"
+    ln -svf "$_rtd_theme_path/sphinx_rtd_theme/static" "build/sphinx/html/_static"
 }

-build() {
-  set -u
-  cd "${_srcdir}"
-  python setup.py build
-  set +u
+check() {
+    cd "$pkgname"
+    python setup.py test
 }

 package() {
-  set -u
-  cd "${_srcdir}"
-  python setup.py install --root="${pkgdir}" --optimize=1
-  set +u
+    cd "$pkgname"
+    python setup.py install --root="$pkgdir" --optimize=1 --skip-build
+    mkdir -p "$pkgdir/usr/share/doc"
+    cp -rv "build/sphinx/html" "$pkgdir/usr/share/doc/$pkgname"
 }
-set +u
-- 
2.25.1

Also you might want to remove the pin from @kamazee's no longer relevant comment as that repo isn't the place for PRs with him not being a maintainer any more ;-)

I don't actively use csvkit, so if anyone is willing to take it over, please feel free to do so.

Had to change the sources again to allow for proxy, see below. source=( 'csvkit::git+https://github.com/wireservice/csvkit.git' 'agate::git+https://github.com/wireservice/agate.git' 'agate-excel::git+https://github.com/wireservice/agate-excel.git' 'agate-dbf::git+https://github.com/wireservice/agate-dbf.git' )

Dear all, Feel free to submit a pull request at https://github.com/kamazee/aur-csvkit-git instead of posting patches here as it seems way more convenient (and doesn't swallow leading spaces). You can also make a patch using `git format-patch` and email it to me (address is in my profile).

@ekkelett, I've pulled your changes in, thanks.

I've made a minor update to the PKGBUILD as there's a new organization hosting the repository, and the development has been relatively stable over new year's. diff --git a/.SRCINFO b/.SRCINFO index e19fc0f..cd1b3f6 100644 --- a/.SRCINFO +++ b/.SRCINFO @@ -1,20 +1,22 @@ +# Generated by mksrcinfo v8 +# Mon Mar 7 19:29:28 UTC 2016 pkgbase = csvkit-git pkgdesc = A suite of utilities for converting to and working with CSV. - pkgver = 0.9.1.gbf18815 - pkgrel = 2 + pkgver = 1.0.0.g48f9602 + pkgrel = 1 url = http://csvkit.readthedocs.org arch = any license = MIT - makedepends = tar - makedepends = gzip + makedepends = git + makedepends = python-setuptools depends = python depends = python-xlrd>=0.9.2 depends = python-dateutil>=2.0 depends = python-sqlalchemy>=0.9.3 depends = python-openpyxl>=2.0.3 depends = python-six>=1.6.1 - source = csvkit-0.9.1.tar.gz::https://github.com/onyxfish/csvkit/archive/0.9.1.tar.gz - md5sums = 5708394caa93408254a15269a83cfeed + source = csvkit::git://github.com/wireservice/csvkit.git + md5sums = SKIP pkgname = csvkit-git diff --git a/.gitignore b/.gitignore index effcdbf..f0d06db 100644 --- a/.gitignore +++ b/.gitignore @@ -1,3 +1 @@ -/csvkit-*.tar.[gx]z -/src -/pkg +/csvkit-git-*.tar.gz diff --git a/PKGBUILD b/PKGBUILD index 5eea2d7..9672e08 100644 --- a/PKGBUILD +++ b/PKGBUILD @@ -1,22 +1,27 @@ # Maintainer: Alexander Kurilo <alex@kurilo.me> # Contributor: Mariusz Szczepańczyk <mszczepanczyk@gmail.com> -gittag=0.9.1 -gitrev=gbf18815 +# Contributor: Thor K. H. <thor alfakrøll roht dott no> + pkgname=csvkit-git -pkgver=$gittag.$gitrev -pkgrel=2 +pkgver=1.0.0.g48f9602 +pkgrel=1 pkgdesc="A suite of utilities for converting to and working with CSV." arch=('any') url="http://csvkit.readthedocs.org" license=('MIT') makedepends=('git') depends=('python' 'python-xlrd>=0.9.2' 'python-dateutil>=2.0' 'python-sqlalchemy>=0.9.3' 'python-openpyxl>=2.0.3' 'python-six>=1.6.1') -makedepends=('tar' 'gzip') -md5sums=('5708394caa93408254a15269a83cfeed') -source=("csvkit-$gittag.tar.gz::https://github.com/onyxfish/csvkit/archive/$gittag.tar.gz") +makedepends=('git' 'python-setuptools') +md5sums=('SKIP') +source=('csvkit::git://github.com/wireservice/csvkit.git') + +pkgver() { + cd "$srcdir/csvkit" + git describe --long | sed -r 's/([^-]*-g)/r\1/;s/-/./g' +} build() { - cd "$srcdir/csvkit-$gittag" + cd "$srcdir/csvkit" # Quick and dirty fix until the author officially supports dateutil>=2.2 # (see https://github.com/onyxfish/csvkit/issues/370) sed -i 's/python-dateutil==2.2/python-dateutil>=2.2/' setup.py @@ -24,7 +29,7 @@ build() { } package() { - cd "$srcdir/csvkit-$gittag" + cd "$srcdir/csvkit" python setup.py install --root="$pkgdir/" }

Please add 'python-setuptools' as a dependency.