patch to bsd.ports.mk to support out-of-tree patches.

[Bryan Drewery]

On 3/24/2015 5:32 AM, Marcus von Appen wrote:
> Julian Elischer <julian at freebsd.org>:
> 
> [...]
>>          esac | ${PATCH} ${PATCH_DIST_ARGS} `patch_dist_strip $$i` ; \
>>      done )
>>  .endif
>> +.if defined(EXTRA_PATCH_TREE)
> [...]
>> +.endif
>>  .if defined(EXTRA_PATCHES)
>>      @set -e ; \
>>      for i in ${EXTRA_PATCHES}; do \
>>
>>
>> ============
> 
> Nice. I'd however change the patch behaviour to the following:
> 
> - patch-* from FreeBSD
> - EXTRA_PATCHES from FreeBSD
> - local patches
> 
> Your patch looks like it appleis the out-of-tree patches prior
> to any EXTRA_PATCHES defined by the port itself. This should not be
> the case, in my opinion. Locally managed patches should always come
> last to ensure that all FreeBSD/maintainer-specific bits have been
> applied and the local changes are just added on top of those.
> 
> Cheers
> Marcus
> 

Agreed. The local patches are intended to avoid touching the port. If
they are not last then you may have to go tweaking some of the other
ports patches to fix your own.

-- 
Regards,
Bryan Drewery

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freebsd.org/pipermail/freebsd-ports/attachments/20150324/2a2968e9/attachment.sig>