Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 1 Apr 2012 13:51:07 -0400
From:      Eitan Adler <lists@eitanadler.com>
To:        Ruslan Mahmatkhanov <cvs-src@yandex.ru>
Cc:        FreeBSD Ports Mailing List <ports@freebsd.org>
Subject:   Re: Preferred way of applying OSVERSION-dependend patches
Message-ID:  <CAF6rxg=BU4NzRrcAQGJfZTJXLN0550mgscPaFq1pq=FjSO6bkA@mail.gmail.com>
In-Reply-To: <4F7827A7.3030804@yandex.ru>
References:  <4F7827A7.3030804@yandex.ru>

next in thread | previous in thread | raw e-mail | index | archive | help
On 1 April 2012 06:02, Ruslan Mahmatkhanov <cvs-src@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
> =C2=A0 of checks in Makefile (.if ${OSVERSION}).
> 2. Add an patch-* to files subdirectory that applies unconditionally
> =C2=A0 and that checking for __FreeBSD_version in app code itself.
>
> The second one will guarantee that the patch still applies cleanly on por=
t
> versions update, but it will make it more hard to find conditional checks
> for obsoleted FreeBSD versions (because most people, I believe, only look=
ing
> 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.

--=20
Eitan Adler



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAF6rxg=BU4NzRrcAQGJfZTJXLN0550mgscPaFq1pq=FjSO6bkA>