> On 1 April 2012 06:02, Ruslan Mahmatkhanov <cvs-...@yandex.ru> wrote:
>>
>> Good day!
>>
>> What is a preferred way of doing subj?
>>
>> 1. Add an extra-patch-* to files/ subdirectory and apply it depending
>>   of checks in Makefile (.if ${OSVERSION}).
>> 2. Add an patch-* to files subdirectory that applies unconditionally
>>   and that checking for __FreeBSD_version in app code itself.
>>
>> The second one will guarantee that the patch still applies cleanly on
>> port
>> versions update, but it will make it more hard to find conditional
>> checks
>> for obsoleted FreeBSD versions (because most people, I believe, only
>> looking
>> to Makefile for things like this).
>
> #1 for exactly the reason you specify: it makes it easier to figure
> out why something is being patched and when that patch is no longer
> required.
>

I say #3 :)

So if it is non-conditional in the port, but you choose to make it
conditional in the code -- Send the patch upstream, or to maintainer of
code, so the patch isn't necessary in the portstree.

-jgh


_______________________________________________
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"

Reply via email to