Bug 1097 - Cross-compile fixes
Summary: Cross-compile fixes
Status: CLOSED FIXED
Alias: None
Product: Portable OpenSSH
Classification: Unclassified
Component: Build system (show other bugs)
Version: -current
Hardware: Other All
: P2 minor
Assignee: Assigned to nobody
URL:
Keywords:
Depends on:
Blocks: 1047
  Show dependency tree / graph
 
Reported: 2005-09-30 11:48 AEST by Darren Tucker
Modified: 2006-10-07 11:42 AEST (History)
0 users

See Also:

Attachments
Replace AC_TRY_RUN with AC_COMPILE_IFELSE (3.75 KB, patch)
2005-09-30 11:50 AEST, Darren Tucker 		no flags Details | Diff
update based on Tim's comments, attempt to fix /etc/default/login for crosscompile (4.64 KB, patch)
2005-10-15 14:27 AEST, Darren Tucker 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darren Tucker 2005-09-30 11:48:44 AEST 
It has been pointed out that some of the tests in configure don't work when
cross compiling.  I have a patch (against -current) which I will attach but I
can't test it.

If anyone cross-compiling openssh can test it it would be appreciated.
Comment 1 Darren Tucker 2005-09-30 11:50:48 AEST 
Created attachment 977 [details]
Replace AC_TRY_RUN with AC_COMPILE_IFELSE

Patch against -current.  Will need to run "autoreconf" (autoconf 2.59 or
higher) before rerunning configure.
Comment 2 Tim Rice 2005-10-04 06:44:22 AEST 
>@@ -1002,10 +1003,12 @@ AC_ARG_WITH(skey,
>                       SKEY_MSG="yes"
>
>                       AC_MSG_CHECKING([for s/key support])
>-                      AC_TRY_RUN(
>+                      AC_TRY_COMPILE(
>                               [

I don't see the advantage of replacing one obsolete macro with
another obsolete macro.

Other than that, the patch looks good. I can't test cross compile, but
replaceing some of the obsolete macros is worth doing.
Comment 3 Darren Tucker 2005-10-15 14:27:42 AEST 
Created attachment 998 [details]
update based on Tim's comments, attempt to fix /etc/default/login for crosscompile
Comment 4 Darren Tucker 2005-10-17 23:29:40 AEST 
Tested cross compiling by aabaker at iee.org.  Applied patch id #998.
Comment 5 Damien Miller 2005-11-24 19:28:53 AEDT 
Put this in the list of 4.3 fixed bugs
Comment 6 Darren Tucker 2006-10-07 11:42:37 AEST 
Change all RESOLVED bug to CLOSED with the exception of the ones fixed post-4.4.