Re: mail/alpine new Makefile

2017-06-09 Thread Marco Beishuizen
On Fri, 9 Jun 2017, the wise Kubilay Kocak wrote: I haven't looked, but might this be due to: # Option PICO PICO_RUN_DEPENDS=pico:editors/pico-alpine Since pico-alpine is a slave port of alpine, you'll want to make sure you're overriding things correctly, and something that was replac

Re: mail/alpine new Makefile

2017-06-09 Thread Marco Beishuizen
On Fri, 9 Jun 2017, the wise Kubilay Kocak wrote: I think I had a brainfail, and I believe it is in-fact fine to use both (if necessary) and the wording of the handbook might need to be tweaked not to imply it. That aside, I can't see use (testing) of PORT_OPTIONS:M or other variables that p

Re: mail/alpine new Makefile

2017-06-09 Thread Kubilay Kocak
On 6/9/17 9:50 PM, Marco Beishuizen wrote: > On Fri, 9 Jun 2017, the wise Kubilay Kocak wrote: > >> On 6/9/17 7:41 PM, Marco Beishuizen wrote: >> >> Hi Marco, >> >> I believe the reason is: >> >>> .include >> >> The port includes this ... AND >> >> >> >>> .include >> >> This -^

Re: mail/alpine new Makefile

2017-06-09 Thread Kubilay Kocak
On 6/9/17 9:26 PM, Marco Beishuizen wrote: > On Fri, 9 Jun 2017, the wise Kubilay Kocak wrote: > >> On 6/9/17 7:41 PM, Marco Beishuizen wrote: >> >> Hi Marco, >> >> I believe the reason is: >> >>> .include >> >> The port includes this ... AND >> >> >> >>> .include >> >> This -^

Re: mail/alpine new Makefile

2017-06-09 Thread Marco Beishuizen
On Fri, 9 Jun 2017, the wise Kubilay Kocak wrote: On 6/9/17 7:41 PM, Marco Beishuizen wrote: Hi Marco, I believe the reason is: .include The port includes this ... AND .include This -^ Use one or the other, or neither if its *all* options helpers, but not both. Se

Re: mail/alpine new Makefile

2017-06-09 Thread Marco Beishuizen
On Fri, 9 Jun 2017, the wise Kubilay Kocak wrote: On 6/9/17 7:41 PM, Marco Beishuizen wrote: Hi Marco, I believe the reason is: .include The port includes this ... AND .include This -^ Use one or the other, or neither if its *all* options helpers, but not both. Se

Re: mail/alpine new Makefile

2017-06-09 Thread Kubilay Kocak
On 6/9/17 7:41 PM, Marco Beishuizen wrote: Hi Marco, I believe the reason is: > OPTIONS_DEFINE=ASPELL CONS25 DOCS IPV6 LDAP MAILDIR MOUSE NLS > NOSPELL PASSFILE PICO THREADS > OPTIONS_DEFAULT+=MOUSE PICO THREADS > > CONS25_DESC=Add a patch to support color for default console >

mail/alpine new Makefile

2017-06-09 Thread Marco Beishuizen
Hi, Being the maintainer of mail/alpine, I'm trying to cleanup it's Makefile. What I now have is this: PORTNAME?= alpine PORTVERSION=2.21 PORTREVISION?= 0 CATEGORIES?=mail news ipv6 MASTER_SITES= http://alpine.freeiz.com/alpine/release/src/ DIST_SUBDIR=alpine-${PORTVERSION}