NetBSD Problem Report #46017

From martin@aprisoft.de  Mon Feb 13 15:30:16 2012
Return-Path: <martin@aprisoft.de>
Received: from mail.netbsd.org (mail.netbsd.org [149.20.53.66])
	by www.NetBSD.org (Postfix) with ESMTP id D152563E093
	for <gnats-bugs@gnats.NetBSD.org>; Mon, 13 Feb 2012 15:30:15 +0000 (UTC)
Message-Id: <20120213153007.EF135AF581D@emmas.aprisoft.de>
Date: Mon, 13 Feb 2012 16:30:07 +0100 (CET)
From: martin@NetBSD.org
Reply-To: martin@NetBSD.org
To: gnats-bugs@gnats.NetBSD.org
Subject: pango build weiredness
X-Send-Pr-Version: 3.95

>Number:         46017
>Category:       pkg
>Synopsis:       building pango on sparc64 emits and ignores serious errors
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    drochner
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Mon Feb 13 15:35:00 +0000 2012
>Closed-Date:    Wed Feb 15 16:43:12 +0000 2012
>Last-Modified:  Wed Feb 15 16:45:01 +0000 2012
>Originator:     Martin Husemann
>Release:        NetBSD 5.99.65
>Organization:
The NetBSD Foundation, Inc.
>Environment:
System: NetBSD after-hours.aprisoft.de 5.99.65 NetBSD 5.99.65 (MODULAR) #140: Mon Feb 13 07:52:19 CET 2012 martin@after-hours.aprisoft.de:/usr/src/sys/arch/sparc64/compile/MODULAR sparc64
Architecture: sparc64
Machine: sparc64
>Description:

When building pango on a -current sparc64 machine I noticed lots of messages
like this:

  GISCAN GLib-2.0.gir
/usr/include/sparc/mcontext.h:168: syntax error, unexpected identifier, expecting ',' or ';' in ' __asm volatile("mov %%g7, %0" : "=r" (__tmp));' at 'o'
g-ir-scanner: GLib: warning: 720 warnings suppressed (use --warn-all to see them)


>How-To-Repeat:

Just build pango-1.29.4nb2 and closely watch the log (the build does not
stop)

>Fix:
n/a

>Release-Note:

>Audit-Trail:
From: Martin Husemann <martin@duskware.de>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/46017: strange error messages
Date: Mon, 13 Feb 2012 17:31:56 +0100

 Ooops, it was not pango being compiled but devel/gobject-introspection.
 GISCAN seems to be a python script and I'm lost there ;-)

 I checked the preprocessed GLib-2.0.i file (created with CFLAGS=-save-temps)
 and it is ok:

 # 163 "/usr/include/sparc/mcontext.h" 3 4
 static __inline void *
 __lwp_getprivate_fast(void)
 {
  register void *__tmp;

  __asm volatile("mov %%g7, %0" : "=r" (__tmp));

  return __tmp;
 }
 # 7 "/usr/include/machine/mcontext.h" 2 3 4

 Bug in the python code, I assume.

 Martin

Responsible-Changed-From-To: pkg-manager->drochner
Responsible-Changed-By: obache@NetBSD.org
Responsible-Changed-When: Tue, 14 Feb 2012 11:26:33 +0000
Responsible-Changed-Why:
drochner@ enabled introspection support.


From: "OBATA Akio" <obache@netbsd.org>
To: gnats-bugs@netbsd.org
Cc: 
Subject: Re: pkg/46017: pango build weiredness
Date: Wed, 15 Feb 2012 22:41:33 +0900

 On Tue, 14 Feb 2012 00:35:00 +0900, <martin@netbsd.org> wrote:

 >> Description:
 >
 > When building pango on a -current sparc64 machine I noticed lots of messages
 > like this:
 >
 >   GISCAN GLib-2.0.gir
 > /usr/include/sparc/mcontext.h:168: syntax error, unexpected identifier, expecting ',' or ';' in ' __asm volatile("mov %%g7, %0" : "=r" (__tmp));' at 'o'
 > g-ir-scanner: GLib: warning: 720 warnings suppressed (use --warn-all to see them)

 In giscanner/scannerlexer.l line 137 of gobject-introspection-1.30.0,  "__asm" will be skipped, but "__asm volatile" not.

From: Martin Husemann <martin@duskware.de>
To: gnats-bugs@NetBSD.org
Cc: drochner@NetBSD.org
Subject: Re: pkg/46017: pango build weiredness
Date: Wed, 15 Feb 2012 16:14:01 +0100

 Ah great, thanks for pointing it out - lex I can do. I'm testing a patch.
 Should I just commit it if it works?

 Martin

From: Matthias Drochner <M.Drochner@fz-juelich.de>
To: Martin Husemann <martin@duskware.de>
Cc: <gnats-bugs@NetBSD.org>
Subject: Re: pkg/46017: pango build weiredness 
Date: Wed, 15 Feb 2012 16:59:20 +0100

 obache@netbsd.org said:
 >  In giscanner/scannerlexer.l line 137 of gobject-introspection-1.30.0,
 >  "__asm" will be skipped, but "__asm volatile" not.

 good catch!

 martin@duskware.de said:
 > I'm testing a patch.

 something like a line:
 "__asm"[\t\f\v\r ]+"volatile" { if (!parse_ignored_macro()) REJECT; }
 ? (untested, the whitespace pattern is borrowed from some lines above)

 > Should I just commit it if it works?

 Yes, please.

 best regards
 Matthias



 ---------------------------------------------------------------------------=
 ----
 ---------------------------------------------------------------------------=
 ----
 Forschungszentrum Juelich GmbH
 52425 Juelich
 Sitz der Gesellschaft: Juelich
 Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
 Vorsitzender des Aufsichtsrats: MinDir Dr. Karl Eugen Huthmacher
 Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
 Karsten Beneke (stellv. Vorsitzender), Prof. Dr.-Ing. Harald Bolt,
 Prof. Dr. Sebastian M. Schmidt
 ---------------------------------------------------------------------------=
 ----
 ---------------------------------------------------------------------------=
 ----

 Kennen Sie schon unsere app? http://www.fz-juelich.de/app

From: Martin Husemann <martin@duskware.de>
To: Matthias Drochner <M.Drochner@fz-juelich.de>
Cc: gnats-bugs@NetBSD.org
Subject: Re: pkg/46017: pango build weiredness
Date: Wed, 15 Feb 2012 17:01:08 +0100

 On Wed, Feb 15, 2012 at 04:59:20PM +0100, Matthias Drochner wrote:
 > something like a line:
 > "__asm"[\t\f\v\r ]+"volatile" { if (!parse_ignored_macro()) REJECT; }
 > ? (untested, the whitespace pattern is borrowed from some lines above)

 Yes, and another identical for __asm__. Seems to work so far.

 Martin

State-Changed-From-To: open->closed
State-Changed-By: martin@NetBSD.org
State-Changed-When: Wed, 15 Feb 2012 16:43:12 +0000
State-Changed-Why:
fixed


From: "Martin Husemann" <martin@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/46017 CVS commit: pkgsrc/devel/gobject-introspection
Date: Wed, 15 Feb 2012 16:42:36 +0000

 Module Name:	pkgsrc
 Committed By:	martin
 Date:		Wed Feb 15 16:42:36 UTC 2012

 Modified Files:
 	pkgsrc/devel/gobject-introspection: distinfo
 Added Files:
 	pkgsrc/devel/gobject-introspection/patches:
 	    patch-giscanner_scannerlexer.l

 Log Message:
 Make giscanner ignore the "volatile" variants of __asm{,__} as well.
 Found by OBATA Akio, fixes PR pkg/46017.


 To generate a diff of this commit:
 cvs rdiff -u -r1.7 -r1.8 pkgsrc/devel/gobject-introspection/distinfo
 cvs rdiff -u -r0 -r1.1 \
     pkgsrc/devel/gobject-introspection/patches/patch-giscanner_scannerlexer.l

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

>Unformatted:

Home	PR Database Search