Date: Tue, 20 May 2014 22:16:25 +0200 From: Olivier Duchateau <duchateau.olivier@gmail.com> To: "J.R. Oldroyd" <fbsd@opal.com> Cc: freebsd-xfce@freebsd.org Subject: Re: sysutils/xfce4-wavelan-plugin Message-ID: <CAHcdu2qJgUxfg8O5DfmdeU1sadcrPfKXTG1niSTB_jBz7gj4mw@mail.gmail.com> In-Reply-To: <20140516185220.4be1e976@shibato> References: <20140516180324.2fcf73f0@shibato> <CAHcdu2q6e2UvY-KhLJrKSYhwQ8kFtJEP8LnCrp7O_Q6R0ePY_Q@mail.gmail.com> <20140516185220.4be1e976@shibato>
next in thread | previous in thread | raw e-mail | index | archive | help
2014-05-16 18:52 GMT+02:00 J.R. Oldroyd <fbsd@opal.com>: > On Fri, 16 May 2014 18:31:04 +0200 Olivier Duchateau <duchateau.olivier@gmail.com> wrote: >> >> Thanks you for your help. >> > You're welcome. Feel free to upstream these changes. > >> Check of kernel version is used for Debian GNU/kFreeBSD port (I know >> it's overkill). >> > I won't touch that for now, then, although I do think it would be cleaner > to check the kernel version once at the top, then use plain __FreeBSD__ > and __FreeBSD_version throughout the rest. > > And __FreeBSD_kernel_kernel__?? That can't be right. > > -jr > > Patch [1] is it ok? I removed __FreeBSD_kernel_version macro, because FreeBSD <= 7.x is not anymore supported. [1] https://people.freebsd.org/~olivierd/patches/xfce4-wavelan-plugin.diff -- olivier
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAHcdu2qJgUxfg8O5DfmdeU1sadcrPfKXTG1niSTB_jBz7gj4mw>