From owner-freebsd-ports-bugs@FreeBSD.ORG Thu Dec 6 16:20:01 2007 Return-Path: Delivered-To: freebsd-ports-bugs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id B326E16A469 for ; Thu, 6 Dec 2007 16:20:01 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 92D4D13C468 for ; Thu, 6 Dec 2007 16:20:01 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.14.2/8.14.2) with ESMTP id lB6GK1Jr028364 for ; Thu, 6 Dec 2007 16:20:01 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.2/8.14.1/Submit) id lB6GK1GH028363; Thu, 6 Dec 2007 16:20:01 GMT (envelope-from gnats) Resent-Date: Thu, 6 Dec 2007 16:20:01 GMT Resent-Message-Id: <200712061620.lB6GK1GH028363@freefall.freebsd.org> Resent-From: FreeBSD-gnats-submit@FreeBSD.org (GNATS Filer) Resent-To: freebsd-ports-bugs@FreeBSD.org Resent-Reply-To: FreeBSD-gnats-submit@FreeBSD.org, "Pietro Cerutti" Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 7639116A421 for ; Thu, 6 Dec 2007 16:19:28 +0000 (UTC) (envelope-from gahr@gahr.ch) Received: from mr1.bfh.ch (mr1.bfh.ch [147.87.250.50]) by mx1.freebsd.org (Postfix) with ESMTP id B7DEF13C4E5 for ; Thu, 6 Dec 2007 16:19:27 +0000 (UTC) (envelope-from gahr@gahr.ch) Received: from square.bfh.ch (square.bfh.ch [147.87.224.33]) by mr1.bfh.ch (8.13.8/8.13.8) with ESMTP id lB6GJOMe008617 for ; Thu, 6 Dec 2007 17:19:24 +0100 Received: from gahrtop.localhost ([147.87.108.162]) by square.bfh.ch with Microsoft SMTPSVC(6.0.3790.3959); Thu, 6 Dec 2007 17:19:24 +0100 Message-Id: <1196957892.78251@gahrtop.localhost> Date: Thu, 6 Dec 2007 17:18:12 +0100 From: "Pietro Cerutti" To: "FreeBSD gnats submit" X-Send-Pr-Version: gtk-send-pr 0.4.8 Cc: Subject: ports/118466: [patch] audio/gsi unbreak fix build with GCC 4.2 + adopt maintainship X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 06 Dec 2007 16:20:01 -0000 >Number: 118466 >Category: ports >Synopsis: [patch] audio/gsi unbreak fix build with GCC 4.2 + adopt maintainship >Confidential: no >Severity: serious >Priority: medium >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: sw-bug >Submitter-Id: current-users >Arrival-Date: Thu Dec 06 16:20:01 UTC 2007 >Closed-Date: >Last-Modified: >Originator: Pietro Cerutti >Release: FreeBSD 8.0-CURRENT i386 >Organization: >Environment: System: FreeBSD 8.0-CURRENT #15: Thu Dec 6 12:47:48 CET 2007 root@gahrtop.localhost:/usr/obj/usr/src/sys/MSI1034 >Description: The patch below: - fixes a "cast where lvalue is required" error with GCC 4.2 - sets maintainship to me Tested on: 8.0-CURRENT i386 (little endian *) 7.0-BETA3 sparc64 (big endian *) 6.2-RELEASE-p6 i386 (little endian *) * endianess is specified because of the patch dealing with endianess-aware code >How-To-Repeat: cd /usr/ports/audio/gsi && make >Fix: --- _gsi.diff begins here --- --- Makefile.orig 2007-12-06 16:41:53.000000000 +0100 +++ Makefile 2007-12-06 17:00:01.000000000 +0100 @@ -12,7 +12,7 @@ MASTER_SITES= ${MASTER_SITE_SOURCEFORGE} MASTER_SITE_SUBDIR= ${PORTNAME} -MAINTAINER= ports@FreeBSD.org +MAINTAINER= gahr@gahr.ch COMMENT= General sound interface USE_BZIP2= yes @@ -28,24 +28,18 @@ MAN7= gsi_overview.7 MAN8= gsi_server.8 -.include - -.if ${OSVERSION} >= 700042 -BROKEN= Broken with gcc 4.2 -.endif - post-extract: @${RM} -f ${WRKSRC}/lib/include/gsi/gsisynth.h post-patch: -.for file in lib/device/gsimixer/api/mixer_fbsd.c \ +.for f in lib/device/gsimixer/api/mixer_fbsd.c \ lib/device/gsipcm/api/pcm_fbsd.c @${REINPLACE_CMD} -e \ - 's|machine/soundcard.h|sys/soundcard.h|g' ${WRKSRC}/${file} + 's|machine/soundcard.h|sys/soundcard.h|g' ${WRKSRC}/${f} .endfor -.for file in server/FINISH_host_allow.c server/client.c server/conf.c +.for f in server/FINISH_host_allow.c server/client.c server/conf.c @${REINPLACE_CMD} -e \ - 's|"/etc/|"${PREFIX}/etc/|g' ${WRKSRC}/${file} + 's|"/etc/|"${PREFIX}/etc/|g' ${WRKSRC}/${f} .endfor do-configure: @@ -63,4 +57,4 @@ @${ECHO_MSG} @${CAT} ${PKGMESSAGE} -.include +.include --- /dev/null 2007-12-06 17:04:51.000000000 +0100 +++ files/patch-lib-gsif-sample_convert_block.c 2007-12-06 17:05:24.000000000 +0100 @@ -0,0 +1,44 @@ +--- lib/gsif/sample/convert_block.c.orig 2001-01-30 13:28:09.000000000 +0100 ++++ lib/gsif/sample/convert_block.c 2007-12-06 17:04:45.000000000 +0100 +@@ -122,9 +122,9 @@ + break; + case GSI_16BIT: + #ifdef SYSCONFIG_LITTLE_ENDIAN_MACHINE +- input_val = (int) (int16) ( ((*input++) << 8) | (*input++) ); ++ input_val = (int) (int16) ( (*input++ << 8) | *input++ ); + #else +- input_val = *((int16 *)input)++; ++ input_val = *input++; + #endif + break; + case GSI_8BIT_SIGNED: +@@ -132,9 +132,9 @@ + break; + case GSI_16BIT_LE: + #ifdef SYSCONFIG_LITTLE_ENDIAN_MACHINE +- input_val = *((int16 *)input)++; ++ input_val = *input++; + #else +- input_val = (int) (int16) (*input++ | ((*input++) << 8) ); ++ input_val = (int) (int16) (*(input++) | ((*input++) << 8) ); + #endif + break; + case GSI_24BIT: +@@ -170,7 +170,7 @@ + *output++ = (input_val >> 8) & 255; + *output++ = (input_val & 255); + #else +- *((int16 *) output)++ = input_val; ++ *output++ = input_val; + #endif + break; + case GSI_8BIT_SIGNED: +@@ -178,7 +178,7 @@ + break; + case GSI_16BIT_LE: + #ifdef SYSCONFIG_LITTLE_ENDIAN_MACHINE +- *((int16 *) output)++ = input_val; ++ *output++ = input_val; + #else + *output++ = (input_val >> 8) & 255; + *output++ = (input_val & 255); --- _gsi.diff ends here --- >Release-Note: >Audit-Trail: >Unformatted: