NetBSD Problem Report #55367

From www@netbsd.org  Thu Jun 11 08:52:52 2020
Return-Path: <www@netbsd.org>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(Client CN "mail.NetBSD.org", Issuer "mail.NetBSD.org CA" (not verified))
	by mollari.NetBSD.org (Postfix) with ESMTPS id A06001A9219
	for <gnats-bugs@gnats.NetBSD.org>; Thu, 11 Jun 2020 08:52:52 +0000 (UTC)
Message-Id: <20200611085251.AB7821A921E@mollari.NetBSD.org>
Date: Thu, 11 Jun 2020 08:52:51 +0000 (UTC)
From: rares.aioanei@gmail.com
Reply-To: rares.aioanei@gmail.com
To: gnats-bugs@NetBSD.org
Subject: failure to compile amd64 current with clang
X-Send-Pr-Version: www-1.0

>Number:         55367
>Category:       toolchain
>Synopsis:       failure to compile amd64 current with clang
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    toolchain-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Jun 11 08:55:00 +0000 2020
>Last-Modified:  Wed Jun 17 07:44:58 +0000 2020
>Originator:     Rares Aioanei
>Release:        CURRENT
>Organization:
>Environment:
>Description:
This happens since the 9th of June, while cross-compiling on a FreeBSD amd64 box : 

#   compile  drvctl/drvctl.o                                                                                                                                                                  
/stpool/devel/netbsd/current/amd64/usr/src/obj/tooldir.FreeBSD-12.1-RELEASE-p5-amd64/bin/x86_64--netbsd-clang -O2   -fPIE   -std=gnu99     -Wno-sign-compare -Wno-pointer-sign  -Wall -Wstrict
-prototypes -Wmissing-prototypes -Wpointer-arith -Wno-sign-compare    -Wreturn-type -Wswitch -Wshadow -Wcast-qual -Wwrite-strings -Wextra -Wno-unused-parameter -Wno-sign-compare -Wold-style-
definition -Wsign-compare -Wformat=2   -Wpointer-sign -Wmissing-noreturn  -Werror     --sysroot=/stpool/devel/netbsd/current/amd64/usr/src/obj/destdir.amd64  -c    /stpool/devel/netbsd/curre
nt/amd64/usr/src/sbin/drvctl/drvctl.c                                                                                                                                                         
/stpool/devel/netbsd/current/amd64/usr/src/obj/tooldir.FreeBSD-12.1-RELEASE-p5-amd64/bin/nbctfconvert -g -L VERSION drvctl.o                                                                  
#      link  drvctl/drvctl                                                                                                                                                                    
/stpool/devel/netbsd/current/amd64/usr/src/obj/tooldir.FreeBSD-12.1-RELEASE-p5-amd64/bin/x86_64--netbsd-clang    --sysroot=/stpool/devel/netbsd/current/amd64/usr/src/obj/destdir.amd64   -pie
  -Wl,--fatal-warnings -Wl,--warn-shared-textrel -Wl,-z,relro     -o drvctl  drvctl.o  -Wl,-dynamic-linker=/libexec/ld.elf_so -Wl,-rpath,/lib  -L=/lib -lprop                                 
/stpool/devel/netbsd/current/amd64/usr/src/obj/tooldir.FreeBSD-12.1-RELEASE-p5-amd64/bin/x86_64--netbsd-ld: drvctl.o: in function `main':                                                     
drvctl.c:(.text+0x233): warning: this program uses prop_string_create_cstring(); all functions supporting mutable prop_strings are deprecated.                                                
/stpool/devel/netbsd/current/amd64/usr/src/obj/tooldir.FreeBSD-12.1-RELEASE-p5-amd64/bin/x86_64--netbsd-ld: drvctl.c:(.text+0x20e): warning: this program uses prop_string_create_cstring_noco
py(), which is deprecated; use prop_string_create_nocopy() instead.                                                                                                                           
/stpool/devel/netbsd/current/amd64/usr/src/obj/tooldir.FreeBSD-12.1-RELEASE-p5-amd64/bin/x86_64--netbsd-ld: drvctl.o: in function `display_object':                                           
drvctl.c:(.text+0x953): warning: this program uses prop_string_cstring_nocopy(), which is deprecated; use prop_string_value() instead.
/stpool/devel/netbsd/current/amd64/usr/src/obj/tooldir.FreeBSD-12.1-RELEASE-p5-amd64/bin/x86_64--netbsd-ld: drvctl.o: in function `main':                                                     drvctl.c:(.text+0x2b4): warning: this program uses prop_number_integer_value(), which is deprecated; use prop_number_signed_value() instead.                                                  x86_64--netbsd-clang: error: linker command failed with exit code 1 (use -v to see invocation)                                                                                                

*** Failed target:  drvctl
*** Failed command: /stpool/devel/netbsd/current/amd64/usr/src/obj/tooldir.FreeBSD-12.1-RELEASE-p5-amd64/bin/x86_64--netbsd-clang --sysroot=/stpool/devel/netbsd/current/amd64/usr/src/obj/des
tdir.amd64 -pie -Wl,--fatal-warnings -Wl,--warn-shared-textrel -Wl,-z,relro -o drvctl drvctl.o -Wl,-dynamic-linker=/libexec/ld.elf_so -Wl,-rpath,/lib -L=/lib -lprop 
*** Error code 1

>How-To-Repeat:

>Fix:

>Release-Note:

>Audit-Trail:
From: Sevan Janiyan <venture37@geeklan.co.uk>
To: "gnats-bugs@netbsd.org" <gnats-bugs@NetBSD.org>
Cc: 
Subject: Re: port-amd64/55367: failure to compile amd64 current with clang
Date: Thu, 11 Jun 2020 13:12:35 +0100

 This is due to proplib update and drvctl using the now depracted API. It
 results in warnings which the compiler is set to treat as errors. Keep
 an eye out for update commits as thorpej is in the process of updating
 things.


 Sevan

Responsible-Changed-From-To: port-amd64-maintainer->toolchain-maintainer
Responsible-Changed-By: thorpej@NetBSD.org
Responsible-Changed-When: Thu, 11 Jun 2020 13:33:53 +0000
Responsible-Changed-Why:
This is a clang problem -- it should not consider those warning messages
from the linker to be fatal.


From: Jason Thorpe <thorpej@me.com>
To: gnats-bugs@netbsd.org
Cc: port-amd64-maintainer@netbsd.org,
 gnats-admin@netbsd.org,
 netbsd-bugs@netbsd.org,
 rares.aioanei@gmail.com
Subject: Re: port-amd64/55367: failure to compile amd64 current with clang
Date: Thu, 11 Jun 2020 06:38:43 -0700

 > On Jun 11, 2020, at 5:15 AM, Sevan Janiyan <venture37@geeklan.co.uk> =
 wrote:
 >=20
 > The following reply was made to PR port-amd64/55367; it has been noted =
 by GNATS.
 >=20
 > From: Sevan Janiyan <venture37@geeklan.co.uk>
 > To: "gnats-bugs@netbsd.org" <gnats-bugs@NetBSD.org>
 > Cc:=20
 > Subject: Re: port-amd64/55367: failure to compile amd64 current with =
 clang
 > Date: Thu, 11 Jun 2020 13:12:35 +0100
 >=20
 > This is due to proplib update and drvctl using the now depracted API. =
 It
 > results in warnings which the compiler is set to treat as errors. Keep
 > an eye out for update commits as thorpej is in the process of updating
 > things.

 This is a clang problem.  GCC does not treat these linker warnings as =
 fatal, nor should it.

 -- thorpej

From: Joerg Sonnenberger <joerg@bec.de>
To: Jason Thorpe <thorpej@me.com>
Cc: gnats-bugs@netbsd.org, port-amd64-maintainer@netbsd.org,
	gnats-admin@netbsd.org, netbsd-bugs@netbsd.org,
	rares.aioanei@gmail.com
Subject: Re: port-amd64/55367: failure to compile amd64 current with clang
Date: Thu, 11 Jun 2020 22:07:42 +0200

 On Thu, Jun 11, 2020 at 06:38:43AM -0700, Jason Thorpe wrote:
 > 
 > > On Jun 11, 2020, at 5:15 AM, Sevan Janiyan <venture37@geeklan.co.uk> wrote:
 > > 
 > > The following reply was made to PR port-amd64/55367; it has been noted by GNATS.
 > > 
 > > From: Sevan Janiyan <venture37@geeklan.co.uk>
 > > To: "gnats-bugs@netbsd.org" <gnats-bugs@NetBSD.org>
 > > Cc: 
 > > Subject: Re: port-amd64/55367: failure to compile amd64 current with clang
 > > Date: Thu, 11 Jun 2020 13:12:35 +0100
 > > 
 > > This is due to proplib update and drvctl using the now depracted API. It
 > > results in warnings which the compiler is set to treat as errors. Keep
 > > an eye out for update commits as thorpej is in the process of updating
 > > things.
 > 
 > This is a clang problem.  GCC does not treat these linker warnings as fatal, nor should it.

 Yes, it should. We don't with GCC for various hysterical reasons, but
 they should be an error nevertheless.

 Joerg

From: matthew green <mrg@eterna.com.au>
To: Jason Thorpe <thorpej@me.com>
Cc: port-amd64-maintainer@netbsd.org, gnats-admin@netbsd.org,
    netbsd-bugs@netbsd.org, rares.aioanei@gmail.com,
    gnats-bugs@netbsd.org
Subject: re: port-amd64/55367: failure to compile amd64 current with clang
Date: Fri, 12 Jun 2020 16:31:43 +1000

 > This is a clang problem.  GCC does not treat these linker warnings as
 > fatal, nor should it.

 actually, some GCC platforms enabled it by default.

 can you turn these warnings off for now?  most of us don't
 want to see them, they break clang, and they make the build
 output a lot more verbose.

 having these enabled in your own tree while converting is
 clearly useful, but the rest of us shouldn't have to deal
 with it -- it's several pages of output for my kernel link
 that, with a single .c file change, would otherwise fit in
 a single screen easily.

 thanks.


 .mrg.

From: Rares Aioanei <rares.aioanei@gmail.com>
To: gnats-bugs@netbsd.org
Cc: toolchain-maintainer@netbsd.org, gnats-admin@netbsd.org, 
	netbsd-bugs@netbsd.org
Subject: Re: port-amd64/55367: failure to compile amd64 current with clang
Date: Sat, 13 Jun 2020 13:07:37 +0300

 On Fri, Jun 12, 2020 at 9:35 AM matthew green <mrg@eterna.com.au> wrote:
 >
 [...]

 I can confirm that, after updating the tree, the compilation process
 finishes successfully.

Responsible-Changed-From-To: toolchain-maintainer->toolchain-manager
Responsible-Changed-By: spz@NetBSD.org
Responsible-Changed-When: Wed, 17 Jun 2020 07:44:58 +0000
Responsible-Changed-Why:
pick a "responsible" that exists
.


>Unformatted:

Home	PR Database Search