Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 09 Jul 2015 15:33:20 -0500
From:      Larry Rosenman <ler@lerctr.org>
To:        Baptiste Daroussin <bapt@freebsd.org>
Cc:        Peter Wemm <peter@wemm.org>, svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Mariusz Zaborski <oshogbo@freebsd.org>, owner-svn-src-all@freebsd.org
Subject:   Re: svn commit: r285139 - in head: lib/libnv sys/conf sys/contrib/libnv sys/kern sys/sys
Message-ID:  <2c517ae466390360541ba3e169b19ded@thebighonker.lerctr.org>
In-Reply-To: <20150709203154.GB72952@ivaldir.etoilebsd.net>
References:  <201507041633.t64GXcmT097180@repo.freebsd.org> <12132227.txsMvZmjyr@overcee.wemm.org> <20150709203154.GB72952@ivaldir.etoilebsd.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2015-07-09 15:31, Baptiste Daroussin wrote:
> On Thu, Jul 09, 2015 at 01:29:54PM -0700, Peter Wemm wrote:
>> On Saturday, July 04, 2015 04:33:38 PM Mariusz Zaborski wrote:
>> > Author: oshogbo
>> > Date: Sat Jul  4 16:33:37 2015
>> > New Revision: 285139
>> > URL: https://svnweb.freebsd.org/changeset/base/285139
>> >
>> > Log:
>> >   Move the nvlist source and private includes from sys/kern to seperate
>> >   directory sys/contrib/libnv.
>> >
>> >   The goal of this operation is to NOT install header files which shouldn't
>> >   be used outside the nvlist library.
>> 
>> >   head/sys/contrib/libnv/nvlist.c
>> >      - copied, changed from r285128, head/sys/kern/subr_nvlist.c
>> 
>> You have broken kernel builds for the freebsd.org cluster.  By 
>> renaming
>> "subr_nvlist.o" to "nvlist.o" you now cause:
>> 
>> make[2]: ".../Makefile" line 3143: warning: duplicate script for 
>> target
>> "nvpair.o" ignored
>> make[2]: ".../Makefile" line 1260: warning: using previous script for
>> "nvpair.o" defined here
>> 
>> ....
>> 
>> linking kernel.debug
>> nvpair.o: In function `illumos_nvlist_add_nvlist':
>> /usr/src/sys/cddl/contrib/opensolaris/common/nvpair/nvpair.c:1136: 
>> multiple
>> definition of `illumos_nvlist_add_nvlist'
>> nvpair.o:/usr/src/sys/cddl/contrib/opensolaris/common/nvpair/nvpair.c:1136:
>> first defined here
>> nvpair.o: In function `illumos_nvlist_add_nvpair':
>> /usr/src/sys/cddl/contrib/opensolaris/common/nvpair/nvpair.c:1971: 
>> multiple
>> definition of `illumos_nvlist_add_nvpair'
>> nvpair.o:/usr/src/sys/cddl/contrib/opensolaris/common/nvpair/nvpair.c:1971:
>> first defined here
>> ....
>> 
>> I'm going to attempt to revert this for the freebsd.org cluster 
>> because I need
>> it to compile for openssl.
>> 
> You probably want to specify that this happens when zfs is built in the 
> kernel
> not as a module.
> 
> Best regards,
> Bapt
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=201356

has a patch

-- 
Larry Rosenman                     http://www.lerctr.org/~ler
Phone: +1 214-642-9640                 E-Mail: ler@lerctr.org
US Mail: 108 Turvey Cove, Hutto, TX 78634-5688



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