Re: svn commit: r352707 - in head/sys: conf kern net sys

2019-09-26 Thread Charlie Li via svn-src-all
Kyle Evans wrote:
> On Thu, Sep 26, 2019 at 9:49 AM Charlie Li wrote:
>> This breaks building the drm-kmod ports, as the build cannot find
>> opt_epoch.h (drm-devel-kmod example shown, drm-current-kmod dies the
>> exact same way):
>>
>> --- linux_anon_inodes.o ---
>> cc  -O2 -pipe -fno-strict-aliasing -include
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/drivers/gpu/drm/drm_os_config.h
>> '-DKBUILD_MODNAME="linuxkpi_gplv2"'  -Werror -D_KERNEL -DKLD_MODULE
>> -nostdinc
>> -I/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/include 
>> -I/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/dummy/include
>> -I/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include
>> -I/usr/src/sys/compat/linuxkpi/common/include -I. -I/usr/src/sys
>> -I/usr/src/sys/contrib/ck/include -fno-common  -fno-omit-frame-pointer
>> -mno-omit-leaf-frame-pointer
>> -fdebug-prefix-map=./machine=/usr/src/sys/amd64/include
>> -fdebug-prefix-map=./x86=/usr/src/sys/x86/include -MD
>> -MF.depend.linux_anon_inodes.o -MTlinux_anon_inodes.o -mcmodel=kernel
>> -mno-red-zone -mno-mmx -mno-sse -msoft-float
>> -fno-asynchronous-unwind-tables -ffreestanding -fwrapv -fstack-protector
>> -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes
>> -Wmissing-prototypes -Wpointer-arith -Wcast-qual -Wundef
>> -Wno-pointer-sign -D__printf__=__freebsd_kprintf__
>> -Wmissing-include-dirs -fdiagnostics-show-option -Wno-unknown-pragmas
>> -Wno-error-tautological-compare -Wno-error-empty-body
>> -Wno-error-parentheses-equality -Wno-error-unused-function
>> -Wno-error-pointer-sign -Wno-error-shift-negative-value
>> -Wno-address-of-packed-member -Wno-format-zero-length -Wno-pointer-arith
>>   -mno-aes -mno-avx  -std=iso9899:1999 -c
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/src/linux_anon_inodes.c
>> -o linux_anon_inodes.o
>> In file included from
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/src/linux_anon_inodes.c:12:
>> In file included from
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/anon_inodes.h:4:
>> In file included from
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/fs.h:6:
>> In file included from
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/shrinker.h:5:
>> In file included from
>> /usr/src/sys/compat/linuxkpi/common/include/linux/list.h:56:
>> In file included from /usr/src/sys/net/if_var.h:83:
>> /usr/src/sys/sys/epoch.h:44:10: fatal error: 'opt_epoch.h' file not found
>> #include "opt_epoch.h"
>>  ^
>> --- linux_anon_inodefs.o ---
>> In file included from
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/src/linux_anon_inodefs.c:45:
>> In file included from
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/debugfs.h:18:
>> In file included from
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/fs.h:6:
>> In file included from
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/shrinker.h:5:
>> In file included from
>> /usr/src/sys/compat/linuxkpi/common/include/linux/list.h:56:
>> In file included from /usr/src/sys/net/if_var.h:83:
>> /usr/src/sys/sys/epoch.h:44:10: fatal error: 'opt_epoch.h' file not found
>> #include "opt_epoch.h"
>>  ^
>> --- linux_anon_inodes.o ---
>> 1 error generated.
>> *** [linux_anon_inodes.o] Error code 1
>>
>> make[2]: stopped in
>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi
>> --- linux_anon_inodefs.o ---
>> 1 error generated.
>> *** [linux_anon_inodefs.o] Error code 1
>>
>> Interestingly enough, does not happen when drm-current-kmod is built as
>> part of buildkernel (using an existing installed package with SOURCE on).
>>
> 
> FWIW, johalun noticed this yesterday and addressed it here:
> https://github.com/FreeBSDDesktop/kms-drm/commit/b486949e7e9f0cfe8dac5f0ac7fe1a660300981d
> 
Ah, of course I would miss these commits in the kms-drm repo,
considering that I watch them roll in. Will wait for the updated
snapshots in ports.

-- 
Charlie Li
…nope, still don't have an exit line.

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r352707 - in head/sys: conf kern net sys

2019-09-26 Thread Charlie Li via svn-src-all
Gleb Smirnoff wrote:
> Modified: head/sys/conf/options
> ==
> --- head/sys/conf/options Wed Sep 25 18:09:19 2019(r352706)
> +++ head/sys/conf/options Wed Sep 25 18:26:31 2019(r352707)
> @@ -712,6 +712,8 @@ WITNESS_SKIPSPIN  opt_witness.h
>  WITNESS_COUNTopt_witness.h
>  OPENSOLARIS_WITNESS  opt_global.h
>  
> +EPOCH_TRACE  opt_epoch.h
> +
>  # options for ACPI support
>  ACPI_DEBUG   opt_acpi.h
>  ACPI_MAX_TASKS   opt_acpi.h
> 
> Modified: head/sys/sys/epoch.h
> ==
> --- head/sys/sys/epoch.h  Wed Sep 25 18:09:19 2019(r352706)
> +++ head/sys/sys/epoch.h  Wed Sep 25 18:26:31 2019(r352707)
> @@ -41,6 +41,8 @@ typedef struct epoch_context *epoch_context_t;
>  #include 
>  #include 
>  
> +#include "opt_epoch.h"
> +
>  struct epoch;
>  typedef struct epoch *epoch_t;
>  
This breaks building the drm-kmod ports, as the build cannot find
opt_epoch.h (drm-devel-kmod example shown, drm-current-kmod dies the
exact same way):

--- linux_anon_inodes.o ---
cc  -O2 -pipe -fno-strict-aliasing -include
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/drivers/gpu/drm/drm_os_config.h
'-DKBUILD_MODNAME="linuxkpi_gplv2"'  -Werror -D_KERNEL -DKLD_MODULE
-nostdinc
-I/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/include 
-I/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/dummy/include
-I/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include
-I/usr/src/sys/compat/linuxkpi/common/include -I. -I/usr/src/sys
-I/usr/src/sys/contrib/ck/include -fno-common  -fno-omit-frame-pointer
-mno-omit-leaf-frame-pointer
-fdebug-prefix-map=./machine=/usr/src/sys/amd64/include
-fdebug-prefix-map=./x86=/usr/src/sys/x86/include -MD
-MF.depend.linux_anon_inodes.o -MTlinux_anon_inodes.o -mcmodel=kernel
-mno-red-zone -mno-mmx -mno-sse -msoft-float
-fno-asynchronous-unwind-tables -ffreestanding -fwrapv -fstack-protector
-Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes
-Wmissing-prototypes -Wpointer-arith -Wcast-qual -Wundef
-Wno-pointer-sign -D__printf__=__freebsd_kprintf__
-Wmissing-include-dirs -fdiagnostics-show-option -Wno-unknown-pragmas
-Wno-error-tautological-compare -Wno-error-empty-body
-Wno-error-parentheses-equality -Wno-error-unused-function
-Wno-error-pointer-sign -Wno-error-shift-negative-value
-Wno-address-of-packed-member -Wno-format-zero-length -Wno-pointer-arith
  -mno-aes -mno-avx  -std=iso9899:1999 -c
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/src/linux_anon_inodes.c
-o linux_anon_inodes.o
In file included from
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/src/linux_anon_inodes.c:12:
In file included from
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/anon_inodes.h:4:
In file included from
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/fs.h:6:
In file included from
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/shrinker.h:5:
In file included from
/usr/src/sys/compat/linuxkpi/common/include/linux/list.h:56:
In file included from /usr/src/sys/net/if_var.h:83:
/usr/src/sys/sys/epoch.h:44:10: fatal error: 'opt_epoch.h' file not found
#include "opt_epoch.h"
 ^
--- linux_anon_inodefs.o ---
In file included from
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/src/linux_anon_inodefs.c:45:
In file included from
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/debugfs.h:18:
In file included from
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/fs.h:6:
In file included from
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi/gplv2/include/linux/shrinker.h:5:
In file included from
/usr/src/sys/compat/linuxkpi/common/include/linux/list.h:56:
In file included from /usr/src/sys/net/if_var.h:83:
/usr/src/sys/sys/epoch.h:44:10: fatal error: 'opt_epoch.h' file not found
#include "opt_epoch.h"
 ^
--- linux_anon_inodes.o ---
1 error generated.
*** [linux_anon_inodes.o] Error code 1

make[2]: stopped in
/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/linuxkpi
--- linux_anon_inodefs.o ---
1 error generated.
*** [linux_anon_inodefs.o] Error code 1

Interestingly enough, does not happen when drm-current-kmod is built as
part of buildkernel (using an existing installed package with SOURCE on).

-- 
Charlie Li
…nope, still don't have an exit line.

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r352275 - in head/lib/libedit: . TEST edit readline

2019-09-13 Thread Charlie Li via svn-src-all
Baptiste Daroussin wrote:
> Author: bapt
> Date: Fri Sep 13 06:50:02 2019
> New Revision: 352275
> URL: https://svnweb.freebsd.org/changeset/base/352275
> 
> Log:
>   Update libedit to a snapshot from 2019-09-10
>   Reviewed by:Yuichiro Naito 
>   MFC after:  3 weeks
>   Differential Revision:  https://reviews.freebsd.org/D21584
> 
I posted in phab but repeating here, this breaks the build on (at least)
aarch64:

--- terminal.o ---
/usr/src/contrib/libedit/terminal.c:569:41: error: comparison of
integers of different signs: 'wint_t' (aka 'int') and 'wchar_t' (aka
'unsigned int') [-Werror,-Wsign-compare]
el->el_cursor.v][where &
0370] !=
~~ ^
/usr/src/contrib/libedit/terminal.c:659:28: error: comparison of
integers of different signs: 'wint_t' (aka 'int') and 'wchar_t' (aka
'unsigned int') [-Werror,-Wsign-compare]
[el->el_cursor.h] ==
MB_FILL_CHAR)
~ ^

2 errors generated.
*** [terminal.o] Error code 1

-- 
Charlie Li
…nope, still don't have an exit line.

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r346410 - in head: lib/libc/gen share/man/man9 sys/conf sys/libkern sys/sys

2019-09-03 Thread Charlie Li via svn-src-all
O. Hartmann wrote:
> Am Fri, 19 Apr 2019 20:05:48 + (UTC) Conrad Meyer schrieb:
>> Modified: head/lib/libc/gen/Makefile.inc
>> ==
>> --- head/lib/libc/gen/Makefile.inc   Fri Apr 19 19:45:19 2019
>> (r346409)
>> +++ head/lib/libc/gen/Makefile.inc   Fri Apr 19 20:05:47 2019
>> (r346410)
>> @@ -3,6 +3,7 @@
> 
>>  # machine-independent gen sources
>>  .PATH: ${LIBC_SRCTOP}/${LIBC_ARCH}/gen ${LIBC_SRCTOP}/gen
>> +.PATH: ${SRCTOP}/sys/libkern
> 
>>  CONFS=  shells
> 
> 
> After applying r346410 to our source tree, buildworld fails (WITH_META_MODE 
> is set):
> 
> [...]
> Building /usr/obj/usr/src/amd64.amd64/lib/libc/ev_timers.o
> --- inet_ntoa.o ---
> In file included from /usr/src/sys/libkern/inet_ntoa.c:34:
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:172:8: error: 
> unknown type name 'bool'
> extern bool dynamic_kenv;
>^
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:211:5: warning: 
> declaration of
> built-in function 'setjmp' requires inclusion of the header 
> [-Wbuiltin-requires-header] int setjmp(struct _jmp_buf *) 
> __returns_twice; ^
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:212:6: warning: 
> declaration of
> built-in function 'longjmp' requires inclusion of the header 
> [-Wbuiltin-requires-header] voidlongjmp(struct _jmp_buf *, int) __dead2; ^
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:285:6: warning: 
> incompatible
> redeclaration of library function 'log' 
> [-Wincompatible-library-redeclaration] void
> log(int, const char *, ...) __printflike(2, 3); ^
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:285:6: note: 'log' 
> is a builtin with
> type 'double (double)' 
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:382:39: error:
> unknown type name 'uintfptr_t'; did you mean 'uintptr_t'? void
> profclock(int cnt, int
> usermode, uintfptr_t pc); ^
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/_stdint.h:80:22: note: 
> 'uintptr_t' declared
> here typedef __uintptr_t uintptr_t;
> 
> 
> Is there an include missing?
> Beat me to the initial email. With the specific file change quoted
above, looks like inet_*.c are getting pulled from sys/libkern instead
of the proper lib/libc/inet.

-- 
Charlie Li
…nope, still don't have an exit line.

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r346023 - head/usr.bin/strings

2019-09-03 Thread Charlie Li via svn-src-all
Shawn Webb wrote:
> On Mon, Apr 08, 2019 at 03:35:48AM +, Mariusz Zaborski wrote:
>> Author: oshogbo
>> Date: Mon Apr  8 03:35:47 2019
>> New Revision: 346023
>> URL: https://svnweb.freebsd.org/changeset/base/346023
>>
>> Log:
>>   strings: disable Casper support while building native-xtools
> 
> Why?
> 
Discussed in previous thread. Capsicum is not part of the native-xtools
build, so linking fails. This is regardless of both the host and target
architectures. Example:

--- all_subdir_usr.bin/strings ---
Building
/usr/obj/usr/local/poudriere/jails/aarch64-current/usr/src/amd64.amd64/nxb/arm64.aarch64/usr.bin/strings/strings
--- strings ---
ld: error: unable to find library -lcasper
ld: error: unable to find library -lcap_fileargs
cc: error: linker command failed with exit code 1 (use -v to see invocation)
*** [strings] Error code 1

make[4]: stopped in
/usr/local/poudriere/jails/aarch64-current/usr/src/usr.bin/strings

-- 
Charlie Li
…nope, still don't have an exit line.

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r345875 - in head: contrib/elftoolchain/strings usr.bin/strings

2019-09-03 Thread Charlie Li via svn-src-all
Mariusz Zaborski wrote:
> Log:
>   strings: capsicumize it
>   
>   Reviewed by:cem
>   Discussed with: emaste
>   Differential Revision:  https://reviews.freebsd.org/D18038
> 
> Modified:
>   head/contrib/elftoolchain/strings/strings.c
>   head/usr.bin/strings/Makefile
> 
This change breaks the native-xtools build, because capsicum is not part
of the native-xtools target.

--- all_subdir_usr.bin/strings ---
Building
/usr/obj/usr/local/poudriere/jails/aarch64-current/usr/src/amd64.amd64/nxb/arm64.aarch64/usr.bin/strings/strings
--- strings ---
ld: error: unable to find library -lcasper
ld: error: unable to find library -lcap_fileargs
cc: error: linker command failed with exit code 1 (use -v to see invocation)
*** [strings] Error code 1

make[4]: stopped in
/usr/local/poudriere/jails/aarch64-current/usr/src/usr.bin/strings

-- 
Charlie Li
…nope, still don't have an exit line.

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r346410 - in head: lib/libc/gen share/man/man9 sys/conf sys/libkern sys/sys

2019-04-19 Thread Charlie Li via svn-src-all
O. Hartmann wrote:
> Am Fri, 19 Apr 2019 20:05:48 + (UTC) Conrad Meyer schrieb:
>> Modified: head/lib/libc/gen/Makefile.inc
>> ==
>> --- head/lib/libc/gen/Makefile.inc   Fri Apr 19 19:45:19 2019
>> (r346409)
>> +++ head/lib/libc/gen/Makefile.inc   Fri Apr 19 20:05:47 2019
>> (r346410)
>> @@ -3,6 +3,7 @@
> 
>>  # machine-independent gen sources
>>  .PATH: ${LIBC_SRCTOP}/${LIBC_ARCH}/gen ${LIBC_SRCTOP}/gen
>> +.PATH: ${SRCTOP}/sys/libkern
> 
>>  CONFS=  shells
> 
> 
> After applying r346410 to our source tree, buildworld fails (WITH_META_MODE 
> is set):
> 
> [...]
> Building /usr/obj/usr/src/amd64.amd64/lib/libc/ev_timers.o
> --- inet_ntoa.o ---
> In file included from /usr/src/sys/libkern/inet_ntoa.c:34:
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:172:8: error: 
> unknown type name 'bool'
> extern bool dynamic_kenv;
>^
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:211:5: warning: 
> declaration of
> built-in function 'setjmp' requires inclusion of the header 
> [-Wbuiltin-requires-header] int setjmp(struct _jmp_buf *) 
> __returns_twice; ^
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:212:6: warning: 
> declaration of
> built-in function 'longjmp' requires inclusion of the header 
> [-Wbuiltin-requires-header] voidlongjmp(struct _jmp_buf *, int) __dead2; ^
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:285:6: warning: 
> incompatible
> redeclaration of library function 'log' 
> [-Wincompatible-library-redeclaration] void
> log(int, const char *, ...) __printflike(2, 3); ^
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:285:6: note: 'log' 
> is a builtin with
> type 'double (double)' 
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/systm.h:382:39: error:
> unknown type name 'uintfptr_t'; did you mean 'uintptr_t'? void
> profclock(int cnt, int
> usermode, uintfptr_t pc); ^
> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/sys/_stdint.h:80:22: note: 
> 'uintptr_t' declared
> here typedef __uintptr_t uintptr_t;
> 
> 
> Is there an include missing?
> Beat me to the initial email. With the specific file change quoted
above, looks like inet_*.c are getting pulled from sys/libkern instead
of the proper lib/libc/inet.

-- 
Charlie Li
…nope, still don't have an exit line.

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r346023 - head/usr.bin/strings

2019-04-08 Thread Charlie Li via svn-src-all
Shawn Webb wrote:
> On Mon, Apr 08, 2019 at 03:35:48AM +, Mariusz Zaborski wrote:
>> Author: oshogbo
>> Date: Mon Apr  8 03:35:47 2019
>> New Revision: 346023
>> URL: https://svnweb.freebsd.org/changeset/base/346023
>>
>> Log:
>>   strings: disable Casper support while building native-xtools
> 
> Why?
> 
Discussed in previous thread. Capsicum is not part of the native-xtools
build, so linking fails. This is regardless of both the host and target
architectures. Example:

--- all_subdir_usr.bin/strings ---
Building
/usr/obj/usr/local/poudriere/jails/aarch64-current/usr/src/amd64.amd64/nxb/arm64.aarch64/usr.bin/strings/strings
--- strings ---
ld: error: unable to find library -lcasper
ld: error: unable to find library -lcap_fileargs
cc: error: linker command failed with exit code 1 (use -v to see invocation)
*** [strings] Error code 1

make[4]: stopped in
/usr/local/poudriere/jails/aarch64-current/usr/src/usr.bin/strings

-- 
Charlie Li
…nope, still don't have an exit line.

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r345875 - in head: contrib/elftoolchain/strings usr.bin/strings

2019-04-06 Thread Charlie Li via svn-src-all
Mariusz Zaborski wrote:
> Log:
>   strings: capsicumize it
>   
>   Reviewed by:cem
>   Discussed with: emaste
>   Differential Revision:  https://reviews.freebsd.org/D18038
> 
> Modified:
>   head/contrib/elftoolchain/strings/strings.c
>   head/usr.bin/strings/Makefile
> 
This change breaks the native-xtools build, because capsicum is not part
of the native-xtools target.

--- all_subdir_usr.bin/strings ---
Building
/usr/obj/usr/local/poudriere/jails/aarch64-current/usr/src/amd64.amd64/nxb/arm64.aarch64/usr.bin/strings/strings
--- strings ---
ld: error: unable to find library -lcasper
ld: error: unable to find library -lcap_fileargs
cc: error: linker command failed with exit code 1 (use -v to see invocation)
*** [strings] Error code 1

make[4]: stopped in
/usr/local/poudriere/jails/aarch64-current/usr/src/usr.bin/strings

-- 
Charlie Li
…nope, still don't have an exit line.

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r341825 - in head: . contrib/compiler-rt contrib/compiler-rt/include/sanitizer contrib/compiler-rt/include/xray contrib/compiler-rt/lib/BlocksRuntime contrib/compiler-rt/lib/asan contr

2019-02-18 Thread Charlie Li via svn-src-all
On 18/02/2019 06:49, Dimitry Andric wrote:
> On 18 Feb 2019, at 12:40, Charlie Li via freebsd-toolchain wrote:
>>
>> On 11/12/2018 14:05, Dimitry Andric wrote:
>>>  head/contrib/libc++/include/type_traits
>>>
>> The change to the above named file breaks building any C++ code
>> containing _Float16 with devel/llvm80-8.0.0.r2 (and probably later, but
>> not 8.0.0.r1 or any earlier LLVM) on head (but not
>> projects/clang800-import, obviously). Example error, from our own copy
>> of LLVM libunwind:
>>
>> --- libunwind.o ---
>> In file included from
>> /usr/src/contrib/llvm/projects/libunwind/src/libunwind.cpp:18:
>> In file included from
>> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/c++/v1/new:91:
>> In file included from
>> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/c++/v1/exception:83:
>> /usr/obj/usr/src/amd64.amd64/tmp/usr/include/c++/v1/type_traits:740:56:
>> error: _Float16 is not supported on this target
>> template <>  struct __libcpp_is_floating_point<_Float16>:
>> public true_type {};
>>   ^
>> 1 error generated.
>> *** [libunwind.o] Error code 1
> 
> Hm, which target did you compile for?  I have run multiple universe
> builds, so I thought I had covered all possible targets.  Or are you
> building with gcc?
> 
Building head using xtoolchain-llvm80 on amd64, for amd64. Interestingly
didn't error when cross-building for aarch64.

The projects/clang800-import branch already has the bit in question
reverted, so is a moot point there.

-- 
Charlie Li
Can't think of a witty .sigline today…

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r341825 - in head: . contrib/compiler-rt contrib/compiler-rt/include/sanitizer contrib/compiler-rt/include/xray contrib/compiler-rt/lib/BlocksRuntime contrib/compiler-rt/lib/asan contr

2019-02-18 Thread Charlie Li via svn-src-all
On 11/12/2018 14:05, Dimitry Andric wrote:
>   head/contrib/libc++/include/type_traits
> 
The change to the above named file breaks building any C++ code
containing _Float16 with devel/llvm80-8.0.0.r2 (and probably later, but
not 8.0.0.r1 or any earlier LLVM) on head (but not
projects/clang800-import, obviously). Example error, from our own copy
of LLVM libunwind:

--- libunwind.o ---
In file included from
/usr/src/contrib/llvm/projects/libunwind/src/libunwind.cpp:18:
In file included from
/usr/obj/usr/src/amd64.amd64/tmp/usr/include/c++/v1/new:91:
In file included from
/usr/obj/usr/src/amd64.amd64/tmp/usr/include/c++/v1/exception:83:
/usr/obj/usr/src/amd64.amd64/tmp/usr/include/c++/v1/type_traits:740:56:
error: _Float16 is not supported on this target
template <>  struct __libcpp_is_floating_point<_Float16>:
public true_type {};
   ^
1 error generated.
*** [libunwind.o] Error code 1

Upstream LLVM has already reverted this: https://reviews.llvm.org/D53670

world builds successfully after applying the upstream revert. Ports that
use the clang portion of devel/llvm80 when set in DEFAULT_VERSIONS, ie
gecko@, also build successfully.

-- 
Charlie Li
Can't think of a witty .sigline today…

(This email address is for mailing list use; replace local-part with
vishwin for off-list communication if possible)



signature.asc
Description: OpenPGP digital signature


Re: svn commit: r343111 - in head: cddl/lib/libdtrace contrib/libc++/include contrib/libxo/tests/gettext contrib/libxo/tests/gettext/po/pig_latin contrib/openbsm/libbsm contrib/openbsm/sys/bsm lib/lib

2019-01-16 Thread Charlie Li via svn-src-all
On 17/01/2019 01:35, Kirk McKusick wrote:
> Modified: head/contrib/libc++/include/errno.h
> ==
> --- head/contrib/libc++/include/errno.h   Thu Jan 17 06:34:39 2019
> (r343110)
> +++ head/contrib/libc++/include/errno.h   Thu Jan 17 06:35:45 2019
> (r343111)
> @@ -33,49 +33,72 @@ Macros:
>  
>  #ifdef __cplusplus
>  
> -#if !defined(EOWNERDEAD) || !defined(ENOTRECOVERABLE)
> +#if !defined(EOWNERDEAD) || !defined(ENOTRECOVERABLE) || !defined(EINTEGRITY)
>  
>  #ifdef ELAST
>  
>  static const int __elast1 = ELAST+1;
>  static const int __elast2 = ELAST+2;
> +static const int __elast2 = ELAST+3;
>  
>  #else
>  
>  static const int __elast1 = 104;
>  static const int __elast2 = 105;
> +static const int __elast2 = 106;
>  
>  #endif
>  
> -#ifdef ENOTRECOVERABLE
> +#if !defined(EOWNERDEAD) && !defined(ENOTRECOVERABLE) && !defined(EINTEGRITY)
> +#define ENOTRECOVERABLE __elast1
> +#define EOWNERDEAD __elast2
> +#define EINTEGRITY __elast3
> +#ifdef ELAST
> +#undef ELAST
> +#define ELAST EINTEGRITY
>  
> +#elif !defined(EOWNERDEAD) && !defined(ENOTRECOVERABLE) && 
> defined(EINTEGRITY)
> +#define ENOTRECOVERABLE __elast1
> +#define EOWNERDEAD __elast2
> +#ifdef ELAST
> +#undef ELAST
> +#define ELAST EOWNERDEAD
> +
> +#elif !defined(EOWNERDEAD) && defined(ENOTRECOVERABLE) && 
> !defined(EINTEGRITY)
>  #define EOWNERDEAD __elast1
> +#define EINTEGRITY __elast2
> +#ifdef ELAST
> +#undef ELAST
> +#define ELAST EINTEGRITY
>  
> +#elif !defined(EOWNERDEAD) && defined(ENOTRECOVERABLE) && defined(EINTEGRITY)
> +#define EOWNERDEAD __elast1
>  #ifdef ELAST
>  #undef ELAST
>  #define ELAST EOWNERDEAD
> -#endif
>  
> -#elif defined(EOWNERDEAD)
> +#elif defined(EOWNERDEAD) && !defined(ENOTRECOVERABLE) && 
> !defined(EINTEGRITY)
> +#define ENOTRECOVERABLE __elast1
> +#define EINTEGRITY __elast2
> +#ifdef ELAST
> +#undef ELAST
> +#define ELAST EINTEGRITY
>  
> +#elif defined(EOWNERDEAD) && !defined(ENOTRECOVERABLE) && defined(EINTEGRITY)
>  #define ENOTRECOVERABLE __elast1
>  #ifdef ELAST
>  #undef ELAST
>  #define ELAST ENOTRECOVERABLE
> -#endif
>  
> -#else  // defined(EOWNERDEAD)
> -
> -#define EOWNERDEAD __elast1
> -#define ENOTRECOVERABLE __elast2
> +#elif defined(EOWNERDEAD) && defined(ENOTRECOVERABLE) && !defined(EINTEGRITY)
> +#define EINTEGRITY __elast1
>  #ifdef ELAST
>  #undef ELAST
> -#define ELAST ENOTRECOVERABLE
> -#endif
> +#define ELAST EINTEGRITY
>  
> -#endif  // defined(EOWNERDEAD)
> +#endif // !defined(OWNERDEAD) && !defined(NOTRECOVERABLE) && 
> !defined(INTEGRITY)
>  
> -#endif  // !defined(EOWNERDEAD) || !defined(ENOTRECOVERABLE)
> +#endif // !defined(OWNERDEAD) || !defined(NOTRECOVERABLE) || 
> !defined(INTEGRITY)
>  
>  //  supply errno values likely to be missing, particularly on Windows
>  
> @@ -391,6 +414,10 @@ static const int __elast2 = 105;
>  
>  #ifndef EMLINK
>  #define EMLINK 9979
> +#endif
> +
> +#ifndef EINTEGRITY
> +#define EINTEGRITY 9980
>  #endif
>  
>  #endif // __cplusplus
> 
The build errors out as Clang complains about all of these conditional
directives being unterminated.

--- algorithm.o ---
In file included from /usr/src/contrib/libc++/src/algorithm.cpp:11:
In file included from /usr/src/contrib/libc++/include/random:1646:
In file included from /usr/src/contrib/libc++/include/istream:163:
In file included from /usr/src/contrib/libc++/include/ostream:138:
In file included from /usr/src/contrib/libc++/include/ios:216:
In file included from /usr/src/contrib/libc++/include/__locale:18:
In file included from /usr/src/contrib/libc++/include/mutex:191:
In file included from /usr/src/contrib/libc++/include/__mutex_base:16:
In file included from /usr/src/contrib/libc++/include/system_error:146:
In file included from /usr/src/contrib/libc++/include/__errc:106:
In file included from /usr/src/contrib/libc++/include/cerrno:27:
/usr/src/contrib/libc++/include/errno.h:70:2: error: unterminated
conditional directive
#ifdef ELAST
 ^
/usr/src/contrib/libc++/include/errno.h:63:2: error: unterminated
conditional directive
#ifdef ELAST
 ^
/usr/src/contrib/libc++/include/errno.h:56:2: error: unterminated
conditional directive
#ifdef ELAST
 ^
/usr/src/contrib/libc++/include/errno.h:52:2: error: unterminated
conditional directive
#if !defined(EOWNERDEAD) && !defined(ENOTRECOVERABLE) &&
!defined(EINTEGRITY)
 ^
/usr/src/contrib/libc++/include/errno.h:36:2: error: unterminated
conditional directive
#if !defined(EOWNERDEAD) || !defined(ENOTRECOVERABLE) ||
!defined(EINTEGRITY)
 ^
/usr/src/contrib/libc++/include/errno.h:34:2: error: unterminated
conditional directive
#ifdef __cplusplus
 ^
/usr/src/contrib/libc++/include/errno.h:11:2: error: unterminated
conditional directive
#ifndef _LIBCPP_ERRNO_H
 ^
In file included from /usr/src/contrib/libc++/src/algorithm.cpp:11:
In file included from /usr/src/contrib/libc++/include/random:1646:
In file included from /usr/src/contrib/libc++/include/istream:163:
In file in