Garrett Cooper wrote:
> On Wed, Nov 24, 2010 at 12:01 AM, Bian Naimeng <[email protected]> wrote:
>>
>> Mitani wrote:
>>> Hi,
>>>
>>> I reported failure of "configure" on the other day.
>>> http://www.mail-archive.com/[email protected]/msg11137.html
>>>
>>> But I think that it's hard to notice because my report is contributed to
>>> the report of "realtime" failure. It's my clumsiness.
>>> Therefore, I report about "configure" again with the recent git.
>>>
>>>
>> Hi mitani-san,
>>
>> Please try the following patch.
>>
>> -----------------------
>> fix configure error.
>>
>> Signed-off-by: Bian Naimeng <[email protected]>
>>
>> ---
>> m4/ltp-quota.m4 | 4 ++--
>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/m4/ltp-quota.m4 b/m4/ltp-quota.m4
>> index af1212a..5a367e9 100644
>> --- a/m4/ltp-quota.m4
>> +++ b/m4/ltp-quota.m4
>> @@ -29,7 +29,7 @@ int main(void) {
>> struct dqblk dq;
>> return quotactl(QCMD(Q_GETINFO, USRQUOTA), (const char *) "/dev/null",
>> geteuid(), (caddr_t) &dq);
>> -}],[has_quotav2="yes"],[dnl END quota v2 check
>> +}])],[has_quotav2="yes"],[dnl END quota v2 check
>> AC_LINK_IFELSE([AC_LANG_SOURCE([
>> #define _LINUX_QUOTA_VERSION 1
>> #include <sys/types.h>
>> @@ -46,4 +46,4 @@ if test "x$has_quotav1" = "xyes"; then
>> elif test "x$has_quotav2" = "xyes"; then
>> AC_DEFINE(HAVE_QUOTAV2,1,[Define to 1 if you have quota v2])
>> fi
>> -])])
>> +])
>> --
>> 1.7.0.4
>>
>>
>>
>>> ---
>>>
>>> "configure" failed with git "ltp-7d2055e.tar.gz":
>>> ------------<RHEL4.8 - x86>
>>> ./configure: line 8228: syntax error near unexpected token `elif'
>>> ./configure: line 8228: `elif test "x$has_quotav2" = "xyes"; then'
>>> ------------
>>>
>>> Environmants are RHEL5.5/4.8 -- x86/x86_64/ia64.
>>>
>>>
>>> "${LTPROOT}/configure" file of "ltp-7d2055e.tar.gz" is following:
>>>
>>> ------------< RHEL4.8 - x86>
>>> [...]
>>>
>>> cat >conftest.$ac_ext <<_ACEOF
>>> /* confdefs.h. */
>>> _ACEOF
>>> cat confdefs.h >>conftest.$ac_ext
>>> cat >>conftest.$ac_ext <<_ACEOF
>>> ---
>>> LINE 8210
>>> /* end confdefs.h. */
>>>
>>> #define _LINUX_QUOTA_VERSION 2
>>> #include <sys/types.h>
>>> #include <sys/quota.h>
>>> #include <unistd.h>
>>> int main(void) {
>>> struct dqblk dq;
>>> return quotactl(QCMD(Q_GETINFO, USRQUOTA), (const char *)
>>> "/dev/null",
>>> geteuid(), (caddr_t) &dq);
>>> }
>>> --- LINE 8221
>>> if test "x$has_quotav1" = "xyes"; then
>>>
>>> cat >>confdefs.h <<\_ACEOF
>>> --- LINE 8224
>>> #define HAVE_QUOTAV1 1
>>> _ACEOF
>>>
>>> elif test "x$has_quotav2" = "xyes"; then
>>>
>>> cat >>confdefs.h <<\_ACEOF
>>> #define HAVE_QUOTAV2 1
>>> _ACEOF
>>>
>>> fi
>>>
>>> _ACEOF
>>> --- LINE 8236
>>> rm -f conftest.$ac_objext conftest$ac_exeext
>>> if { (ac_try="$ac_link"
>>> [...]
>>> ------------
>>>
>>>
>>> The line 8210 (" 8210 cat >>conftest.$ac_ext <<_ACEOF") is the "here
>>> document".
>>> First "_ACEOF" which appears after this line is the line 8224.
>>> But the line 8224 is "here document", too.
>>>
>>> I think that it is syntax error.
>>>
>>> In the line 8236, there is "_ACEOF".
>>> This "_ACEOF" may be just after line 8221, I think.
>>>
>>> I tried to execute "configure" after above revision, and it succeeded.
>>> After this, "make" and "make install" succeeded, too.
>
> Thanks for better pointing out the problem. I spread the autoconf
> tests out so that (hopefully) the same mistake won't happen again.
Hi Garrett,
I have reviewed your patch, it can work.
However, if you can attach your patch at this thread(not a URL) before
it committed, it's helpful to review, and can avoid new bugs, right? ^_^
Thanks
Bian
> Cheers,
> -Garrett
>
------------------------------------------------------------------------------
Increase Visibility of Your 3D Game App & Earn a Chance To Win $500!
Tap into the largest installed PC base & get more eyes on your game by
optimizing for Intel(R) Graphics Technology. Get started today with the
Intel(R) Software Partner Program. Five $500 cash prizes are up for grabs.
http://p.sf.net/sfu/intelisp-dev2dev
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
