[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: bash cross with installed readline
From: |
Andrew Kosteltsev |
Subject: |
Re: bash cross with installed readline |
Date: |
Wed, 19 Mar 2014 11:28:59 +0400 |
Hi Mike,
Thanks a lot for quick answer. I'm sure that you in the right way. And I
hope you will test autotool scripts for both cases (system installed
readline; and inside bash src readline) for the future releases. (by the
way, the same situation we have in GCC with gmp)
I will try to use AC_TRY_COMPILE. Unfortunately I have not enough time now
but I hope I will find time slot for this in the nearest weeks.
Best Regards,
Andrey K.
On Wed, Mar 19, 2014 at 7:29 AM, Mike Frysinger <address@hidden> wrote:
> On Sun 16 Mar 2014 13:30:55 Andrew Kosteltsev wrote:
> > When we build bash for some targets the INCLUDES variable for BUILD_CC
> > contains the path to target readline headers. This path points to the
> > target headers which not preferred for utilities which prepared for build
> > machine.
> >
> > Also when we have installed readline on the target the configure script
> > avoids cross_compilation problems with AC_TRY_RUN and substitutes wrong
> > (very old) version of libreadline. If we sure that we installed correct
> > readline version we can change configure script for cross compilation
> > process.
> >
> > Please look at attached patches. If this solution can be used for common
> > case then please apply these patches for the future versions of bash.
>
> i haven't seen the issues you describe for the first patch. maybe it's
> because i don't pass full paths to the target readline but instead let the
> toolchain find it for me. so there is never any -I flag mixing.
>
> the 2nd patch is the wrong way to approach the problem. change the
> AC_TRY_RUN
> into an AC_TRY_COMPILE test by relying on RL_VERSION_{MAJOR,MINOR} being
> defined and doing an incremental search for its value. see how autoconf
> implements its AC_CHECK_SIZEOF macro using only compile tests for the
> algorithm.
> -mike