Re: svn commit: r323365 - in head: cddl/lib/libzpool gnu/usr.bin/binutils/libbfd usr.bin/svn/lib/libsvn_client

2017-09-09 Thread Ngie Cooper (yaneurabeya) 

> On Sep 9, 2017, at 14:51, Konstantin Belousov  wrote:

…

> Then, why peigen.c was left in the SRCS ?

I’m not sure, but you might have thought peigen.c and pepigen.c were the same 
file. If so, they’re not — they have slightly different spelling.
HTH,
-Ngie


signature.asc
Description: Message signed with OpenPGP using GPGMail

Re: svn commit: r323365 - in head: cddl/lib/libzpool gnu/usr.bin/binutils/libbfd usr.bin/svn/lib/libsvn_client

2017-09-09 Thread Konstantin Belousov 
On Sat, Sep 09, 2017 at 12:56:20PM -0700, Ngie Cooper (yaneurabeya) wrote:
> 
> > On Sep 9, 2017, at 11:35, Konstantin Belousov  wrote:
> > 
> > On Sat, Sep 09, 2017 at 01:18:33PM +, Jonathan Anderson wrote:
> >> Author: jonathan
> >> Date: Sat Sep  9 13:18:32 2017
> >> New Revision: 323365
> >> URL: https://svnweb.freebsd.org/changeset/base/323365
> >> 
> >> Log:
> >>  Remove redundant source and object files.
> > 
> >> Modified: head/gnu/usr.bin/binutils/libbfd/Makefile.amd64
> >> ==
> >> --- head/gnu/usr.bin/binutils/libbfd/Makefile.amd64Sat Sep  9 
> >> 12:50:12 2017(r323364)
> >> +++ head/gnu/usr.bin/binutils/libbfd/Makefile.amd64Sat Sep  9 
> >> 13:18:32 2017(r323365)
> >> @@ -9,7 +9,6 @@ DEFAULT_VECTOR=bfd_elf64_x86_64_freebsd_vec
> >> 
> >> SRCS+= elf64-x86-64.c \
> >>efi-app-x86_64.c \
> >> -  efi-app-ia32.c \
> >>elf64.c \
> >>elf64-gen.c \
> >>elf64-target.h \
> > Why did you claimed that x86 efi binfmt is  redundant ?
> 
>   It???s redundant because of how the Makefile gets .include???d (amd64 
> requires i386???s Makefile).
> Cheers,
> -Ngie
> 
> $ grep -r efi-app gnu/usr.bin/binutils/libbfd/
> gnu/usr.bin/binutils/libbfd/Makefile.amd64: efi-app-x86_64.c \
> gnu/usr.bin/binutils/libbfd/Makefile.amd64: efi-app-ia32.c \
> gnu/usr.bin/binutils/libbfd/Makefile.i386:  efi-app-ia32.c \
> $ grep include gnu/usr.bin/binutils/libbfd/Makefile.amd64
> .include "${.CURDIR}/Makefile.i386???

Then, why peigen.c was left in the SRCS ?
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Re: svn commit: r323365 - in head: cddl/lib/libzpool gnu/usr.bin/binutils/libbfd usr.bin/svn/lib/libsvn_client

2017-09-09 Thread Ngie Cooper (yaneurabeya) 

> On Sep 9, 2017, at 11:35, Konstantin Belousov  wrote:
> 
> On Sat, Sep 09, 2017 at 01:18:33PM +, Jonathan Anderson wrote:
>> Author: jonathan
>> Date: Sat Sep  9 13:18:32 2017
>> New Revision: 323365
>> URL: https://svnweb.freebsd.org/changeset/base/323365
>> 
>> Log:
>>  Remove redundant source and object files.
> 
>> Modified: head/gnu/usr.bin/binutils/libbfd/Makefile.amd64
>> ==
>> --- head/gnu/usr.bin/binutils/libbfd/Makefile.amd64  Sat Sep  9 12:50:12 
>> 2017(r323364)
>> +++ head/gnu/usr.bin/binutils/libbfd/Makefile.amd64  Sat Sep  9 13:18:32 
>> 2017(r323365)
>> @@ -9,7 +9,6 @@ DEFAULT_VECTOR=  bfd_elf64_x86_64_freebsd_vec
>> 
>> SRCS+=   elf64-x86-64.c \
>>  efi-app-x86_64.c \
>> -efi-app-ia32.c \
>>  elf64.c \
>>  elf64-gen.c \
>>  elf64-target.h \
> Why did you claimed that x86 efi binfmt is  redundant ?

It’s redundant because of how the Makefile gets .include’d (amd64 
requires i386’s Makefile).
Cheers,
-Ngie

$ grep -r efi-app gnu/usr.bin/binutils/libbfd/
gnu/usr.bin/binutils/libbfd/Makefile.amd64: efi-app-x86_64.c \
gnu/usr.bin/binutils/libbfd/Makefile.amd64: efi-app-ia32.c \
gnu/usr.bin/binutils/libbfd/Makefile.i386:  efi-app-ia32.c \
$ grep include gnu/usr.bin/binutils/libbfd/Makefile.amd64
.include "${.CURDIR}/Makefile.i386”


signature.asc
Description: Message signed with OpenPGP using GPGMail

Re: svn commit: r323365 - in head: cddl/lib/libzpool gnu/usr.bin/binutils/libbfd usr.bin/svn/lib/libsvn_client

2017-09-09 Thread Konstantin Belousov 
On Sat, Sep 09, 2017 at 01:18:33PM +, Jonathan Anderson wrote:
> Author: jonathan
> Date: Sat Sep  9 13:18:32 2017
> New Revision: 323365
> URL: https://svnweb.freebsd.org/changeset/base/323365
> 
> Log:
>   Remove redundant source and object files.

> Modified: head/gnu/usr.bin/binutils/libbfd/Makefile.amd64
> ==
> --- head/gnu/usr.bin/binutils/libbfd/Makefile.amd64   Sat Sep  9 12:50:12 
> 2017(r323364)
> +++ head/gnu/usr.bin/binutils/libbfd/Makefile.amd64   Sat Sep  9 13:18:32 
> 2017(r323365)
> @@ -9,7 +9,6 @@ DEFAULT_VECTOR=   bfd_elf64_x86_64_freebsd_vec
>  
>  SRCS+=   elf64-x86-64.c \
>   efi-app-x86_64.c \
> - efi-app-ia32.c \
>   elf64.c \
>   elf64-gen.c \
>   elf64-target.h \
Why did you claimed that x86 efi binfmt is  redundant ?
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Re: svn commit: r323365 - in head: cddl/lib/libzpool gnu/usr.bin/binutils/libbfd usr.bin/svn/lib/libsvn_client

2017-09-09 Thread O. Hartmann 
Am Sat, 9 Sep 2017 13:18:33 + (UTC)
Jonathan Anderson  schrieb:

This bug has just been introduced in the transition r323342 -> r323365:


[...]

building shared library libncurses.so.8
--- lib/libgeom__L ---
/usr/obj/usr/src/tmp/usr/bin/ld: 
/usr/obj/usr/src/tmp/usr/lib/libsbuf.a(subr_sbuf.o):
relocation R_X86_64_32 against `_DefaultRuneLocale' can not be used when making 
a shared
object; recompile with -fPIC /usr/obj/usr/src/tmp/usr/lib/libsbuf.a: could not 
read
symbols: Bad value cc: error: linker command failed with exit code 1 (use -v to 
see
invocation) *** [libgeom.so.5] Error code 1

make[4]: stopped in /usr/src/lib/libgeom
.ERROR_TARGET='libgeom.so.5'
.ERROR_META_FILE='/usr/obj/usr/src/lib/libgeom/libgeom.so.5.meta'
.MAKE.LEVEL='4'
MAKEFILE=''
.MAKE.MODE='meta missing-filemon=yes missing-meta=yes silent=yes verbose'
_ERROR_CMD='@echo building shared library libgeom.so.5; @rm -f libgeom.so.5 
libgeom.so;
@sh /usr/src/tools/install.sh -l s  libgeom.so.5 libgeom.so; cc -target
x86_64-unknown-freebsd12.0 --sysroot=/usr/obj/usr/src/tmp 
-B/usr/obj/usr/src/tmp/usr/bin
-fstack-protector-strong -shared -Wl,-x -Wl,--fatal-warnings 
-Wl,--warn-shared-textrel
-o libgeom.so.5 -Wl,-soname,libgeom.so.5  `NM='nm' NMFLAGS='' lorder 
geom_getxml.pico
geom_stats.pico geom_xml2tree.pico geom_ctl.pico geom_util.pico |  tsort -q`  
-lbsdxml
-lsbuf;' .CURDIR='/usr/src/lib/libgeom'

> Author: jonathan
> Date: Sat Sep  9 13:18:32 2017
> New Revision: 323365
> URL: https://svnweb.freebsd.org/changeset/base/323365
> 
> Log:
>   Remove redundant source and object files.
>   
>   Reviewed by:bdrewery, ngie
>   MFC after:  1 week
>   Sponsored by:   DARPA, AFRL
>   Differential Revision:  https://reviews.freebsd.org/D12208
> 
> Modified:
>   head/cddl/lib/libzpool/Makefile
>   head/gnu/usr.bin/binutils/libbfd/Makefile.amd64
>   head/usr.bin/svn/lib/libsvn_client/Makefile
> 
> Modified: head/cddl/lib/libzpool/Makefile
> ==
> --- head/cddl/lib/libzpool/Makefile   Sat Sep  9 12:50:12 2017
> (r323364)
> +++ head/cddl/lib/libzpool/Makefile   Sat Sep  9 13:18:32 2017
> (r323365)
> @@ -26,7 +26,7 @@ ATOMIC_SRCS=opensolaris_atomic.c
>  
>  LIB= zpool
>  
> -ZFS_COMMON_SRCS= ${ZFS_COMMON_OBJS:C/.o$/.c/} vdev_file.c trim_map.c
> +ZFS_COMMON_SRCS= ${ZFS_COMMON_OBJS:C/.o$/.c/} trim_map.c
>  ZFS_SHARED_SRCS= ${ZFS_SHARED_OBJS:C/.o$/.c/}
>  KERNEL_SRCS= kernel.c taskq.c util.c
>  LIST_SRCS=   list.c
> 
> Modified: head/gnu/usr.bin/binutils/libbfd/Makefile.amd64
> ==
> --- head/gnu/usr.bin/binutils/libbfd/Makefile.amd64   Sat Sep  9 12:50:12
> 2017  (r323364) +++ head/gnu/usr.bin/binutils/libbfd/Makefile.amd64   Sat
> Sep  9 13:18:32 2017  (r323365) @@ -9,7 +9,6 @@ DEFAULT_VECTOR=
> bfd_elf64_x86_64_freebsd_vec 
>  SRCS+=   elf64-x86-64.c \
>   efi-app-x86_64.c \
> - efi-app-ia32.c \
>   elf64.c \
>   elf64-gen.c \
>   elf64-target.h \
> 
> Modified: head/usr.bin/svn/lib/libsvn_client/Makefile
> ==
> --- head/usr.bin/svn/lib/libsvn_client/Makefile   Sat Sep  9 12:50:12 2017
> (r323364) +++ head/usr.bin/svn/lib/libsvn_client/Makefile Sat Sep  9 
> 13:18:32
> 2017  (r323365) @@ -12,7 +12,7 @@ SRCS=   add.c blame.c cat.c
> changelist.c checkout.c cle copy_foreign.c ctx.c delete.c deprecated.c diff.c 
> \
>   diff_local.c diff_summarize.c export.c externals.c import.c \
>   info.c iprops.c list.c locking_commands.c log.c merge.c \
> - mergeinfo.c mtcc.c patch.c log.c prop_commands.c \
> + mergeinfo.c mtcc.c patch.c prop_commands.c \
>   ra.c relocate.c repos_diff.c resolved.c revert.c revisions.c \
>   status.c switch.c update.c upgrade.c url.c util.c version.c
>  
> ___
> svn-src-h...@freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/svn-src-head
> To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"



-- 
O. Hartmann

Ich widerspreche der Nutzung oder Übermittlung meiner Daten für
Werbezwecke oder für die Markt- oder Meinungsforschung (§ 28 Abs. 4 BDSG).


pgpOLcU6iEnP6.pgp
Description: OpenPGP digital signature