<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 26, 2014 at 2:45 AM, Chris Johns <span dir="ltr"><<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On 26/05/2014 10:07 am, Hesham Moustafa wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">
On Mon, May 26, 2014 at 12:57 AM, Chris Johns <<a href="mailto:chris@contemporary.net.au" target="_blank">chris@contemporary.net.au</a><br></div><div class="">
<mailto:<a href="mailto:chris@contemporary.net.au" target="_blank">chris@contemporary.<u></u>net.au</a>>> wrote:<br>
<br>
<br>
Just looking at this file I cannot see what has changed ? Why do we<br>
need a special gdb build script ?<br>
<br>
There is a configuration option added to disable linking with libsim.a.<br>
The current port for gdb only support remote debugging with or1ksim<br>
simulator. Can we get it working with if (target==or1k) or something<br>
similar in the original gdb build script?<br>
<br>
</div></blockquote>
<br>
Ah ok. See below..<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">
<br>
@@ -0,0 +1,95 @@<br>
+#<br>
+# GDB 7.xx Version 1.<br>
+#<br>
+# This configuration file configure's, make's and install's gdb.<br>
+#<br>
+<br>
+%include %{_configdir}/checks.cfg<br>
+<br>
+#<br>
+# Select Snapshot Macro Maps<br>
+#<br>
+%select gdb-snapshot<br>
+%select expat-snapshot<br>
+<br>
+#<br>
+# The description.<br>
+#<br></div>
+Name: %{_target}-gdb-%{gdb_version}-<u></u>__%{release}<div class=""><br>
+Summary: GDB v%{gdb_version} for target %{_target} on host<br>
%{_host}<br>
+Version: %{gdb_version}<br>
+Release: %{release}<br></div>
+URL: <a href="http://www.gnu.org/software/__gdb/" target="_blank">http://www.gnu.org/software/__<u></u>gdb/</a><br>
<<a href="http://www.gnu.org/software/gdb/" target="_blank">http://www.gnu.org/software/<u></u>gdb/</a>><br>
+BuildRoot: %{_tmppath}/%{name}-root-%(%{_<u></u>___id_u} -n)<div class=""><br>
+<br>
+#<br>
+# Source<br>
+#<br>
+%source set gdb<br></div>
<a href="http://ftp.gnu.org/gnu/gdb/__gdb-%{gdb_version}.tar.bz2" target="_blank">http://ftp.gnu.org/gnu/gdb/__<u></u>gdb-%{gdb_version}.tar.bz2</a><div class=""><br>
<<a href="http://ftp.gnu.org/gnu/gdb/gdb-%{gdb_version}.tar.bz2" target="_blank">http://ftp.gnu.org/gnu/gdb/<u></u>gdb-%{gdb_version}.tar.bz2</a>><br>
+<br>
+#<br>
+#<br>
+# Prepare the source code.<br>
+#<br>
+%prep<br>
+ build_top=$(pwd)<br>
+<br></div>
+ source_dir_gdb="gdb-%{gdb___<u></u>version}"<div><div class="h5"><br>
+ %source setup gdb -q -n gdb-%{gdb_version}<br>
+ %patch setup gdb -p1<br>
+<br>
+ cd ${build_top}<br>
+<br>
+%build<br>
+ build_top=$(pwd)<br>
+<br>
+ %{build_directory}<br>
+<br>
+ mkdir -p ${build_dir}<br>
+ cd ${build_dir}<br>
+<br>
+ %{host_build_flags}<br>
+<br>
+ if test "%{_build}" != "%{_host}" ; then<br>
+ GDB_LIBS_STATIC="-lexpat"<br>
+ else<br>
+ GDB_LIBS_STATIC="-lexpat"<br>
+ GDB_LIBS="%{_forced_static}"<br>
+ fi<br>
+<br>
+ LIBS_STATIC=${GDB_LIBS_STATIC} \<br>
+ LIBS=${GDB_LIBS} \<br>
+ ../${source_dir_gdb}/configure \<br>
+ --build=%{_build} --host=%{_host} \<br>
+ --target=%{_target} \<br>
+ --verbose --disable-nls \<br>
+ --without-included-gettext \<br>
+ --disable-win32-registry \<br>
+ --disable-werror \<br>
+ --disable-sim \<br>
</div></div></blockquote>
<br>
Is this the change ?<br>
<br>
We could make this conditional and enabled by default then disable it in the bset file. An example in the gcc configuration is:<br>
<br>
<a href="http://git.rtems.org/rtems-source-builder/tree/source-builder/config/gcc-common-1.cfg#n145" target="_blank">http://git.rtems.org/rtems-<u></u>source-builder/tree/source-<u></u>builder/config/gcc-common-1.<u></u>cfg#n145</a></blockquote>
<div>Great! Before submitting the next fixed patch, I would like you give a feedback regarding this change.</div><div>1- I have added a new "%define gdb-disable-sim 1" line to rtems-or1k.bset</div><div>2- Add "%{!?gdb-disable-sim:--enable-sim}%{?gdb-disable-sim:--disable-sim}" to gdb-7-1.cfg (the common gdb file)</div>
<div><br></div><div>Should gdb-disable-sim be defined (with 0 or 1) for all targets? or not defining it is OK (means that it's 0) ? </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<span class=""><font color="#888888"><br>
<br>
Chris<br>
</font></span></blockquote></div><br></div></div>