Patchwork [RFA,1/5] New port: CR16: Remove "gdb" from noconfigdirs in configure.ac

login
register
mail settings
Submitter Kaushik Phatak
Date Oct. 5, 2012, 7:22 a.m.
Message ID <C6CA53A2A46BA7469348BDBD663AB65845B36CFD@KCHJEXMB02.kpit.com>
Download mbox | patch
Permalink /patch/189418/
State New
Headers show

Comments

Kaushik Phatak - Oct. 5, 2012, 7:22 a.m.
Hi,
Thanks for the feedback.

> I can't see how adding target-libgloss is relevant to a gdb port.
This was actually a part of some simulator fixes to update nltvals.def
as it was not emulating the correct system calls. 
The Newlib changes have been already checked in and nltvals.def has 
been regenerated in latest build, so I guess the target-libgloss 
will not be necessary.

> config.sub is maintained elsewhere.
Ok, I will send this patch to config-patches at gnu dot org.

Regards,
Kaushik

Updated Patch:
 - Removed "target-libgloss"
 - Removed config.sub from this mailing list

2012-10-05  Kaushik Phatak  <kaushik.phatak@kpitcummins.com>
Changelog
	* configure.ac (noconfigdirs): Remove gdb.
	* configure: Regenerate.

Patch

--- ./gdb_src.orig/configure	2012-06-28 17:20:52.000000000 +0530
+++ ./gdb_src/configure	2012-08-17 16:56:10.000000000 +0530
@@ -3447,7 +3447,7 @@  case "${target}" in
     noconfigdirs="$noconfigdirs target-libgloss gdb"
     ;;
   cr16-*-*)
-    noconfigdirs="$noconfigdirs gdb"
+    noconfigdirs="$noconfigdirs"
     ;;
   d10v-*-*)
     noconfigdirs="$noconfigdirs target-libgloss"
--- ./gdb_src.orig/configure.ac	2012-08-17 23:07:18.000000000 +0530
+++ ./gdb_src/configure.ac	2012-08-17 16:56:17.000000000 +0530
@@ -873,7 +873,7 @@  case "${target}" in
     noconfigdirs="$noconfigdirs target-libgloss gdb"
     ;;
   cr16-*-*)
-    noconfigdirs="$noconfigdirs gdb"
+    noconfigdirs="$noconfigdirs"
     ;;
   d10v-*-*)
     noconfigdirs="$noconfigdirs target-libgloss"