Date: Thu, 16 Oct 2014 18:53:44 -0700 From: Davide Italiano <davide@freebsd.org> To: "Bjoern A. Zeeb" <bz@freebsd.org> Cc: "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org> Subject: Re: svn commit: r273174 - in head/sys: amd64/amd64 amd64/vmm arm/arm cam/ata cddl/contrib/opensolaris/uts/common/dtrace dev/acpica dev/asmc dev/cfi dev/drm2 dev/hatm dev/mii dev/nfe dev/patm dev/pci de... Message-ID: <CACYV=-FZJTwZSBX_mHr8UtOXgMoEmGP0sGtqHggP6DHj7v%2BZHg@mail.gmail.com> In-Reply-To: <E9D0F7BC-484A-4369-B706-8999F8E41B0D@FreeBSD.org> References: <201410161804.s9GI4hed091471@svn.freebsd.org> <E9D0F7BC-484A-4369-B706-8999F8E41B0D@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Oct 16, 2014 at 4:44 PM, Bjoern A. Zeeb <bz@freebsd.org> wrote: > > On 16 Oct 2014, at 18:04 , Davide Italiano <davide@FreeBSD.org> wrote: > >> Author: davide >> Date: Thu Oct 16 18:04:43 2014 >> New Revision: 273174 >> URL: https://svnweb.freebsd.org/changeset/base/273174 >> >> Log: >> Follow up to r225617. In order to maximize the re-usability of kernel c= ode >> in userland rename in-kernel getenv()/setenv() to kern_setenv()/kern_ge= tenv(). >> This fixes a namespace collision with libc symbols. >> >> Submitted by: kmacy >> Tested by: make universe > > Hmm I still see a lot of mips kernels broken with > > /scratch/tmp/bz/head.svn/sys/mips/rmi/xlr_machdep.c: In function 'platfor= m_start': > /scratch/tmp/bz/head.svn/sys/mips/rmi/xlr_machdep.c:461: warning: implici= t declaration of function 'setenv' > /scratch/tmp/bz/head.svn/sys/mips/rmi/xlr_machdep.c:461: warning: nested = extern declaration of =E2=80=98setenv=E2=80=99 [-Wnested-externs] > > on my incremental builds. > > =E2=80=94 > Bjoern A. Zeeb "Come on. Learn, goddamn it.", WarGames, 1983 > Hi, I think you're right and I apologize for the breakage (apparently I didn't look at all the log files properly). Can you tell me if this fixes for you: https://people.freebsd.org/~davide/patches/kernsetenv_fixup.diff ? If yes, I'll commit. BTW, it would be cool if universe will put logs of successful builds and logs of failed builds in different directories for easier inspection. If there's already something like this and you can point me to that, it would be appreciated. --=20 Davide "There are no solved problems; there are only problems that are more or less solved" -- Henri Poincare
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CACYV=-FZJTwZSBX_mHr8UtOXgMoEmGP0sGtqHggP6DHj7v%2BZHg>