[PATCH v2] All FreeBSD targets are ELF except i?86-*-freebsd\[12\].*.

[PATCH v2] All FreeBSD targets are ELF except i?86-*-freebsd\[12\].*.

[Ed Maste]

FreeBSD has used ELF for all targets for more than 15 years. Add
*-*-freebsd* except i?86-*-freebsd\[12\].* to is_elf_format. Update
is_aout_format to include a trailing dot, so that it does not match
freebsd-10.0 and later.
---
Updates from v1: added trailing . after [12], and fix whitespace in
ChangeLog.

 binutils/testsuite/ChangeLog               | 6 ++++++
 binutils/testsuite/lib/binutils-common.exp | 7 ++++++-
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/binutils/testsuite/ChangeLog b/binutils/testsuite/ChangeLog
index 7d0b551..6324821 100644
--- a/binutils/testsuite/ChangeLog
+++ b/binutils/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2015-05-20  Ed Maste  <emaste@freebsd.org>
+
+	* lib/binutils-common.exp (is_elf_format): Include *-*-freebsd* except
+	  i?86-*-freebsd\[12\].*.
+	* (is_aout_format): Update FreeBSD to i?86-*-freebsd\[12\].*.
+
 2015-05-18  Nick Clifton  <nickc@redhat.com>
 
 	* binutils-all/dw2-3.S: Replace .int with .4byte.
diff --git a/binutils/testsuite/lib/binutils-common.exp b/binutils/testsuite/lib/binutils-common.exp
index 5dadc42..c01b493 100644
--- a/binutils/testsuite/lib/binutils-common.exp
+++ b/binutils/testsuite/lib/binutils-common.exp
@@ -36,12 +36,17 @@ proc is_elf_format {} {
 	 && ![istarget tic6x*-*-uclinux*]
 	 && ![istarget *-*-irix5*]
 	 && ![istarget *-*-irix6*]
+	 && ![istarget *-*-freebsd*]
 	 && ![istarget *-*-netbsd*]
 	 && ![istarget *-*-openbsd*]
 	 && ![istarget *-*-solaris2*] } {
 	return 0
     }
 
+    if { [istarget i?86-*-freebsd\[12\].*] } {
+        return 0
+    }
+
     if { [istarget *-*-linux*aout*]
 	 || [istarget *-*-linux*ecoff*]
 	 || [istarget *-*-linux*oldld*]
@@ -92,7 +97,7 @@ proc is_aout_format {} {
 	 || [istarget arm-*-netbsd*]
 	 || [istarget arm-*-openbsd*]
 	 || [istarget arm-*-riscix*]
-	 || [istarget i?86-*-freebsd\[12\]*]
+	 || [istarget i?86-*-freebsd\[12\].*]
 	 || [istarget i?86-*-netbsd*]
 	 || [istarget i?86-*-openbsd\[0-2\]*]
 	 || [istarget i?86-*-openbsd3.\[0-2\]*]
-- 
2.2.2

Re: [PATCH v2] All FreeBSD targets are ELF except i?86-*-freebsd\[12\].*.

[Alan Modra]

On Mon, May 25, 2015 at 12:12:12PM -0400, Ed Maste wrote:
> FreeBSD has used ELF for all targets for more than 15 years. Add
> *-*-freebsd* except i?86-*-freebsd\[12\].* to is_elf_format. Update
> is_aout_format to include a trailing dot, so that it does not match
> freebsd-10.0 and later.
> ---
> Updates from v1: added trailing . after [12], and fix whitespace in
> ChangeLog.

I committed this for you.

-- 
Alan Modra
Australia Development Lab, IBM

Re: [PATCH v2] All FreeBSD targets are ELF except i?86-*-freebsd\[12\].*.

[Ed Maste]

On 26 May 2015 at 09:59, Alan Modra <amodra@gmail.com> wrote:
> On Mon, May 25, 2015 at 12:12:12PM -0400, Ed Maste wrote:
>> FreeBSD has used ELF for all targets for more than 15 years. Add
>> *-*-freebsd* except i?86-*-freebsd\[12\].* to is_elf_format. Update
>> is_aout_format to include a trailing dot, so that it does not match
>> freebsd-10.0 and later.
>> ---
>> Updates from v1: added trailing . after [12], and fix whitespace in
>> ChangeLog.
>
> I committed this for you.

Thank you, and thanks for the ChangeLog update (I wasn't sure how best
to express it in ChangeLog style).