#18 [CMake] error for lseek64 and open64

[M.S. Breitenfeld]

I think there is an error in CMakeLists.txt on line 84 of the trunk version:

81 if (HAVE_OPEN64)
82 add_definitions(-DHAVE_LSEEK64)
83 endif (HAVE_OPEN64)
84 if (HAVE_OPEN64)
85 add_definitions(-DHAVE_LSEEK64)
86 endif (HAVE_LSEEK64)

it should be:

81 if (HAVE_OPEN64)
82 add_definitions(-DHAVE_LSEEK64)
83 endif (HAVE_OPEN64)
84 if (HAVE_LSEEK64)
85 add_definitions(-DHAVE_LSEEK64)
86 endif (HAVE_LSEEK64)

[M.S. Breitenfeld]

I would think that it should be this:

if (CGNS_ENABLE_LFS)
  if (WIN32)

NOTE: why are we not checking if these are available?
      In the current source -DHAVE_LSEEK64 is repeated

    add_definitions(-DHAVE_OPEN64 -DHAVE_LSEEK64)
  else (WIN32)
    include(CheckFunctionExists)
    CHECK_FUNCTION_EXISTS(open64 HAVE_OPEN64)
    CHECK_FUNCTION_EXISTS(lseek64 HAVE_LSEEK64)
    if (HAVE_OPEN64)
      add_definitions(-DHAVE_OPEN64)
    endif (HAVE_OPEN64)
    if (HAVE_LSEEK64)
      add_definitions(-DHAVE_LSEEK64)
    endif (HAVE_LSEEK64)
  endif (WIN32)
else (CGNS_ENABLE_LFS)
  remove_definitions(-DHAVE_OPEN64 -DHAVE_LSEEK64)
endif (CGNS_ENABLE_LFS)

These are only used in the adf routine ADF_internals.c