Re: devel/libelf needs 64bit hack^whelp

["Cherry G. Mathew" <cherry.g.mathew%gmail.com@localhost>]

On 23 August 2010 14:44, Cherry G. Mathew <cherry.g.mathew%gmail.com@localhost> 
wrote:
> Hi,
>
> NetBSD elf.h 64bit support is broken [1].
>
> Would the following patch be the appropriate way to fix this breakage
> for devel/libelf, by using libelf's own elf_repl.h instead of native
> elf.h ?
>
> Thanks,
>
> --
> ~Cherry
>
> [1] http://mail-index.netbsd.org/tech-toolchain/2010/08/20/msg001355.html
>
>
> $NetBSD$
>
> --- configure.in.orig   2008-05-23 08:17:56.000000000 +0000
> +++ configure.in
> @@ -60,10 +60,19 @@ AC_CACHE_CHECK([if ${CC} can compile elf
>     [libelf_cv_elf_h_works=yes],
>     [libelf_cv_elf_h_works=no])
>  ])
> +
> +# NetBSD defines a type Elf64_Quarter
> +# We don't use the NetBSD elf.h because of 64bit breakage
> +# See:
> +
> +AC_CACHE_CHECK([for Elf64_Quarter], libelf_cv_dummyelf_h_works,
> +    [libelf_cv_elf_h_works=no])
> +
>  if test "$libelf_cv_elf_h_works" = no; then
>   ac_cv_header_elf_h=no
>   ac_cv_header_sys_elf_h=no
>  fi
> +
>  if test "$ac_cv_header_elf_h" = yes; then
>   AC_DEFINE(__LIBELF_HEADER_ELF_H, [<elf.h>])
>  elif test "$ac_cv_header_sys_elf_h" = yes; then
>


I'm going to commit this later today, if there are no objections.

Cheers,
-- 
~Cherry