Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 09 Jun 2015 14:12:54 +0300
From:      Andriy Gapon <avg@FreeBSD.org>
To:        Jan Beich <jbeich@FreeBSD.org>
Cc:        src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org
Subject:   Re: svn commit: r284107 - head/sys/cddl/compat/opensolaris/sys
Message-ID:  <5576CA36.8060403@FreeBSD.org>
In-Reply-To: <twuh-9nr0-wny@FreeBSD.org>
References:  <201506070854.t578sPdR012092@svn.freebsd.org> <twuh-9nr0-wny@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 09/06/2015 13:31, Jan Beich wrote:
> Andriy Gapon <avg@FreeBSD.org> writes:
> 
>> Author: avg Date: Sun Jun  7 08:54:25 2015 New Revision: 284107 URL:
>> https://svnweb.freebsd.org/changeset/base/284107
>> 
>> Log: compat nvpair.h: make sure that the names are mangled only for
>> kernel
>> 
>> Currently there is no good reason to mangle the userland API. The change
>> was introduced in eac1d566b46edef765754203bef22c75c1699966, r279437.
>> Also see https://reviews.freebsd.org/D1881.
> 
> Can you bump __FreeBSD_version? At least sysutils/geomWatch requires it.
> 
> https://svnweb.freebsd.org/changeset/ports/388906
> 

There should have been a bug report and an src fix instead of (parts of)
https://svnweb.freebsd.org/ports?view=revision&revision=386870

I wonder why the libnvpair ABI breakage was silently accepted but the ABI fix
requires some special magic...

-- 
Andriy Gapon



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5576CA36.8060403>