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: linkBe 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).