From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-14331-listarch-gdb-patches=sourceware.cygnus.com@sources.redhat.com>
Received: (qmail 10874 invoked by alias); 16 Apr 2002 14:31:59 -0000
Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:gdb-patches-subscribe@sources.redhat.com>
List-Archive: <http://sources.redhat.com/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sources.redhat.com>
List-Help: <mailto:gdb-patches-help@sources.redhat.com>, <http://sources.redhat.com/ml/#faqs>
Sender: gdb-patches-owner@sources.redhat.com
Received: (qmail 10855 invoked from network); 16 Apr 2002 14:31:54 -0000
Received: from unknown (HELO cygnus.com) (205.180.83.203)
  by sources.redhat.com with SMTP; 16 Apr 2002 14:31:54 -0000
Received: from localhost.redhat.com (romulus.sfbay.redhat.com [172.16.27.251])
	by runyon.cygnus.com (8.8.7-cygnus/8.8.7) with ESMTP id HAA10066
	for <gdb-patches@sources.redhat.com>; Tue, 16 Apr 2002 07:31:53 -0700 (PDT)
Received: by localhost.redhat.com (Postfix, from userid 469)
	id E05E011436; Tue, 16 Apr 2002 10:31:32 -0400 (EDT)
From: Elena Zannoni <ezannoni@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Message-ID: <15548.13764.336835.513002@localhost.redhat.com>
Date: Tue, 16 Apr 2002 07:31:00 -0000
To: Daniel Jacobowitz <drow@mvista.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: gdb.asm/asm-source.exp creation of arch.inc doesn't work for relative ${srcdir}
In-Reply-To: <20020416010713.A32200@nevyn.them.org>
References: <20020416010713.A32200@nevyn.them.org>
X-SW-Source: 2002-04/txt/msg00552.txt.bz2

Daniel Jacobowitz writes:
 > ${srcdir} in the testsuite is for gdb/testsuite/; but a symbolic link in
 > gdb.asm needs to have an extra '..' if it's an absolute path.  I always
 > configure as '../srcdir/configure', so this breaks for me.
 > 

Ahhh, relative paths! I gave up on those. Sometimes I was seeing odd
testsuite failures due to that. Anyway....

 > Do you think creating the link in objdir/gdb/testsuite/ instead would be a
 > good solution?  I can't think of another portable one.
 > 

How about the following.  Somebody reminded me that there is no
technical reason to use a symlink at all. I tried with relative and
absolute paths, and seems to work fine. Can you try it?

Other files (twice.exp, selftest.exp, list.exp) use this technique,
but copy files in the "testsuite" directory as opposed to
testsuite/gdb.base. (maybe these can be fixed too).

Elena



Index: asm-source.exp
===================================================================
RCS file: /cvs/uberbaum/gdb/testsuite/gdb.asm/asm-source.exp,v
retrieving revision 1.21
diff -u -p -r1.21 asm-source.exp
--- asm-source.exp	8 Apr 2002 00:01:06 -0000	1.21
+++ asm-source.exp	16 Apr 2002 14:27:16 -0000
@@ -87,9 +87,7 @@ set binfile ${objdir}/${subdir}/${testfi
 set src1 ${srcdir}/${subdir}/asmsrc1.s
 set src2 ${srcdir}/${subdir}/asmsrc2.s
 
-set insn_file ${srcdir}/${subdir}/${asm-arch}.inc
-remote_exec build "rm -f ${objdir}/${subdir}/arch.inc"
-remote_exec build "ln -s ${insn_file} ${objdir}/${subdir}/arch.inc"
+remote_download host ${srcdir}/${subdir}/${asm-arch}.inc ${subdir}/arch.inc
 
 if { "${asm-flags}" == "" } {
     #set asm-flags "-Wa,-gstabs,-I${srcdir}/${subdir},-I${objdir}/${subdir}"
@@ -278,3 +276,4 @@ gdb_test "print staticvar" ".* = 5" "loo
 gdb_test "disassem foostatic" ".*<foostatic>:.*End of assembler dump." \
 	"look at static function"
 
+remote_exec build "rm -f ${objdir}/${subdir}/arch.inc"