Re: devel/shellcheck - dependency issue

2018-04-21 Thread Matthias Kilian
On Sat, Apr 21, 2018 at 09:27:09PM +0100, Nigel Taylor wrote: > REVISION-main = R looks like a typo, should be kept at 0, Oh shit, i indeed now have a shellcheck-0.4.7pR.tgz in my package repository ;-) > Otherwise ok with me > > I would question the use of lang/ghc, as MODULE = lang/ghc

Re: devel/shellcheck - dependency issue

2018-04-21 Thread Nigel Taylor
REVISION-main = R looks like a typo, should be kept at 0, Otherwise ok with me I would question the use of lang/ghc, as MODULE = lang/ghc already sets things, then need to use += in BUILD_DEPENDS, for now ok. On 04/21/18 21:10, Matthias Kilian wrote: > Hi, > > On Fri, Apr 20, 2018 at

Re: devel/shellcheck - dependency issue

2018-04-21 Thread Matthias Kilian
Hi, On Fri, Apr 20, 2018 at 11:00:48PM +0100, Nigel Taylor wrote: > Makefile has > > RUN_DEPENDS-lib = ${BUILD_DEPENDS} > > In the case of USE_CCACHE being set to Yes devel/ccache is added to the > build dependencies, when not required at runtime. Right. The diff basically swaps those

devel/shellcheck - dependency issue

2018-04-20 Thread Nigel Taylor
Makefile has RUN_DEPENDS-lib = ${BUILD_DEPENDS} In the case of USE_CCACHE being set to Yes devel/ccache is added to the build dependencies, when not required at runtime. This shouldn't be used in Makefile's.