[aur-general] PKGBUILD for zetacoin-qt-git
scimmia22 at outlook.com
Sat Nov 2 13:15:19 EDT 2013
> From: brunojimen at gmail.com
> To: aur-general at archlinux.org
> Date: Sat, 2 Nov 2013 17:21:36 +0100
> Subject: Re: [aur-general] PKGBUILD for zetacoin-qt-git
> Updated version of the PKGBUILD: http://pastebin.com/LRyYqpuZ
>> Whoops, I misread the PKGBUILD. Ignore my suggestion. No, you do not need
>> to add zetacoin-qt-git.install to the `source` and `sha256sums` arrays.
> Ok, nothing added then.
>> The `pkgver` you added is OK, but it could be better. The zetacoin git
>> repository has tagged releases, and the wiki states: 
>>> It is recommended to have following version format: *RELEASE.rREVISION*[...]
>> Here's a better command for generating a package version number:
>>> git describe --long | sed -E 's/([^-]*-g)/r\1/;s/-/./g'
>> That command generates the following version number:
> Added the new package version number. Althought it's a bit strange to
> see a package whose version starts with 'v'.
You're right, the v shouldn't be there, so strip it. Just add s/^v// to the sed command.
The pkgver functions in the Wiki are just suggestions, they need customized for the individual repo you're using.
>> Consider adding `cd "$srcdir"` to the beginning of the `package()` function.
>> Please excuse my use of bastardized english. I could have worded my
>> suggestion more clearly.
> It was my fault, your english is good, it was just that I didn't
> understand what you meant me to add. (^_^)
>> The bitcoin docs discuss how to build bitcoin on unix.  (zetacoin is
>> based on bitcoin, so the bitcoin docs should be relevant) They state:
>>> miniupnpc may be used for UPnP port mapping. It can be downloaded from
>> here. UPnP support is compiled in and turned off by default.
>> Also, namcap complains about the zetacoin-qt-git package if miniupnpc is
>> not listed as a dependency:
>>> zetacoin-qt-git E: Dependency miniupnpc detected and not included
>> (libraries ['usr/lib/libminiupnpc.so.9'] needed in files
>> These two facts indicate that miniupnpc should be included as a dependency.
>> I suggest adding a comment to the top of the PKGBUILD explaining why
>> miniupnpc is included as a dependency.
> Ok, readded it as a dependency and added the comment at the top.
>>  https://github.com/bitcoin/bitcoin/blob/master/doc/build-unix.md
> Thanks for all the help!
More information about the aur-general