public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
From: rmoseley@sourceware.org
To: frysk-cvs@sourceware.org
Subject: [SCM]  master: Fix bz #6576, fix load command not setting path-to-executable correctly.
Date: Thu, 29 May 2008 19:25:00 -0000	[thread overview]
Message-ID: <20080529192541.31537.qmail@sourceware.org> (raw)

The branch, master has been updated
       via  c809d4bc691f791e7c875d841ad3e26ccc7e1d4b (commit)
      from  fe7c57831ee38bef5c6157ba384627587e97da39 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email.

- Log -----------------------------------------------------------------
commit c809d4bc691f791e7c875d841ad3e26ccc7e1d4b
Author: Rick Moseley <rmoseley@dhcp-215.hsv.redhat.com>
Date:   Thu May 29 14:23:50 2008 -0500

    Fix bz #6576, fix load command not setting path-to-executable correctly.
    
    * LoadCommand.java: Fix invalid setting of
      path-to-executable.
    * TestRunCommand.java: Fix bz #6576 that adds test
      for change to LoadCommand above.

-----------------------------------------------------------------------

Summary of changes:
 frysk-core/frysk/hpd/ChangeLog           |    5 +++++
 frysk-core/frysk/hpd/LoadCommand.java    |    4 +++-
 frysk-core/frysk/hpd/TestRunCommand.java |   11 ++---------
 3 files changed, 10 insertions(+), 10 deletions(-)

First 500 lines of diff:
diff --git a/frysk-core/frysk/hpd/ChangeLog b/frysk-core/frysk/hpd/ChangeLog
index 5f65fca..7716747 100644
--- a/frysk-core/frysk/hpd/ChangeLog
+++ b/frysk-core/frysk/hpd/ChangeLog
@@ -15,6 +15,11 @@
 	* TestStartCommand.java: Add tests for above changes.
 	* TestDisplayCommand.java: Fix message testing.
 	* TestFhpdStepping.java: Ditto.
+	
+	* LoadCommand.java: Fix invalid setting of 
+	path-to-executable.
+	* TestRunCommand.java: Fix bz #6576 that adds test
+	for change to LoadCommand above.
 
 2008-05-28  Teresa Thomas  <tthomas@redhat.com>
 
diff --git a/frysk-core/frysk/hpd/LoadCommand.java b/frysk-core/frysk/hpd/LoadCommand.java
index ad65dc5..845ca45 100644
--- a/frysk-core/frysk/hpd/LoadCommand.java
+++ b/frysk-core/frysk/hpd/LoadCommand.java
@@ -153,8 +153,10 @@ public class LoadCommand extends ParameterizedCommand {
 	synchronized (cli) {
 	    cli.loadedProcs.put(new Integer(procID), 
 		    exeProc.getExeFile().getSysRootedPath());
-	    if (params != null)
+	    if (params != null) {
+		params[0] = exeProc.getExeFile().getSysRootedPath();
 		cli.ptsetParams.put(new Integer(procID), params);
+	    }
 	    else {
 		String[] command = { exeProc.getExeFile().getSysRootedPath() };
 		cli.ptsetParams.put(new Integer(procID), command);
diff --git a/frysk-core/frysk/hpd/TestRunCommand.java b/frysk-core/frysk/hpd/TestRunCommand.java
index 625724c..e324e1d 100644
--- a/frysk-core/frysk/hpd/TestRunCommand.java
+++ b/frysk-core/frysk/hpd/TestRunCommand.java
@@ -281,15 +281,8 @@ public class TestRunCommand extends TestLib {
     }
     
     public void testFhpdLoadedParams() {
-	if(unresolved(6576))
-	    return;
 	
-	/*String[] command = new String[] {Prefix.pkgLibFile("funit-parameters").getPath(),
-					 "zzz",
-					 "yyy"};
-    	e = new HpdTestbed(command);
-	//e = new HpdTestbed(Prefix.pkgLibFile("funit-parameters").getPath() + " zzz yyy", 
-	//	"Loaded executable file.*funit-parameters.*");
+    	e = HpdTestbed.load("funit-parameters zzz yyy");
 	e.sendCommandExpectPrompt("run", "running.*zzz yyy.*" +
 		"Attached to process ([0-9]+).*" + "Running process ([0-9]+).*");
 	try { Thread.sleep(500); } catch (Exception e) {}
@@ -300,6 +293,6 @@ public class TestRunCommand extends TestLib {
 		"Attached to process ([0-9]+).*" + "Running process ([0-9]+).*");
 	e.send("quit\n");
 	e.expect("Quitting\\.\\.\\.");
-	e.close(); */
+	e.close();
     }
 }
\ No newline at end of file


hooks/post-receive
--
frysk system monitor/debugger


                 reply	other threads:[~2008-05-29 19:25 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080529192541.31537.qmail@sourceware.org \
    --to=rmoseley@sourceware.org \
    --cc=frysk-cvs@sourceware.org \
    --cc=frysk@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).