Package Details: twtxt 1.2.3-1

Git Clone URL: https://aur.archlinux.org/twtxt.git (read-only)
Package Base: twtxt
Description: Decentralised, minimalist microblogging service for hackers.
Upstream URL: https://github.com/buckket/twtxt
Keywords: mircoblogging python twtxt
Licenses: MIT
Submitter: buckket
Maintainer: buckket
Last Packager: buckket
Votes: 2
Popularity: 0.000000
First Submitted: 2016-03-04 20:33
Last Updated: 2017-09-28 16:14

Latest Comments

buckket commented on 2016-04-16 12:30

Hi,

chardet and six are indirect dependencies required by aiohttp and dateutil, they are already listed there.
Wiki says: "Dependencies that are provided by other dependencies do not need to be listed."
But removing python makes sense because of that.

There’s no harm done about being a little bit more explicit than implicit about the build process of the package.
So I guess the call to setup.py build is ok.

Will add the ${} though. ;)

Thanks for your input!

ogarcia commented on 2016-04-16 10:38

Hi,

Some considerations in your pkgbuild:

You have missing depends, the correct is:
depends=('python-six' 'python-dateutil' 'python-chardet' 'python-aiohttp'
'python-click' 'python-humanize')

Not necessary add python as depend cause is implied by "python-*" packages.

In source, use ${} in vars, is more "elegant".

source=("https://pypi.python.org/packages/source/t/${pkgname}/${pkgname}-${pkgver}.tar.gz")

You can remove all build section, not necesary cause "python setup.py install" do it.

And in package section, use ${} too:

package() {
cd ${pkgname}-${pkgver}
install -Dm644 LICENSE "${pkgdir}/usr/share/licenses/${pkgname}/LICENSE"
python setup.py install --root="${pkgdir}" --optimize='1'
}

Greetings ;)