[Bug binutils/10764] New: compiler option added after operands

View: New views
3 Messages — Rating Filter:   Alert me  

[Bug binutils/10764] New: compiler option added after operands

by Bugzilla from sourceware-bugzilla@sourceware.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

The compiler option -Wno-error is added last, after the operand.
This is something my compiler isn't happy about

===================================================================
RCS file: /cvs/src/src/binutils/Makefile.am,v
retrieving revision 1.119
diff -w -b -B -u -p -r1.119 Makefile.am
--- Makefile.am 5 Sep 2009 11:09:34 -0000 1.119
+++ Makefile.am 13 Oct 2009 02:16:37 -0000
@@ -255,16 +255,16 @@ sysinfo$(EXEEXT_FOR_BUILD): sysinfo.o sy
 
 syslex.o: syslex.c sysinfo.h config.h
  if [ -r syslex.c ]; then \
-  $(CC_FOR_BUILD) -c -I. $(AM_CFLAGS) $(CFLAGS_FOR_BUILD) syslex.c -Wno-error ; \
+  $(CC_FOR_BUILD) -c -I. $(AM_CFLAGS) $(CFLAGS_FOR_BUILD) -Wno-error syslex.c ; \
  else \
-  $(CC_FOR_BUILD) -c -I. -I$(srcdir) $(AM_CFLAGS) $(CFLAGS_FOR_BUILD)
$(srcdir)/syslex.c -Wno-error ;\
+  $(CC_FOR_BUILD) -c -I. -I$(srcdir) $(AM_CFLAGS) $(CFLAGS_FOR_BUILD)
-Wno-error $(srcdir)/syslex.c ;\
  fi
 
 sysinfo.o: sysinfo.c
  if [ -r sysinfo.c ]; then \
-  $(CC_FOR_BUILD) -c -I. $(AM_CFLAGS) $(CFLAGS_FOR_BUILD) sysinfo.c -Wno-error ; \
+  $(CC_FOR_BUILD) -c -I. $(AM_CFLAGS) $(CFLAGS_FOR_BUILD) -Wno-error sysinfo.c ; \
  else \
-  $(CC_FOR_BUILD) -c -I. $(AM_CFLAGS) $(CFLAGS_FOR_BUILD) $(srcdir)/sysinfo.c
-Wno-error ; \
+  $(CC_FOR_BUILD) -c -I. $(AM_CFLAGS) $(CFLAGS_FOR_BUILD) -Wno-error
$(srcdir)/sysinfo.c ; \
  fi

--
           Summary: compiler option added after operands
           Product: binutils
           Version: 2.21 (HEAD)
            Status: NEW
          Severity: normal
          Priority: P2
         Component: binutils
        AssignedTo: unassigned at sources dot redhat dot com
        ReportedBy: jerker dot back at gmail dot com
                CC: bug-binutils at gnu dot org
 GCC build triplet: x86_64-unknown-interix6.1
  GCC host triplet: x86_64-unknown-interix6.1
GCC target triplet: x86_64-unknown-interix6.1


http://sourceware.org/bugzilla/show_bug.cgi?id=10764

------- 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@...
http://lists.gnu.org/mailman/listinfo/bug-binutils

[Bug binutils/10764] compiler option added after operands

by Bugzilla from sourceware-bugzilla@sourceware.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


------- Additional Comments From cvs-commit at gcc dot gnu dot org  2009-10-16 14:09 -------
Subject: Bug 10764

CVSROOT: /cvs/src
Module name: src
Changes by: nickc@... 2009-10-16 14:08:58

Modified files:
        binutils       : ChangeLog Makefile.am Makefile.in

Log message:
        PR 10764
        * Makefile.am (sysinfo.o, syslex.o): Place source file after all
        command line switches.
        * Makefile.in: Regenerate.

Patches:
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/binutils/ChangeLog.diff?cvsroot=src&r1=1.1547&r2=1.1548
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/binutils/Makefile.am.diff?cvsroot=src&r1=1.119&r2=1.120
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/binutils/Makefile.in.diff?cvsroot=src&r1=1.140&r2=1.141



--


http://sourceware.org/bugzilla/show_bug.cgi?id=10764

------- 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@...
http://lists.gnu.org/mailman/listinfo/bug-binutils

[Bug binutils/10764] compiler option added after operands

by Bugzilla from sourceware-bugzilla@sourceware.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


------- Additional Comments From nickc at redhat dot com  2009-10-16 14:09 -------
Hi Jerker,

  Thanks for reporting this problem and for providing a patch.  I have checked
it in along with the following changelog entry.

Cheers
  Nick

binutils/ChangeLog
2009-10-16  Jerker Bäck <jerker.back@...>

        PR 10764
        * Makefile.am (sysinfo.o, syslex.o): Place source file after all
        command line switches.
        * Makefile.in: Regenerate.


--
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


http://sourceware.org/bugzilla/show_bug.cgi?id=10764

------- 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@...
http://lists.gnu.org/mailman/listinfo/bug-binutils