bugGNU TeXmacs - Bugs: bug #18060, check for 64bit host broken

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #18060: check for 64bit host broken

Submitted by:  Lionel Elie Mamane <lmamane>
Submitted on:  Thu 19 Oct 2006 04:29:44 PM UTC  
 
Category: CompilationPriority: 5 - Normal
Item Group: ErrorStatus: Fixed
Privacy: PublicAssigned to: None
Originator Name: Open/Closed: Closed
Release: CVSRelease: None
Fixed Release: NoneFixed Release: 
Keywords: 

Thu 14 Dec 2006 08:11:30 PM UTC, comment #3:

I was not able to find a place in the autoconf manual where the ac_cv_* syntax for shell variables is officially introduced, but they suggest using these variables at several places.

Could you attach the corrected patch?

Felix Breuer <fbreuer>
Project Member
Wed 13 Dec 2006 10:24:01 AM UTC, comment #2:

It seems it does not work:

checking if we are on a 64-bits computer... checking for void *... yes
checking size of void *... 8
no

Glancing at the patch, it seems you confuse C preprocessor symbols (such as SIZEOF_VOID_P) and shell variables in the configure script itself. Replacing SIZEOF_VOID_P by ac_cv_sizeof_void_p works, but I'm not sure whether the ac_cv_sizeof_void_p is considered as an internal name (not a stable interface that won't change).

Lionel Elie Mamane <lmamane>
Fri 08 Dec 2006 04:43:57 PM UTC, comment #1:

Patch attached. I have yet to test it. Does it work properly on 64bit machines?

(file #11484)

Felix Breuer <fbreuer>
Project Member
Thu 19 Oct 2006 04:29:44 PM UTC, original submission:

$ gcc -dumpmachine
x86_64-linux-gnu

but:

checking if we are on a 64-bits computer... no

because:

configure:8288: checking if we are on a 64-bits computer
configure:8318: g++ -o conftest conftest.cpp >&5
conftest.cpp: In function 'int main()':
conftest.cpp:32: error: 'exit' was not declared in this scope
configure:8321: $? = 1
configure: program exited with status 1
configure: failed program was:
| /* confdefs.h. */
| #define PACKAGE_NAME ""
| #define PACKAGE_TARNAME ""
| #define PACKAGE_VERSION ""
| #define PACKAGE_STRING ""
| #define PACKAGE_BUGREPORT ""
| #define STDC_HEADERS 1
| #define HAVE_SYS_TYPES_H 1
| #define HAVE_SYS_STAT_H 1
| #define HAVE_STDLIB_H 1
| #define HAVE_STRING_H 1
| #define HAVE_MEMORY_H 1
| #define HAVE_STRINGS_H 1
| #define HAVE_INTTYPES_H 1
| #define HAVE_STDINT_H 1
| #define HAVE_UNISTD_H 1
| #define HAVE_FILE 1
| #define HAVE_INTPTR_T 1
| #define HAVE_TIME_T 1
| #define HAVE_GETTIMEOFDAY 1
| #define HAVE_LIBDL 1
| #define DYNAMIC_LINKING 1
| #define USE_ICONV 1
| #define USE_FREETYPE 1
| #define LINKED_FREETYPE 1
| #define guile_str_size_t size_t
| /* end confdefs.h. */
|
| int
| main ()
| {
| exit(sizeof(void*))
| ;
| return 0;
| }
configure:8353: result: no

Solution: Add "#include <stdlib.h>" to the test program, or use an adequate built-in autoconf test (such as AC_CHECK_SIZEOF) instead of a self-hacked one.

Lionel Elie Mamane <lmamane>

 

Attached Files
file #11528:  configure.patch added by lmamane (563B - text/x-patch - Corrected patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by mdbenito (Updated the item)
  • -unavailable- added by lmamane (Posted a comment)
  • -unavailable- added by fbreuer (Updated the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 10 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 05 Mar 2012 11:48:42 PM UTCmdbenitoOpen/ClosedAccepted=>Closed
    Mon 18 Dec 2006 07:25:08 PM UTCfbreuerOpen/ClosedOpen=>Accepted
    Sat 16 Dec 2006 09:09:16 PM UTCfbreuerStatusIn Progress=>Fixed
    Sat 16 Dec 2006 08:04:24 PM UTCfbreuerAttached File#11484=>Removed
    Fri 15 Dec 2006 07:10:22 AM UTClmamaneAttached File-=>Added configure.patch, #11528
    Thu 14 Dec 2006 08:11:30 PM UTCfbreuerStatusReady For Test=>In Progress
    Wed 13 Dec 2006 10:24:17 AM UTClmamaneCarbon-CopyRemoved lmamane=>-
    Wed 13 Dec 2006 09:56:57 AM UTClmamaneCarbon-Copy-=>Added lmamane
    Fri 08 Dec 2006 04:43:57 PM UTCfbreuerAttached File-=>Added configure.patch, #11484
      StatusNone=>Ready For Test

    Back to the top


    Powered by Savane 3.1-cleanup1