Warning: file_exists(): open_basedir restriction in effect. File(/srv/http/vhosts/aur.archlinux.org/public/web/locale//en/LC_MESSAGES/aurweb.mo) is not within the allowed path(s): (/srv/http/vhosts/aur-dev.archlinux.org/:/etc/aurweb/) in /srv/http/vhosts/aur-dev.archlinux.org/public/web/lib/streams.php on line 90
AUR (en) - obconf-qt

Notice: Undefined variable: name in /srv/http/vhosts/aur-dev.archlinux.org/public/web/lib/pkgfuncs.inc.php on line 248

Package Details: obconf-qt 0.11.1-1

Git Clone URL: https://aur-dev.archlinux.org/obconf-qt.git (read-only)
Package Base: obconf-qt
Description: Openbox configuration tool. Qt port of ObConf
Upstream URL: http://lxqt.org
Licenses: GPL2
Submitter: Faerbit
Maintainer: Faerbit
Last Packager: Faerbit
Votes: 33
Popularity: 3.701383
First Submitted: 2015-05-30 07:38
Last Updated: 2017-01-18 19:18

Latest Comments

Faerbit commented on 2017-01-21 16:09


I don't get what changes you are proposing, and why I should apply them.
Please be more specific.

pmattern commented on 2016-11-28 17:46

You could have stumbled upon the missing execution of the script simply by checking the effect it's supposed to have or looking at the pacman output which always includes corresponding messages when scripts are executed. Pacman hooks may not have seen the most aggressive announcements, e. g. they were lacking on www.archlinux.org/news. But having a regular look at the forums or the blog of Allan McRae actually made get aware of them. Btw. man (5) alpm-hooks is a rather good reading.
The option to download files named <component>-<version> via command line from GitHub was introduced in release 0.11.0 only (it was available on downloads.lxqt.org earlier). But assigning the archives arbitrary names is explained in the docs. Verification of source archives is mentioned e. g. in man 5 PKGBUILD.
As for not using provides and conflicts in packages providing stable releases I don't remember exactly, but actually I'm pretty sure this is covered by the official docs, too.

Btw. you didn't bump pkgver. Also, you may want to change url to 'https://github.com/lxde/obconf-qt' the next commit as this represents the more helpful URL.

Faerbit commented on 2016-11-28 16:04

I implemented your suggestions.
I'm glad for your feedback, but did I not educate myself well enough, or is this not exactly documented properly?

pmattern commented on 2016-11-28 15:47

The install script isn't executed as PKGBUILD does not state variable install. Anyway it would be better to use pacman hooks instead of that script. You may just want to drop the script and make the package depend on gtk-update-icon-cache.

Right now the source archive is saved as <ver>.tar.gz, which isn't exactly convenient, in particular when a single directory is used to store the sources of all AUR packages used on some system.
You could simply use source=("https://github.com/lxde/$pkgname/releases/download/$pkgver/$pkgname-$pkgver.tar.xz"{,.asc}). If you complemented this with validpgpkeys=(7C733BA5F585AAD669E4D23A42C9C8D3AF5EA5E3) the sources would be verified as well.

Packages providing stable releases shouldn't state variables provides and conflicts, even if they are in the AUR. It's the packages providing other versions like VCS checkouts which have to make sure they don't conflict with the release packages or anything else.

Faerbit commented on 2015-12-28 23:28

Sure thing. Just updated it.

goetzc commented on 2015-12-28 22:26

Hi Faerbit, can you change the PKGBUILD to use Qt5? As it is upstream default https://github.com/lxde/obconf-qt/commit/f046ce6