alpine @2.00_5: build fails with duplicate function declaration

[schnide (Joe Schnide)]

Running DP 5 on MacBook Pro

Please let me know if I can provide further information.

Thanks Joe

see attached for log

[schnide (Joe Schnide)]

alpine build log

[larryv (Lawrence Velázquez)]

:info:build /usr/bin/clang -DHAVE_CONFIG_H   -I../include -I../include -I/opt/local/include -I/opt/local/include -I/opt/local/include -g -D_THREAD_SAFE  -Dbsd -pipe -Os -arch x86_64 -MT ablookup.o -MD -MP -MF .deps/ablookup.Tpo -c -o ablookup.o ablookup.c
:info:build In file included from ablookup.c:20:
:info:build ./../pith/util.h:59:8: error: conflicting types for 'panic'
:info:build void      panic(char *);
:info:build           ^
:info:build /usr/include/mach/mach.h:86:15: note: previous declaration is here
:info:build extern void                     panic(const char *, ...);
:info:build                                 ^
:info:build 1 error generated.

[ryandesign (Ryan Carsten Schmidt)]

We could rename alpine's panic function so it does not conflict with mach's.

[greeneg (Gary Greene)]

I've tested a build after doing the proper SnR on the tree to rename panic to uw_panic. Builds fine and runs correctly. Attaching patch for review.

[greeneg (Gary Greene)]

Rename all uses of panic to uw_panic to work around mach headers scoping issues.

[schnide (Joe Schnide)]

How do I go about testing this solution? Thanks.

[schnide (Joe Schnide)]

Did some research and got the patch in and built fine. No problems. Thanks.

[infostud (Geoff Fellows)]

Cc Me!

[Liontooth (David Liontooth)]

This bug is still hitting; could someone please apply the tested and verified patch?

[SickTeddyBear]

Cc Me!

[glen@…]

Cc Me!

[jcvernaleo (John C. Vernaleo)]

Cc Me!

[jcvernaleo (John C. Vernaleo)]

I manually applied that patch and alpine builds on Yosemite and alpine builds again. Nice work. Hopefully it can get added in soon.

[Russell-Jones-OxPhys (Russell Jones)]

Cc Me!

[Liontooth (David Liontooth)]

The mail client alpine has been uninstallable for four months because of a variable name conflict. The patch simply replaces "panic" with "uw_panic" --

  sudo port install alpine --without_tcl
  wget https://trac.macports.org/raw-attachment/ticket/44634/alpine-2.00_panic_rename.patch -P /tmp
  cd /opt/local/var/macports/sources/rsync.macports.org/release/tarballs/ports/mail/alpine/work
  sudo chown -R $USER alpine-2.00
  patch -p0 < /tmp/alpine-2.00_panic_rename.patch
  sudo chown -R macports alpine-2.00
  sudo port install alpine --without_tcl

Could someone please put this bug out of its misery -- the maintainer is not taking care of it and it affects lots of users.

Cheers, David

[jcvernaleo (John C. Vernaleo)]

Does the alpine port have a maintainer even?

If it doesn't, I'm willing to step up and do it. I maintain the alpine port in bitrig (an OpenBSD fork) so I'm pretty familiar with it.

[neverpanic (Clemens Lang)]

Committed in r129886, added new maintainer in r129887, thanks.