Bug 8410 - configure libreplace_cv_HAVE_GETADDRINFO fails with cross-compile
Summary: configure libreplace_cv_HAVE_GETADDRINFO fails with cross-compile
Status: NEW
Alias: None
Product: Samba 3.6
Classification: Unclassified
Component: Build environment (show other bugs)
Version: 3.6.0
Hardware: All All
: P5 normal
Target Milestone: ---
Assignee: Björn Jacke
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-26 23:24 UTC by Todd Richmond
Modified: 2011-09-10 18:40 UTC (History)
0 users

See Also:


Attachments
Patch to remove cross-compile check (900 bytes, application/octet-stream)
2011-09-10 18:40 UTC, Todd Richmond
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Todd Richmond 2011-08-26 23:24:36 UTC
There is no way to get around the check for the required getaddrinfo() replacement that was added for bug 5910 (according to configure comments).

Using libreplace_cv_HAVE_GETADDRINFO=yes gets past the first check for a system where getaddrinfo is correct (most any relatively new system is fine) but the extra check ignores this preset value

I commented out the abort and Samba 3.6 now compiles (and runs with one more hack!) on OpenWRT
Comment 1 Jeremy Allison 2011-08-30 18:22:12 UTC
Can you attach a patch for this so I can understand exactly what you're proposing here ?

Thanks !

Jeremy.
Comment 2 Todd Richmond 2011-08-30 21:46:14 UTC
My workaround was just to comment out the cross-compile restriction, so it isn't a full patch. It does show the problem code though


*** samba-3.6.0.orig/source3/configure	2011-08-25 15:24:07.989937998 -0700
--- samba-3.6.0/source3/configure	2011-08-25 15:29:02.737899252 -0700
***************
*** 13269,13278 ****
  	# see bug 5910, use our replacements if we detect
  	# a broken system.
  	if test "$cross_compiling" = yes; then :
!   { { $as_echo "$as_me:${as_lineno-$LINENO}: error: in \`$ac_pwd':" >&5
! $as_echo "$as_me: error: in \`$ac_pwd':" >&2;}
! as_fn_error "cannot run test program while cross compiling
! See \`config.log' for more details." "$LINENO" 5; }
  else
    cat confdefs.h - <<_ACEOF >conftest.$ac_ext
  /* end confdefs.h.  */
--- 13269,13275 ----
  	# see bug 5910, use our replacements if we detect
  	# a broken system.
  	if test "$cross_compiling" = yes; then :
! $as_echo "assuming valid getaddrinfo without bug 5910" >&2
  else
    cat confdefs.h - <<_ACEOF >conftest.$ac_ext
  /* end confdefs.h.  */
Comment 3 Todd Richmond 2011-09-10 18:40:10 UTC
Created attachment 6876 [details]
Patch to remove cross-compile check