On 02/28/2016 10:02 PM, Dustin Falgout wrote:
I currently maintain six or seven PKGBUILDS in the AUR, of which three are notable: pycharm-eap[1], lightdm-webkit2-greeter[2], and lightdm-webkit-theme-antergos[3]. I've indirectly contributed to other packages in the AUR through comments by offering advice to users, notifying maintainers of new releases as well as any problems with PKGBUILDS (always including a proposed solution), and directing bug reports away from AUR comments and to their proper channel. I always take the time to properly request deletion of PKGBUILDS that shouldnt be in the AUR (for whatever reason) whenever I come across one. I've also reported (and still do report) bugs on http://bugs.archlinux.org when it was/is appropriate. On one or two occasions I've gone as far as reaching out directly to maintainers of official packages to provide them with important information.
Hi Dustin I went through your packages and had a rough look at the PKGBUILDS... don't be scared but i wanted to dump my feedback and some personal opinions about those: lightdm-webkit-theme-antergos: - just my 2 cents but post_install() should not sed around in your lightdm.conf as you can also install multiple theme packages lightdm-webkit-theme-userdock: - just my 2 cents but post_install() should not be used for explaining documentation/setup lightdm-webkit2-greeter: - you can use #tag=2.0.0 instead of doing a checkout in build() - reference to $pkgdir should always be quoted (make call) - should not sed the lightdm.conf in post_install() as you can possibly install multiple greeters obs-service-recompress: - this package actually pulls the git HEAD and should either be renamed to obs-service-recompress-git or pull the 0.3.1 tarball and have the pkgver() removed. - reference to $pkgdir should always be quoted (make call) obs-service-set_version: - same as obs-service-recompress, pulls git HEAD with pkgver() function instead of 0.5.3 tarball - reference to $pkgdir should always be quoted (make call) obs-service-tar_scm: - same as obs-service-recompress, pulls git HEAD with pkgver() function instead of 0.5.3 tarball - reference to $pkgdir should always be quoted (make call) pycharm-eap: - not sure what the _eap="True" variable and the if does there as its the -eap package, guess it can just be dropped (including the else) - just my 2 cents but using groups with "development" "IDE" "editor" and "jetbrains" feels a bit too much, groups should logically group together similarities like "vim-plugins" and not used like generic keywords cheers, Levente PS: instead of inline PGP you may want to have a look at PGP/MIME... besides weird text in clients that don't support PGP it can also result in breakage in various scenarios where the text gets altered by MTA/MUA