Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [PATCH] [configure] Added "nfp" to the build for binutils.
@ 2018-04-30 18:21 Francois H. Theron
  2018-04-30 19:52 ` Joel Brobecker
  2018-05-01 15:28 ` Nick Clifton
  0 siblings, 2 replies; 5+ messages in thread
From: Francois H. Theron @ 2018-04-30 18:21 UTC (permalink / raw)
  To: gcc-patches; +Cc: binutils, gdb-patches

[-- Attachment #1: Type: text/plain, Size: 327 bytes --]


* Excludes: ld gas gdb gprof sim
* For https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=fe944acf8f858cfe6bcfd00670a88847a464717c

Signed-off-by: Francois H. Theron <francois.theron@netronome.com>
---
 ChangeLog    | 6 ++++++
 configure    | 4 ++++
 configure.ac | 4 ++++
 3 files changed, 14 insertions(+)


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-configure-Added-nfp-to-the-build-for-binutils.patch --]
[-- Type: text/x-patch; name="0001-configure-Added-nfp-to-the-build-for-binutils.patch", Size: 1272 bytes --]

diff --git a/ChangeLog b/ChangeLog
index 03e1852114..edb69ed8b5 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2018-04-30  Francois H. Theron <francois.theron@netronome.com>
+
+	* config.sub: Added "nfp" to basic_machine list.
+	* configure.ac: Added "nfp" target.
+	* configure: Regenerate.
+
 2018-02-13  Maciej W. Rozycki  <macro@mips.com>
 
 	* configure.ac <wasm32-*-*> (noconfigdirs): Add `ld'.
diff --git a/configure b/configure
index 0601395512..69c99e99cc 100755
--- a/configure
+++ b/configure
@@ -3777,6 +3777,10 @@ case "${target}" in
   mt-*-*)
     noconfigdirs="$noconfigdirs sim"
     ;;
+  nfp-*-*)
+    noconfigdirs="$noconfigdirs ld gas gdb gprof sim"
+    noconfigdirs="$noconfigdirs $target_libraries"
+    ;;
   powerpc-*-aix*)
     # copied from rs6000-*-* entry
     noconfigdirs="$noconfigdirs gprof"
diff --git a/configure.ac b/configure.ac
index c343333652..a1edc369a2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1108,6 +1108,10 @@ case "${target}" in
   mt-*-*)
     noconfigdirs="$noconfigdirs sim"
     ;;
+  nfp-*-*)
+    noconfigdirs="$noconfigdirs ld gas gdb gprof sim"
+    noconfigdirs="$noconfigdirs $target_libraries"
+    ;;
   powerpc-*-aix*)
     # copied from rs6000-*-* entry
     noconfigdirs="$noconfigdirs gprof"

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] [configure] Added "nfp" to the build for binutils.
  2018-04-30 18:21 [PATCH] [configure] Added "nfp" to the build for binutils Francois H. Theron
@ 2018-04-30 19:52 ` Joel Brobecker
  2018-05-01 11:41   ` Francois H. Theron
  2018-05-01 15:28 ` Nick Clifton
  1 sibling, 1 reply; 5+ messages in thread
From: Joel Brobecker @ 2018-04-30 19:52 UTC (permalink / raw)
  To: Francois H. Theron; +Cc: gcc-patches, binutils, gdb-patches

> +2018-04-30  Francois H. Theron <francois.theron@netronome.com>
> +
> +	* config.sub: Added "nfp" to basic_machine list.
> +	* configure.ac: Added "nfp" target.
> +	* configure: Regenerate.

I am not a maintainer, but I noticed that config.sub is not being
modified by this commit -- a small discrepancy between the patch
and the propose ChangeLog entry.

>  2018-02-13  Maciej W. Rozycki  <macro@mips.com>
>  
>  	* configure.ac <wasm32-*-*> (noconfigdirs): Add `ld'.
> diff --git a/configure b/configure
> index 0601395512..69c99e99cc 100755
> --- a/configure
> +++ b/configure
> @@ -3777,6 +3777,10 @@ case "${target}" in
>    mt-*-*)
>      noconfigdirs="$noconfigdirs sim"
>      ;;
> +  nfp-*-*)
> +    noconfigdirs="$noconfigdirs ld gas gdb gprof sim"
> +    noconfigdirs="$noconfigdirs $target_libraries"
> +    ;;
>    powerpc-*-aix*)
>      # copied from rs6000-*-* entry
>      noconfigdirs="$noconfigdirs gprof"
> diff --git a/configure.ac b/configure.ac
> index c343333652..a1edc369a2 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1108,6 +1108,10 @@ case "${target}" in
>    mt-*-*)
>      noconfigdirs="$noconfigdirs sim"
>      ;;
> +  nfp-*-*)
> +    noconfigdirs="$noconfigdirs ld gas gdb gprof sim"
> +    noconfigdirs="$noconfigdirs $target_libraries"
> +    ;;
>    powerpc-*-aix*)
>      # copied from rs6000-*-* entry
>      noconfigdirs="$noconfigdirs gprof"


-- 
Joel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] [configure] Added "nfp" to the build for binutils.
  2018-04-30 19:52 ` Joel Brobecker
@ 2018-05-01 11:41   ` Francois H. Theron
  0 siblings, 0 replies; 5+ messages in thread
From: Francois H. Theron @ 2018-05-01 11:41 UTC (permalink / raw)
  To: Joel Brobecker; +Cc: gcc-patches, binutils, gdb-patches

On 30 April 2018 at 21:52, Joel Brobecker <brobecker@adacore.com> wrote:
>> +2018-04-30  Francois H. Theron <francois.theron@netronome.com>
>> +
>> +     * config.sub: Added "nfp" to basic_machine list.
>> +     * configure.ac: Added "nfp" target.
>> +     * configure: Regenerate.
>
> I am not a maintainer, but I noticed that config.sub is not being
> modified by this commit -- a small discrepancy between the patch
> and the propose ChangeLog entry.

That was a mistake when splitting the diff to have config.sub
separate. For the record, it has been accepted here after the
maintainer graciously added a ChangeLog entry and a test-case:
http://git.savannah.gnu.org/cgit/config.git/commit/?id=d23f148145b62e7dea44581097f60358118f87f8

For this patch, I can resubmit a new patch with the config.sub line
removed from ChangeLog or ask that the maintainer removes this one
line when committing. Or if I misunderstood the contribution process I
can add the now accepted config.sub to the patch and not change the
ChangeLog.

Francois


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] [configure] Added "nfp" to the build for binutils.
  2018-04-30 18:21 [PATCH] [configure] Added "nfp" to the build for binutils Francois H. Theron
  2018-04-30 19:52 ` Joel Brobecker
@ 2018-05-01 15:28 ` Nick Clifton
  2018-05-01 15:32   ` Francois H. Theron
  1 sibling, 1 reply; 5+ messages in thread
From: Nick Clifton @ 2018-05-01 15:28 UTC (permalink / raw)
  To: Francois H. Theron, gcc-patches; +Cc: binutils, gdb-patches

Hi Francois,

> 2018-05-01  Francois H. Theron  <francois.theron@netronome.com>
> 
> 	* configure.ac: Added "nfp" target.
> 	* configure: Regenerate.

I have applied this change patch to both the gcc and binutils mainline sources.

Cheers
  Nick


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] [configure] Added "nfp" to the build for binutils.
  2018-05-01 15:28 ` Nick Clifton
@ 2018-05-01 15:32   ` Francois H. Theron
  0 siblings, 0 replies; 5+ messages in thread
From: Francois H. Theron @ 2018-05-01 15:32 UTC (permalink / raw)
  To: Nick Clifton; +Cc: gcc-patches, binutils, gdb-patches

Hi

Thank you. The last thing would be config.sub, do I submit the
accepted patch for it here as well or will it propagate from
config.git?

Francois


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-05-01 15:32 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-30 18:21 [PATCH] [configure] Added "nfp" to the build for binutils Francois H. Theron
2018-04-30 19:52 ` Joel Brobecker
2018-05-01 11:41   ` Francois H. Theron
2018-05-01 15:28 ` Nick Clifton
2018-05-01 15:32   ` Francois H. Theron

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox