Menu

#2 Patch to remove -m64, that is probably unneeded

open
nobody
None
5
2008-11-16
2008-11-16
No

Dear maq developers,

The Debian Med project prepared an installation package for maq on the Debian operating system, which summary is at the following URL: http://packages.debian.org/sid/maq

We disabled the -m64 option with the below patch:

--- a/configure.ac
+++ b/configure.ac
@@ -23,10 +23,7 @@ case "${host_cpu}-${host_os}" in
AC_COMPILE_IFELSE([AC_LANG_PROGRAM], [ext_CFLAGS="-m64"], []);;
esac;;
*)
- AC_MSG_CHECKING([if gcc accepts -m64])
- CFLAGS="-m64"
- AC_COMPILE_IFELSE([AC_LANG_PROGRAM], [ext_CFLAGS="-m64"; AC_MSG_RESULT([yes])],
- [ext_CFLAGS="-D_FILE_OFFSET_BITS=64"; AC_MSG_RESULT([no])]);;
+ ext_CFLAGS="-D_FILE_OFFSET_BITS=64";;
esac
AC_ARG_ENABLE(experimental, [ --enable-experimental enable experimental features],
[ext_CFLAGS="${ext_CFLAGS} -DMAQ_SHOW_EXPERIMENTAL"], [])

Our rationale is that on platforms like amd64 it is not necessary because the compiler will be in 64-bit mode anyway, while on the i386 platform it makes compilation impossible with the autotools way. -m64 would apparently be useful only for building a 64-bit binary on machine running a 64-bit powerpc or s390 kernel with a 32-bit userland system.

We welcome any comment on our patch. Please feel free to apply it if you agree with our conclusions.

Best regards,

--
Charles Plessy
Debian Med packaging team
http://www.debian.org/devel/debian-med
Tsurumi, Kanagawa, Japan

Discussion


Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.