Re: CVS commit: pkgsrc/audio/pulseaudio

["Jared D. McNeill" <jmcneill%netbsd.org@localhost>]

On Mon, Dec 22, 2008 at 4:29 PM, Robert Elz <kre%munnari.oz.au@localhost> wrote:
>    Date:        Mon, 22 Dec 2008 02:50:09 +0000 (UTC)
>    From:        "Jared D. McNeill" <jmcneill%NetBSD.org@localhost>
>    Message-ID:  <20081222025009.D4A58175D0%cvs.netbsd.org@localhost>
>
>  | Pull in libatomic_ops if NetBSD native atomic ops are not available, should
>  | fix build on NetBSD < 5.0 and other platforms.
>
> It changes the symptoms, the thing actually starts to compile now,
> but doesn't succeed ...

There's a bug in patch-ba, it shouldn't be trying to use sys/atomic.h
if the header isn't found.

Can you try the following patch to the patch;

Index: patch-ba
===================================================================
RCS file: /cvsroot/pkgsrc/audio/pulseaudio/patches/patch-ba,v
retrieving revision 1.1
diff -u -r1.1 patch-ba
--- patch-ba    20 Dec 2008 16:10:25 -0000      1.1
+++ patch-ba    22 Dec 2008 21:34:04 -0000
@@ -35,7 +35,7 @@
                ;;
          *)
 -          AC_MSG_RESULT([unknown])
-+          if test "x$pulse_target_os" = "xnetbsd" && test
"x$ac_cv_header_sys_atomic_h" != "xno"; then
++          if test "x$pulse_target_os" = "xnetbsd" && test
"x$ac_cv_header_sys_atomic_h" = "xyes"; then
 +              AC_MSG_RESULT([yes])
 +              AC_DEFINE_UNQUOTED(NETBSD_ATOMIC_OPS, 1, [netbsd 
implementation])
 +              need_libatomic_ops=no