public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Iain Sandoe <developer@sandoe-acoustics.co.uk>
To: binutils Development <binutils@sourceware.org>
Cc: Tristan Gingold <gingold@adacore.com>
Subject: [Patch Darwin/testsuite/objcopy] Do not try to run the copy-2/3.d tests for darwin.
Date: Fri, 30 Dec 2011 21:07:00 -0000	[thread overview]
Message-ID: <91B188DC-F39A-4AE5-9B23-8F0ECF1EF160@sandoe-acoustics.co.uk> (raw)

trying to work through the test-suite to find which fails represent  
stuff we still need to implement (and which are noise).

AFAICS, these two tests are not applicable to mach-o, certainly as  
they stand (they would require a different .s file to work).

OK?
Iain

binutils/testsuite:

	* binutils-all/copy-2.d: Do not run for Darwin.
	* binutils-all/copy-3.d: Likewise.

  binutils/testsuite/binutils-all/copy-2.d |    2 +-
  binutils/testsuite/binutils-all/copy-3.d |    2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/binutils/testsuite/binutils-all/copy-2.d b/binutils/ 
testsuite/binutils-all/copy-2.d
index 5636ff6..4640709 100644
--- a/binutils/testsuite/binutils-all/copy-2.d
+++ b/binutils/testsuite/binutils-all/copy-2.d
@@ -3,7 +3,7 @@
  #objcopy: --set-section-flags foo=contents,alloc,load,code
  #name: copy with setting section flags 2
  #source: copytest.s
-#not-target: *-*-*aout *-*-aix* h8300-*-* hppa*-*-hpux* m68k-*-netbsd  
m68k-*-openbsd* mips-sony-bsd* mips-*-ultrix* mips-*-osf* mips-*- 
ecoff* mips-*-irix mips-*-irix[2-4] mips-*-riscos* ns32k-*-netbsd
+#not-target: *-*-*aout *-*-aix* h8300-*-* hppa*-*-hpux* m68k-*-netbsd  
m68k-*-openbsd* mips-sony-bsd* mips-*-ultrix* mips-*-osf* mips-*- 
ecoff* mips-*-irix mips-*-irix[2-4] mips-*-riscos* ns32k-*-netbsd *-*- 
darwin*
  # Note - we use copytest.s and a section named "foo" rather
  # than .text because for some file formats (eg PE) the .text
  # section has a fixed set of flags and these cannot be changed.
diff --git a/binutils/testsuite/binutils-all/copy-3.d b/binutils/ 
testsuite/binutils-all/copy-3.d
index 551517c..2d42ce8 100644
--- a/binutils/testsuite/binutils-all/copy-3.d
+++ b/binutils/testsuite/binutils-all/copy-3.d
@@ -3,7 +3,7 @@
  #objcopy: --set-section-flags .text=alloc,data
  #name: copy with setting section flags 3
  #source: bintest.s
-#not-target: *-*-*aout *-*-*pe *-*-*coff hppa*-*-hpux* i*86-*-cygwin*  
i*86-*-mingw32* m68k-*-netbsd m68k-*-openbsd* ns32k-*-netbsd x86_64-*- 
mingw*
+#not-target: *-*-*aout *-*-*pe *-*-*coff hppa*-*-hpux* i*86-*-cygwin*  
i*86-*-mingw32* m68k-*-netbsd m68k-*-openbsd* ns32k-*-netbsd x86_64-*- 
mingw* *-*-darwin*
  # The .text # section in PE/COFF has a fixed set of flags and these
  # cannot be changed.  We skip it for them.


             reply	other threads:[~2011-12-30 21:07 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-30 21:07 Iain Sandoe [this message]
2012-01-02 11:27 ` Tristan Gingold
2012-01-16 23:31   ` Alan Modra

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=91B188DC-F39A-4AE5-9B23-8F0ECF1EF160@sandoe-acoustics.co.uk \
    --to=developer@sandoe-acoustics.co.uk \
    --cc=binutils@sourceware.org \
    --cc=gingold@adacore.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).