From owner-freebsd-ports-bugs@FreeBSD.ORG Wed Jan 29 16:30:01 2014 Return-Path: Delivered-To: freebsd-ports-bugs@smarthost.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id B32EF3D0 for ; Wed, 29 Jan 2014 16:30:01 +0000 (UTC) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 7C0A51CC8 for ; Wed, 29 Jan 2014 16:30:01 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.7/8.14.7) with ESMTP id s0TGU1mX096314 for ; Wed, 29 Jan 2014 16:30:01 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.7/8.14.7/Submit) id s0TGU1VA096313; Wed, 29 Jan 2014 16:30:01 GMT (envelope-from gnats) Date: Wed, 29 Jan 2014 16:30:01 GMT Message-Id: <201401291630.s0TGU1VA096313@freefall.freebsd.org> To: freebsd-ports-bugs@FreeBSD.org Cc: From: Marcin Cieslak Subject: Re: ports/186219: net-mgmt/libsmi - Clang / GCC issue X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.17 Precedence: list Reply-To: Marcin Cieslak List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 29 Jan 2014 16:30:01 -0000 The following reply was made to PR ports/186219; it has been noted by GNATS. From: Marcin Cieslak To: bug-followup@FreeBSD.org Cc: Subject: Re: ports/186219: net-mgmt/libsmi - Clang / GCC issue Date: Wed, 29 Jan 2014 16:21:00 +0000 On Wed, 29 Jan 2014, Edwin Groothuis wrote: > Maintainer of net-mgmt/libsmi, > > Please note that PR ports/186219 has just been submitted. > > If it contains a patch for an upgrade, an enhancement or a bug fix > you agree on, reply to this email stating that you approve the patch > and a committer will take care of it. Thanks, there is a fix in the SVN already, ===> Generating patch ===> Viewing diff with more diff -ruN --exclude=CVS /usr/ports/net-mgmt/libsmi.orig/Makefile /usr/ports/net-mgmt/libsmi/Makefile --- /usr/ports/net-mgmt/libsmi.orig/Makefile 2014-01-29 17:15:07.000000000 +0100 +++ /usr/ports/net-mgmt/libsmi/Makefile 2014-01-29 17:19:09.000000000 +0100 @@ -3,6 +3,7 @@ PORTNAME= libsmi PORTVERSION= 0.4.8 +PORTREVISION= 1 CATEGORIES= net-mgmt MASTER_SITES= ftp://ftp.ibr.cs.tu-bs.de/pub/local/libsmi/ \ LOCAL/bms diff -ruN --exclude=CVS /usr/ports/net-mgmt/libsmi.orig/files/patch-cppcompat /usr/ports/net-mgmt/libsmi/files/patch-cppcompat --- /usr/ports/net-mgmt/libsmi.orig/files/patch-cppcompat 1970-01-01 01:00:00.000000000 +0100 +++ /usr/ports/net-mgmt/libsmi/files/patch-cppcompat 2014-01-29 17:15:07.000000000 +0100 @@ -0,0 +1,32 @@ +--- lib/smi.h.in.orig 2008-04-18 14:56:31.000000000 +0200 ++++ lib/smi.h.in 2014-01-29 17:11:40.000000000 +0100 +@@ -312,9 +312,10 @@ + + /* SmiElement -- an item in a list (row index column, notification object) */ + typedef struct SmiElement { +-#ifndef __GNUC__ +- char dummy; /* many compilers are unhappy with empty structures. */ +-#endif ++ char dummy; /* C99 does not allow empty structs; some compilers ++ accept them (gcc) but there are size differences ++ between C and C++. So for portability reasons, ++ we have this unused struct member. */ + /* no visible attributes */ + } SmiElement; + +--- lib/smi.h.orig 2008-04-18 14:58:06.000000000 +0200 ++++ lib/smi.h 2014-01-29 17:11:59.000000000 +0100 +@@ -312,9 +312,10 @@ + + /* SmiElement -- an item in a list (row index column, notification object) */ + typedef struct SmiElement { +-#ifndef __GNUC__ +- char dummy; /* many compilers are unhappy with empty structures. */ +-#endif ++ char dummy; /* C99 does not allow empty structs; some compilers ++ accept them (gcc) but there are size differences ++ between C and C++. So for portability reasons, ++ we have this unused struct member. */ + /* no visible attributes */ + } SmiElement; + ===> Done