Bug#656838: oce FTBFS on Alpha: Alpha Linux is misdetected as Alpha OSF1 Unix

[Adam C Powell IV]

Hello Denis,

Would you like me to upload the new oce with this change, or wait for a
new upstream or other changes?

-Adam

On Sat, 2012-02-25 at 00:04 +0100, D. Barbier wrote:
> tags 656838 pending
> thanks
> 
> Le 22 janvier 2012 04:37, Michael Cree  a écrit :
> > Source: oce
> > Version: 0.8.0-1
> > Severity: Normal
> > User: debian-alpha at lists.debian.org
> > Usertags: alpha
> > X-Debbugs-CC: debian-alpha at lists.debian.org
> >
> > oce FTBFS on alpha with the error:
> >
> > [ 39%] Building CXX object
> > adm/cmake/TKService/CMakeFiles/TKService.dir/__/__/__/src/MFT/MFT_FontManager.cxx.o
> > cd
> > /build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/obj-alpha-linux-gnu/adm/cmake/TKService
> > && /usr/bin/c++   -DTKService_EXPORTS -D_OCC64 -DHAVE_FTGL_NEWER212
> > -DHAVE_CONFIG_H -DCSFDB -DOCC_CONVERT_SIGNALS -DNDEBUG -O3 -DNDEBUG
> > -fPIC
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/obj-alpha-linux-gnu/build_inc
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/inc
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/AlienImage
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/AlienImage
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/Aspect
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/Aspect
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/CGM
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/CGM
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/Image
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/Image
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/MFT
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/MFT -I/bui
> >  ld/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/PS
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/PS
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/PlotMgt
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/PlotMgt
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/SelectBasics
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/SelectBasics
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/TColQuantity
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/TColQuantity
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/Viewer
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/Viewer
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/ImageUtility
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/ImageUtility
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/Xw
> > -I/build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/drv/Xw    -o
> > CMakeFiles/TKService.dir/__/__/__/src/MFT/MFT_FontManager.cxx.o -c
> > /build/buildd-oce_0.8.0-1-alpha-
> >  nfIXQh/oce-0.8.0/src/MFT/MFT_FontManager.cxx
> > /build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/MFT/MFT_FontManager.cxx:
> > In static member function 'static Standard_Boolean
> > MFT_FontManager::Read(MFT_FileRecord&)':
> > /build/buildd-oce_0.8.0-1-alpha-nfIXQh/oce-0.8.0/src/MFT/MFT_FontManager.cxx:2870:23:
> > error: 'MAP_VARIABLE' was not declared in this scope
> >
> > Full build log is at:
> > http://buildd.debian-ports.org/status/fetch.php?pkg=oce&arch=alpha&ver=0.8.0-1&stamp=1326555718
> >
> > The build error is caused by a misdetection of Alpha Linux as Alpha OSF1
> > Unix in inc/MFT_FileRecord.hxx beginning at line 50 with:
> >
> > #if defined ( __alpha ) || ( DECOSF1 )
> > #include <sys/mman.h>
> > #define MMAP(file_addr,mmap_size,fildes) \
> >        mmap((caddr_t) 0x1000000000,(size_t) mmap_size,PROT_READ, \
> >                MAP_FILE | MAP_PRIVATE | MAP_VARIABLE,fildes,(off_t)
> > file_addr)
> > #define MUNMAP(mmap_addr,mmap_size) \
> >        munmap((caddr_t) mmap_addr,(size_t) mmap_size)
> > #endif  // __alpha
> >
> > The pre-defined compiler macro __alpha is defined if the hardware is
> > Alpha whatever the OS and so the above code, intended only for OSF1,
> > gets compiled under Linux.
> >
> > Maybe modifying the conditional to be something like:
> >
> > #if (defined ( __alpha ) || ( DECOSF1 )) && ! defined( linux )
> >
> > would fix it --- but I don't know what is the appropriate macro to
> > detect Linux.
> 
> Hello,
> 
> Thanks for your report, your patch has been applied in our repository,
> it should work fine.
> 
> > I did a quick grep and check of all other defined(__alpha) occurences
> > and they look fine to me, but did notice issues that might affect other
> > architectures supported by Debian, that is, the code has the appearance
> > of being written with the assumption that Linux is Intel only.
> 
> You may be right, but we did not get any feedback about such issues yet.
> 
> Denis
-- 
GPG fingerprint: D54D 1AEE B11C CE9B A02B  C5DD 526F 01E8 564E E4B6

Engineering consulting with open source tools
http://www.opennovation.com/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part
URL: <http://lists.alioth.debian.org/pipermail/debian-science-maintainers/attachments/20120226/ad38c2b2/attachment.pgp>