Re: [Bug binutils/4334] MAKE FAILED: cpu-powerpc.o isn't added to libbfd.a ?
Hi Vincent,
For example, a simple script like this in the main configure may be enough (or
maybe wrapped into an autoconf macro) :
echo a >file
if test `cat file` != a
then
echo \`cat\` bug detected.
echo Please run configure from on a binmode filesystem
echo or use the igncr trick
exit 1
fi
At which point the user goes:
"igncr trick" ? What "igncr trick" ?
So we would have to be a bit more explicit. How about modifying the
configure patch already suggested to perform the cat check instead ? I
have attached a patch that should do this, although I have not checked
it - would you care to try it out for me ?
Cheers
Nick
Index: configure.ac
===
--- configure.ac(revision 125308)
+++ configure.ac(working copy)
@@ -965,6 +965,7 @@ case "${host}" in
host_makefile_frag="config/mh-djgpp"
;;
*-cygwin*)
+ACX_CHECK_CYGWIN_CAT_WORKS
host_makefile_frag="config/mh-cygwin"
;;
*-mingw32*)
Index: configure
===
--- configure (revision 125308)
+++ configure (working copy)
@@ -2676,6 +2676,32 @@ case "${host}" in
host_makefile_frag="config/mh-djgpp"
;;
*-cygwin*)
+
+echo "$as_me:$LINENO: checking to see if cat works as expected" >&5
+echo $ECHO_N "checking to see if cat works as expected... $ECHO_C" >&6
+echo a >cygwin-cat-check
+if test `cat cygwin-cat-check` == a ; then
+ echo "$as_me:$LINENO: result: yes" >&5
+echo "${ECHO_T}yes" >&6
+else
+ echo "$as_me:$LINENO: result: no" >&5
+echo "${ECHO_T}no" >&6
+ { { echo "$as_me:$LINENO: error: The cat command does not ignore carriage
return characters.
+ Please either mount the build directory in binary mode or run the following
+ commands before running any configure script:
+set -o igncr
+export SHELLOPTS
+ " >&5
+echo "$as_me: error: The cat command does not ignore carriage return
characters.
+ Please either mount the build directory in binary mode or run the following
+ commands before running any configure script:
+set -o igncr
+export SHELLOPTS
+ " >&2;}
+ { (exit 1); exit 1; }; }
+rm cygwin-cat-check
+fi
+
host_makefile_frag="config/mh-cygwin"
;;
*-mingw32*)
Index: config/acx.m4
===
--- config/acx.m4 (revision 125308)
+++ config/acx.m4 (working copy)
@@ -597,3 +597,25 @@ AC_DEFUN([ACX_BUGURL],[
AC_SUBST(REPORT_BUGS_TO)
AC_SUBST(REPORT_BUGS_TEXI)
])
+
+dnl
+dnl # ACX_CHECK_CYGWIN_CAT_WORKS
+dnl # On Cygwin hosts, check that the cat command ignores
+dnl # carriage returns as otherwise builds will not work.
+dnl # See binutils PR 4334 for more details.
+AC_DEFUN([ACX_CHECK_CYGWIN_CAT_WORKS],[
+AC_MSG_CHECKING([to see if cat works as expected])
+echo a >cygwin-cat-check
+if test `cat cygwin-cat-check` == a ; then
+ AC_MSG_RESULT(yes)
+else
+ AC_MSG_RESULT(no)
+ AC_MSG_ERROR([The cat command does not ignore carriage return characters.
+ Please either mount the build directory in binary mode or run the following
+ commands before running any configure script:
+set -o igncr
+export SHELLOPTS
+ ])
+rm cygwin-cat-check
+fi
+])
___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug binutils/4334] MAKE FAILED: cpu-powerpc.o isn't added to libbfd.a ?
--- Additional Comments From nickc at redhat dot com 2007-06-05 11:55
---
Subject: Re: MAKE FAILED: cpu-powerpc.o isn't added to
libbfd.a ?
Hi Vincent,
> For example, a simple script like this in the main configure may be enough
> (or
> maybe wrapped into an autoconf macro) :
> echo a >file
> if test `cat file` != a
> then
> echo \`cat\` bug detected.
> echo Please run configure from on a binmode filesystem
> echo or use the igncr trick
> exit 1
> fi
At which point the user goes:
"igncr trick" ? What "igncr trick" ?
So we would have to be a bit more explicit. How about modifying the
configure patch already suggested to perform the cat check instead ? I
have attached a patch that should do this, although I have not checked
it - would you care to try it out for me ?
Cheers
Nick
Index: configure.ac
===
--- configure.ac(revision 125308)
+++ configure.ac(working copy)
@@ -965,6 +965,7 @@ case "${host}" in
host_makefile_frag="config/mh-djgpp"
;;
*-cygwin*)
+ACX_CHECK_CYGWIN_CAT_WORKS
host_makefile_frag="config/mh-cygwin"
;;
*-mingw32*)
Index: configure
===
--- configure (revision 125308)
+++ configure (working copy)
@@ -2676,6 +2676,32 @@ case "${host}" in
host_makefile_frag="config/mh-djgpp"
;;
*-cygwin*)
+
+echo "$as_me:$LINENO: checking to see if cat works as expected" >&5
+echo $ECHO_N "checking to see if cat works as expected... $ECHO_C" >&6
+echo a >cygwin-cat-check
+if test `cat cygwin-cat-check` == a ; then
+ echo "$as_me:$LINENO: result: yes" >&5
+echo "${ECHO_T}yes" >&6
+else
+ echo "$as_me:$LINENO: result: no" >&5
+echo "${ECHO_T}no" >&6
+ { { echo "$as_me:$LINENO: error: The cat command does not ignore carriage
return characters.
+ Please either mount the build directory in binary mode or run the following
+ commands before running any configure script:
+set -o igncr
+export SHELLOPTS
+ " >&5
+echo "$as_me: error: The cat command does not ignore carriage return
characters.
+ Please either mount the build directory in binary mode or run the following
+ commands before running any configure script:
+set -o igncr
+export SHELLOPTS
+ " >&2;}
+ { (exit 1); exit 1; }; }
+rm cygwin-cat-check
+fi
+
host_makefile_frag="config/mh-cygwin"
;;
*-mingw32*)
Index: config/acx.m4
===
--- config/acx.m4 (revision 125308)
+++ config/acx.m4 (working copy)
@@ -597,3 +597,25 @@ AC_DEFUN([ACX_BUGURL],[
AC_SUBST(REPORT_BUGS_TO)
AC_SUBST(REPORT_BUGS_TEXI)
])
+
+dnl
+dnl # ACX_CHECK_CYGWIN_CAT_WORKS
+dnl # On Cygwin hosts, check that the cat command ignores
+dnl # carriage returns as otherwise builds will not work.
+dnl # See binutils PR 4334 for more details.
+AC_DEFUN([ACX_CHECK_CYGWIN_CAT_WORKS],[
+AC_MSG_CHECKING([to see if cat works as expected])
+echo a >cygwin-cat-check
+if test `cat cygwin-cat-check` == a ; then
+ AC_MSG_RESULT(yes)
+else
+ AC_MSG_RESULT(no)
+ AC_MSG_ERROR([The cat command does not ignore carriage return characters.
+ Please either mount the build directory in binary mode or run the following
+ commands before running any configure script:
+set -o igncr
+export SHELLOPTS
+ ])
+rm cygwin-cat-check
+fi
+])
--
http://sourceware.org/bugzilla/show_bug.cgi?id=4334
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils
Re: Results for binutils-2.17cvs20070426 (Debian Patches) testsuite on i686-pc-linux
Hi Rob, I hope it is OK to file bug reports this way. I couldn't see a bug report script so I used the one from GCC. We generally use bugzilla for filing binutils bug reports: http://www.sourceware.org/bugzilla/ You do not actually say what bug you are reporting. Presumably you consider the testsuite failures to be bugs, yes ? FAIL: windres/bmpalign (parse) This failure should now be resolved. === ld Summary === # of expected passes 309 # of unexpected failures 103 I am now showing 8 unexpected failures for an x86_64-linux-gnulibc toolchain based on today's gcc and binutils sources. Compiler version: gcc 4.2.0 20070501 (prerelease) Binutils version: binutils-2.17cvs20070426 Platform: i686-pc-linux-gnu configure flags: --enable-64-bit-bfd --enable-shared --enable-targets=i686-pc-linux-gnu,x86_64-pc-linux-gnu,i686-pc-cygwin,arm-epoc-pe,arm-none-symbianelf ,arm-none-eabi This is a complicated set of targets. Is there any particular reason for them ? I applied this patch: http://ftp.debian.org/debian/pool/main/b/binutils/binutils_2.17cvs20070426-6.diff.gz I tried to look at this patch but it was not available for download from that URL. What does it do ? Cheers Nick ___ bug-binutils mailing list bug-binutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-binutils
Re: [Bug gas/4572] division doesn't work on solaris/x86
Hi Clint, You are saying there is no way to do division in integer expressions with gas on solaris, even though it works on all other OSes? Isn't that less than ideal? Actually Alan was saying that you should look at the assembler source file gas/app.c and see how you can modify it to interpret "\/" as a division operator and not a comment. Cheers Nick ___ bug-binutils mailing list bug-binutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug gas/4572] division doesn't work on solaris/x86
--- Additional Comments From nickc at redhat dot com 2007-06-05 12:15 --- Subject: Re: division doesn't work on solaris/x86 Hi Clint, > You are saying there is no way to do division in integer expressions with gas > on > solaris, even though it works on all other OSes? Isn't that less than ideal? Actually Alan was saying that you should look at the assembler source file gas/app.c and see how you can modify it to interpret "\/" as a division operator and not a comment. Cheers Nick -- http://sourceware.org/bugzilla/show_bug.cgi?id=4572 --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ bug-binutils mailing list bug-binutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug gas/4587] Sparc fcmps instruction - extra operand causes SEGFAULT
--- Additional Comments From nickc at redhat dot com 2007-06-05 13:48 --- Created an attachment (id=1878) --> (http://sourceware.org/bugzilla/attachment.cgi?id=1878&action=view) Terminate tls_ops array. -- http://sourceware.org/bugzilla/show_bug.cgi?id=4587 --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ bug-binutils mailing list bug-binutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug gas/4587] Sparc fcmps instruction - extra operand causes SEGFAULT
--- Additional Comments From nickc at redhat dot com 2007-06-05 13:51 --- Hi Joe, I was just about to say, "oh gas 2.14 is so old, you need to upgrade to the latest version (2.17)" when I decided to check first... It is a good thing I did as this bug is present in those sources and the mainline development sources too! Anyway I have uploaded a patch to fix the problem and although it might need a little tweaking, I am sure that you will be able to apply it to your 2.14 sources and obtain a working assembler. I am going to check this patch in, along with a new gas testsuite entry based on this pr, in a few hours time, after I have run a few more checks. Cheers Nick -- What|Removed |Added Status|NEW |RESOLVED Resolution||FIXED http://sourceware.org/bugzilla/show_bug.cgi?id=4587 --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ bug-binutils mailing list bug-binutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-binutils
[Bug binutils/4334] MAKE FAILED: cpu-powerpc.o isn't added to libbfd.a ?
--- Additional Comments From vincent dot riviere at freesbee dot fr 2007-06-05 19:27 --- Hi, Nick ! > "igncr trick" ? What "igncr trick" ? > So we would have to be a bit more explicit. Of course, you're right. My little script was just an example to show that the problem can be easily detected. I prefer to let english speaking people write a correct error message - and you did it perfectly. I've just tested you patch, and it works exactly as expected. Bravo ! There is only a microscopic detail : when the check fails, the file cygwin-cat- check stays in the build directory. This is because rm is done after AC_MSG_ERROR. Excellent job ! You just have to commit. Vincent -- http://sourceware.org/bugzilla/show_bug.cgi?id=4334 --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ bug-binutils mailing list bug-binutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-binutils
