Package Details: airdcpp-webclient 2.12.1-1

Git Clone URL: https://aur.archlinux.org/airdcpp-webclient.git (read-only, click to copy)
Package Base: airdcpp-webclient
Description: A peer-to-peer file sharing client with web user interface
Upstream URL: https://github.com/airdcpp-web/airdcpp-webclient
Keywords: dc++
Licenses: GPL2
Submitter: droserasprout
Maintainer: selmf
Last Packager: selmf
Votes: 9
Popularity: 0.050224
First Submitted: 2016-08-25 09:45 (UTC)
Last Updated: 2023-05-26 06:09 (UTC)

Latest Comments

« First ‹ Previous 1 2

philipflohr commented on 2016-11-09 10:30 (UTC)

Please add npm as makedep

droserasprout commented on 2016-09-26 20:39 (UTC)

Done. Man I just need to sleep more.

whoami commented on 2016-09-26 20:36 (UTC)

"Include the appropriate VCS tool in makedepends=() (cvs, subversion, git, ...)." https://wiki.archlinux.org/index.php/VCS_package_guidelines#Guidelines

droserasprout commented on 2016-09-26 20:33 (UTC)

You're totally right, fixed.

whoami commented on 2016-09-26 20:25 (UTC) (edited on 2016-09-26 20:28 (UTC) by whoami)

agree, but you should specify 'tag' in this case. Otherwise you're fetching and building last commit. source=("${pkgname}::git+${url}#tag=${pkgver}") now you do _unless the package fetches a specific release_ https://github.com/airdcpp-web/airdcpp-webclient/tags man PKGBUILD plz /#fragment

droserasprout commented on 2016-09-26 19:57 (UTC) (edited on 2016-09-26 20:02 (UTC) by droserasprout)

Thanks for the time you've spend, atommixz! Your proposals are great but I disagree with renaming one. VCS package guidelines say: "Suffix pkgname with -cvs, -svn, -hg, -darcs, -bzr, -git etc. _unless the package fetches a specific release_."

whoami commented on 2016-09-26 00:58 (UTC) (edited on 2016-09-26 01:55 (UTC) by whoami)

- pkg should be renamed into airdcpp-webclient-git - also should be cmake -DCMAKE_INSTALL_PREFIX=/usr . - 'make' should be excluded from makedepends cause it's in base-devel group https://wiki.archlinux.org/index.php/PKGBUILD#makedepends - 'glibc' should be excluded from depends. It's installed everywhere. It's in group base - 'bzip2','zlib' also should be excluded cause 'namcap' says that. this is correct version this pkg http://ix.io/1qqN

robertfoster commented on 2016-08-25 10:59 (UTC)

please add cmake as makedep