NetBSD Problem Report #48230

From www@NetBSD.org  Sun Sep 22 10:34:22 2013
Return-Path: <www@NetBSD.org>
Received: from mail.netbsd.org (mail.netbsd.org [149.20.53.66])
	(using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))
	(Client CN "mail.NetBSD.org", Issuer "Postmaster NetBSD.org" (verified OK))
	by mollari.NetBSD.org (Postfix) with ESMTPS id 8579972379
	for <gnats-bugs@gnats.NetBSD.org>; Sun, 22 Sep 2013 10:34:22 +0000 (UTC)
Message-Id: <20130922103421.40FBB725EB@mollari.NetBSD.org>
Date: Sun, 22 Sep 2013 10:34:21 +0000 (UTC)
From: richard@netbsd.org
Reply-To: richard@netbsd.org
To: gnats-bugs@NetBSD.org
Subject: gnome-vfs with gssapi option passes native krb5-config when pkgsrc version should be used
X-Send-Pr-Version: www-1.0

>Number:         48230
>Category:       pkg
>Synopsis:       gnome-vfs with gssapi option passes native krb5-config when pkgsrc version should be used
>Confidential:   no
>Severity:       critical
>Priority:       medium
>Responsible:    solaris-pkg-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sun Sep 22 10:35:00 +0000 2013
>Last-Modified:  Wed Sep 25 16:35:00 +0000 2013
>Originator:     Richard PALO
>Release:        pkgsrc trunk
>Organization:
>Environment:
SunOS devzoneX 5.11 illumos-gate i86pc i386 i86pc

>Description:
somewhat in relation to pkg/48228, building gnome-gvs with option 'gssapi' fails on solaris if pkgsrc mit-krb5, for example, is installed because the solaris supplied krb5-config doesn't [currently] support the gssapi option.

The options seems to presume that "native" is intended.

The attached patch at least allows the build to complete..

NB: There is probably additional work to support current native solaris krb5 gssapi support to get over the incorrect assumption that krb5-config always supports a gssapi option.
>How-To-Repeat:

>Fix:
richard@devzoneX:~/src/pkgsrc/sysutils/gnome-vfs$ git diff .
diff --git a/sysutils/gnome-vfs/options.mk b/sysutils/gnome-vfs/options.mk
index 43fd232..833e46c 100644
--- a/sysutils/gnome-vfs/options.mk
+++ b/sysutils/gnome-vfs/options.mk
@@ -23,7 +23,9 @@ CONFIGURE_ARGS+=      --disable-fam

 .if !empty(PKG_OPTIONS:Mgssapi)
 .include "../../mk/krb5.buildlink3.mk"
+.      if !empty(USE_BUILTIN.${KRB5_TYPE}:M[yY][eE][sS])
 CONFIGURE_ENV+=                KRB5_CONFIG=${SH_KRB5_CONFIG}
+.      endif
 .else
 CONFIGURE_ENV+=                ac_cv_path_KRB5_CONFIG=none
 .endif

>Release-Note:

>Audit-Trail:

Responsible-Changed-From-To: pkg-manager->solaris-pkg-people
Responsible-Changed-By: hauke@NetBSD.org
Responsible-Changed-When: Sun, 22 Sep 2013 15:56:23 +0000
Responsible-Changed-Why:
A Solaris issue.


From: Richard PALO <richard.palo@baou.fr>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/48230: gnome-vfs with gssapi option passes native krb5-config
 when pkgsrc version should be used
Date: Wed, 25 Sep 2013 10:26:59 +0200

 This is a multi-part message in MIME format.
 --------------000009000203000200030305
 Content-Type: text/plain; charset=ISO-8859-1; format=flowed
 Content-Transfer-Encoding: 8bit

 Le 22/09/13 12:35, richard@netbsd.org a écrit :
 >> Number:         48230
 >> Category:       pkg
 >> Synopsis:       gnome-vfs with gssapi option passes native krb5-config when pkgsrc version should be used
 >> Confidential:   no
 >> Severity:       critical
 >> Priority:       medium
 >> Responsible:    pkg-manager
 >> State:          open
 >> Class:          sw-bug
 >> Submitter-Id:   net
 >> Arrival-Date:   Sun Sep 22 10:35:00 +0000 2013
 >> Originator:     Richard PALO
 >> Release:        pkgsrc trunk
 >> Organization:
 >> Environment:
 > SunOS devzoneX 5.11 illumos-gate i86pc i386 i86pc
 >
 >> Description:
 > somewhat in relation to pkg/48228, building gnome-gvs with option 'gssapi' fails on solaris if pkgsrc mit-krb5, for example, is installed because the solaris supplied krb5-config doesn't [currently] support the gssapi option.
 >
 > The options seems to presume that "native" is intended.
 >
 > The attached patch at least allows the build to complete..
 >
 > NB: There is probably additional work to support current native solaris krb5 gssapi support to get over the incorrect assumption that krb5-config always supports a gssapi option.
 >> How-To-Repeat:
 >
 >> Fix:
 > richard@devzoneX:~/src/pkgsrc/sysutils/gnome-vfs$ git diff .
 > diff --git a/sysutils/gnome-vfs/options.mk b/sysutils/gnome-vfs/options.mk
 > index 43fd232..833e46c 100644
 > --- a/sysutils/gnome-vfs/options.mk
 > +++ b/sysutils/gnome-vfs/options.mk
 > @@ -23,7 +23,9 @@ CONFIGURE_ARGS+=      --disable-fam
 >
 >   .if !empty(PKG_OPTIONS:Mgssapi)
 >   .include "../../mk/krb5.buildlink3.mk"
 > +.      if !empty(USE_BUILTIN.${KRB5_TYPE}:M[yY][eE][sS])
 >   CONFIGURE_ENV+=                KRB5_CONFIG=${SH_KRB5_CONFIG}
 > +.      endif
 >   .else
 >   CONFIGURE_ENV+=                ac_cv_path_KRB5_CONFIG=none
 >   .endif
 >
 >

 attached patch seems to fix krb5-config problem on solaris when native 
 mit-krb5 is used.

 --------------000009000203000200030305
 Content-Type: text/plain; charset=UTF-8;
  name="gvfs.patch"
 Content-Transfer-Encoding: 7bit
 Content-Disposition: attachment;
  filename="gvfs.patch"

 Index: distinfo
 ===================================================================
 RCS file: /cvsroot/pkgsrc/sysutils/gnome-vfs/distinfo,v
 retrieving revision 1.34
 diff -u -r1.34 distinfo
 --- distinfo	19 Apr 2013 20:02:22 -0000	1.34
 +++ distinfo	25 Sep 2013 08:07:40 -0000
 @@ -5,7 +5,7 @@
  Size (gnome-vfs-2.24.4.tar.bz2) = 1890370 bytes
  SHA1 (patch-aa) = a5a3ac6be3e2a888eaaab108807d2aff651e62db
  SHA1 (patch-ab) = 6320180870f234a150dfe40f8f990deec7a2a0b5
 -SHA1 (patch-ae) = 2e46f1311674265d2054c7209aa057d8da2aca67
 +SHA1 (patch-ae) = bc6fe6afd2484acb9f1b31469c18f9a3c4ac7d70
  SHA1 (patch-af) = 6724c4a43d0d9525f5361ba7380c91adbf656e56
  SHA1 (patch-ai) = 6afefb35bfe3fc37e84048b95ccc272a01542ec2
  SHA1 (patch-am) = 4e037da71db0d3aae236475a0f2ead12738f35f9
 Index: options.mk
 ===================================================================
 RCS file: /cvsroot/pkgsrc/sysutils/gnome-vfs/options.mk,v
 retrieving revision 1.5
 diff -u -r1.5 options.mk
 --- options.mk	12 Jun 2012 15:46:03 -0000	1.5
 +++ options.mk	25 Sep 2013 08:07:40 -0000
 @@ -23,7 +23,9 @@

  .if !empty(PKG_OPTIONS:Mgssapi)
  .include "../../mk/krb5.buildlink3.mk"
 +.	if !empty(USE_BUILTIN.${KRB5_TYPE}:M[yY][eE][sS])
  CONFIGURE_ENV+=		KRB5_CONFIG=${SH_KRB5_CONFIG}
 +.	endif
  .else
  CONFIGURE_ENV+=		ac_cv_path_KRB5_CONFIG=none
  .endif
 Index: patches/patch-ae
 ===================================================================
 RCS file: /cvsroot/pkgsrc/sysutils/gnome-vfs/patches/patch-ae,v
 retrieving revision 1.10
 diff -u -r1.10 patch-ae
 --- patches/patch-ae	22 Oct 2010 20:11:17 -0000	1.10
 +++ patches/patch-ae	25 Sep 2013 08:07:40 -0000
 @@ -1,4 +1,4 @@
 -$NetBSD: patch-ae,v 1.10 2010/10/22 20:11:17 drochner Exp $
 +$NetBSD$

  --- configure.orig	2010-09-28 09:35:20.000000000 +0000
  +++ configure
 @@ -23,7 +23,25 @@
     return 0;
   }
   _ACEOF
 -@@ -17533,7 +17535,7 @@ $as_echo "#define HAVE_SAMBA /**/" >>con
 +@@ -15717,9 +15719,14 @@ fi
 + 
 + 
 + if test "x$KRB5_CONFIG" != "xnone"; then
 +-   GSSAPI_LIBS="`${KRB5_CONFIG} --libs gssapi`"
 +-   GSSAPI_CFLAGS="`${KRB5_CONFIG} --cflags gssapi`"
 +-
 ++	# make sure krb5-config supports gssapi option
 ++	if $KRB5_CONFIG --libs gssapi 2>&1 | grep "Unknown option" > /dev/null ; then
 ++		GSSAPI_LIBS="`${KRB5_CONFIG} --libs`"
 ++   	GSSAPI_CFLAGS="`${KRB5_CONFIG} --cflags`"
 ++	else
 ++		GSSAPI_LIBS="`${KRB5_CONFIG} --libs gssapi`"
 ++   	GSSAPI_CFLAGS="`${KRB5_CONFIG} --cflags gssapi`"
 ++	fi
 +    saved_CPPFLAGS="$CPPFLAGS"
 +    saved_LIBS="$LIBS"
 +    LIBS="$LIBS $GSSAPI_LIBS"
 +@@ -17533,7 +17540,7 @@ $as_echo "#define HAVE_SAMBA /**/" >>con

   		msg_samba="yes"
                   if test x$with_samba_libs != x/usr/lib; then
 @@ -32,7 +50,7 @@
                   else
                   	SAMBA_LIBS="-lsmbclient"
                   fi
 -@@ -19370,12 +19372,12 @@ if test -n "$LIBGNOMEVFS_CFLAGS"; then
 +@@ -19370,12 +19377,12 @@ if test -n "$LIBGNOMEVFS_CFLAGS"; then
       pkg_cv_LIBGNOMEVFS_CFLAGS="$LIBGNOMEVFS_CFLAGS"
    elif test -n "$PKG_CONFIG"; then
       if test -n "$PKG_CONFIG" && \
 @@ -48,7 +66,7 @@
   else
     pkg_failed=yes
   fi
 -@@ -19386,12 +19388,12 @@ if test -n "$LIBGNOMEVFS_LIBS"; then
 +@@ -19386,12 +19393,12 @@ if test -n "$LIBGNOMEVFS_LIBS"; then
       pkg_cv_LIBGNOMEVFS_LIBS="$LIBGNOMEVFS_LIBS"
    elif test -n "$PKG_CONFIG"; then
       if test -n "$PKG_CONFIG" && \
 @@ -64,7 +82,7 @@
   else
     pkg_failed=yes
   fi
 -@@ -19411,14 +19413,14 @@ else
 +@@ -19411,14 +19418,14 @@ else
           _pkg_short_errors_supported=no
   fi
           if test $_pkg_short_errors_supported = yes; then

 --------------000009000203000200030305--

From: Richard PALO <richard.palo@free.fr>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/48230: gnome-vfs with gssapi option passes native krb5-config
 when pkgsrc version should be used
Date: Wed, 25 Sep 2013 18:32:05 +0200

 Le 25/09/13 10:30, Richard PALO a écrit :
 > The following reply was made to PR pkg/48230; it has been noted by GNATS.
 >
 > From: Richard PALO <richard.palo@baou.fr>
 > To: gnats-bugs@NetBSD.org
 > Cc:
 > Subject: Re: pkg/48230: gnome-vfs with gssapi option passes native krb5-config
 >   when pkgsrc version should be used
 > Date: Wed, 25 Sep 2013 10:26:59 +0200
 >
 >   This is a multi-part message in MIME format.
 >   --------------000009000203000200030305
 >   Content-Type: text/plain; charset=ISO-8859-1; format=flowed
 >   Content-Transfer-Encoding: 8bit
 >
 >   Le 22/09/13 12:35, richard@netbsd.org a �crit :
 >   >> Number:         48230
 >   >> Category:       pkg
 >   >> Synopsis:       gnome-vfs with gssapi option passes native krb5-config when pkgsrc version should be used
 >   >> Confidential:   no
 >   >> Severity:       critical
 >   >> Priority:       medium
 >   >> Responsible:    pkg-manager
 >   >> State:          open
 >   >> Class:          sw-bug
 >   >> Submitter-Id:   net
 >   >> Arrival-Date:   Sun Sep 22 10:35:00 +0000 2013
 >   >> Originator:     Richard PALO
 >   >> Release:        pkgsrc trunk
 >   >> Organization:
 >   >> Environment:
 >   > SunOS devzoneX 5.11 illumos-gate i86pc i386 i86pc
 >   >
 >   >> Description:
 >   > somewhat in relation to pkg/48228, building gnome-gvs with option 'gssapi' fails on solaris if pkgsrc mit-krb5, for example, is installed because the solaris supplied krb5-config doesn't [currently] support the gssapi option.
 >   >
 >   > The options seems to presume that "native" is intended.
 >   >
 >   > The attached patch at least allows the build to complete..
 >   >
 >   > NB: There is probably additional work to support current native solaris krb5 gssapi support to get over the incorrect assumption that krb5-config always supports a gssapi option.
 >   >> How-To-Repeat:
 >   >
 >   >> Fix:
 >   > richard@devzoneX:~/src/pkgsrc/sysutils/gnome-vfs$ git diff .
 >   > diff --git a/sysutils/gnome-vfs/options.mk b/sysutils/gnome-vfs/options.mk
 >   > index 43fd232..833e46c 100644
 >   > --- a/sysutils/gnome-vfs/options.mk
 >   > +++ b/sysutils/gnome-vfs/options.mk
 >   > @@ -23,7 +23,9 @@ CONFIGURE_ARGS+=      --disable-fam
 >   >
 >   >   .if !empty(PKG_OPTIONS:Mgssapi)
 >   >   .include "../../mk/krb5.buildlink3.mk"
 >   > +.      if !empty(USE_BUILTIN.${KRB5_TYPE}:M[yY][eE][sS])
 >   >   CONFIGURE_ENV+=                KRB5_CONFIG=${SH_KRB5_CONFIG}
 >   > +.      endif
 >   >   .else
 >   >   CONFIGURE_ENV+=                ac_cv_path_KRB5_CONFIG=none
 >   >   .endif
 >   >
 >   >
 >
 >   attached patch seems to fix krb5-config problem on solaris when native
 >   mit-krb5 is used.
 >
 >   --------------000009000203000200030305
 >   Content-Type: text/plain; charset=UTF-8;
 >    name="gvfs.patch"
 >   Content-Transfer-Encoding: 7bit
 >   Content-Disposition: attachment;
 >    filename="gvfs.patch"
 >
 >   Index: distinfo
 >   ===================================================================
 >   RCS file: /cvsroot/pkgsrc/sysutils/gnome-vfs/distinfo,v
 >   retrieving revision 1.34
 >   diff -u -r1.34 distinfo
 >   --- distinfo	19 Apr 2013 20:02:22 -0000	1.34
 >   +++ distinfo	25 Sep 2013 08:07:40 -0000
 >   @@ -5,7 +5,7 @@
 >    Size (gnome-vfs-2.24.4.tar.bz2) = 1890370 bytes
 >    SHA1 (patch-aa) = a5a3ac6be3e2a888eaaab108807d2aff651e62db
 >    SHA1 (patch-ab) = 6320180870f234a150dfe40f8f990deec7a2a0b5
 >   -SHA1 (patch-ae) = 2e46f1311674265d2054c7209aa057d8da2aca67
 >   +SHA1 (patch-ae) = bc6fe6afd2484acb9f1b31469c18f9a3c4ac7d70
 >    SHA1 (patch-af) = 6724c4a43d0d9525f5361ba7380c91adbf656e56
 >    SHA1 (patch-ai) = 6afefb35bfe3fc37e84048b95ccc272a01542ec2
 >    SHA1 (patch-am) = 4e037da71db0d3aae236475a0f2ead12738f35f9
 >   Index: options.mk
 >   ===================================================================
 >   RCS file: /cvsroot/pkgsrc/sysutils/gnome-vfs/options.mk,v
 >   retrieving revision 1.5
 >   diff -u -r1.5 options.mk
 >   --- options.mk	12 Jun 2012 15:46:03 -0000	1.5
 >   +++ options.mk	25 Sep 2013 08:07:40 -0000
 >   @@ -23,7 +23,9 @@
 >
 >    .if !empty(PKG_OPTIONS:Mgssapi)
 >    .include "../../mk/krb5.buildlink3.mk"
 >   +.	if !empty(USE_BUILTIN.${KRB5_TYPE}:M[yY][eE][sS])
 >    CONFIGURE_ENV+=		KRB5_CONFIG=${SH_KRB5_CONFIG}
 >   +.	endif
 >    .else
 >    CONFIGURE_ENV+=		ac_cv_path_KRB5_CONFIG=none
 >    .endif
 >   Index: patches/patch-ae
 >   ===================================================================
 >   RCS file: /cvsroot/pkgsrc/sysutils/gnome-vfs/patches/patch-ae,v
 >   retrieving revision 1.10
 >   diff -u -r1.10 patch-ae
 >   --- patches/patch-ae	22 Oct 2010 20:11:17 -0000	1.10
 >   +++ patches/patch-ae	25 Sep 2013 08:07:40 -0000
 >   @@ -1,4 +1,4 @@
 >   -$NetBSD: patch-ae,v 1.10 2010/10/22 20:11:17 drochner Exp $
 >   +$NetBSD$
 >
 >    --- configure.orig	2010-09-28 09:35:20.000000000 +0000
 >    +++ configure
 >   @@ -23,7 +23,25 @@
 >       return 0;
 >     }
 >     _ACEOF
 >   -@@ -17533,7 +17535,7 @@ $as_echo "#define HAVE_SAMBA /**/" >>con
 >   +@@ -15717,9 +15719,14 @@ fi
 >   +
 >   +
 >   + if test "x$KRB5_CONFIG" != "xnone"; then
 >   +-   GSSAPI_LIBS="`${KRB5_CONFIG} --libs gssapi`"
 >   +-   GSSAPI_CFLAGS="`${KRB5_CONFIG} --cflags gssapi`"
 >   +-
 >   ++	# make sure krb5-config supports gssapi option
 >   ++	if $KRB5_CONFIG --libs gssapi 2>&1 | grep "Unknown option" > /dev/null ; then
 >   ++		GSSAPI_LIBS="`${KRB5_CONFIG} --libs`"
 >   ++   	GSSAPI_CFLAGS="`${KRB5_CONFIG} --cflags`"
 >   ++	else
 >   ++		GSSAPI_LIBS="`${KRB5_CONFIG} --libs gssapi`"
 >   ++   	GSSAPI_CFLAGS="`${KRB5_CONFIG} --cflags gssapi`"
 >   ++	fi
 >   +    saved_CPPFLAGS="$CPPFLAGS"
 >   +    saved_LIBS="$LIBS"
 >   +    LIBS="$LIBS $GSSAPI_LIBS"
 >   +@@ -17533,7 +17540,7 @@ $as_echo "#define HAVE_SAMBA /**/" >>con
 >
 >     		msg_samba="yes"
 >                     if test x$with_samba_libs != x/usr/lib; then
 >   @@ -32,7 +50,7 @@
 >                     else
 >                     	SAMBA_LIBS="-lsmbclient"
 >                     fi
 >   -@@ -19370,12 +19372,12 @@ if test -n "$LIBGNOMEVFS_CFLAGS"; then
 >   +@@ -19370,12 +19377,12 @@ if test -n "$LIBGNOMEVFS_CFLAGS"; then
 >         pkg_cv_LIBGNOMEVFS_CFLAGS="$LIBGNOMEVFS_CFLAGS"
 >      elif test -n "$PKG_CONFIG"; then
 >         if test -n "$PKG_CONFIG" && \
 >   @@ -48,7 +66,7 @@
 >     else
 >       pkg_failed=yes
 >     fi
 >   -@@ -19386,12 +19388,12 @@ if test -n "$LIBGNOMEVFS_LIBS"; then
 >   +@@ -19386,12 +19393,12 @@ if test -n "$LIBGNOMEVFS_LIBS"; then
 >         pkg_cv_LIBGNOMEVFS_LIBS="$LIBGNOMEVFS_LIBS"
 >      elif test -n "$PKG_CONFIG"; then
 >         if test -n "$PKG_CONFIG" && \
 >   @@ -64,7 +82,7 @@
 >     else
 >       pkg_failed=yes
 >     fi
 >   -@@ -19411,14 +19413,14 @@ else
 >   +@@ -19411,14 +19418,14 @@ else
 >             _pkg_short_errors_supported=no
 >     fi
 >             if test $_pkg_short_errors_supported = yes; then
 >
 >   --------------000009000203000200030305--
 >
 >
 after a similar situation with www/curl, I'd like to consider updating 
 the line:
  >   ++		GSSAPI_LIBS="`${KRB5_CONFIG} --libs`"
 to
  >   ++		GSSAPI_LIBS="`${KRB5_CONFIG} --libs` -lgss"

>Unformatted:

NetBSD Home
NetBSD PR Database Search

(Contact us) $NetBSD: query-full-pr,v 1.39 2013/11/01 18:47:49 spz Exp $
$NetBSD: gnats_config.sh,v 1.8 2006/05/07 09:23:38 tsutsui Exp $
Copyright © 1994-2007 The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.