[Bug testsuite/14574] Add test coverage for stapdyn's runtime

dsmith at redhat dot com sourceware-bugzilla@sourceware.org
Fri Sep 21 16:17:00 GMT 2012


http://sourceware.org/bugzilla/show_bug.cgi?id=14574

--- Comment #6 from David Smith <dsmith at redhat dot com> 2012-09-21 16:16:52 UTC ---
Created attachment 6643
  --> http://sourceware.org/bugzilla/attachment.cgi?id=6643
buildok testsuite patch

Here's a patch that modifies testsuite/systemtap.pass1-4/buildok.exp to work
for the default (linux) runtime and the dyninst runtime.

However, now that I've done it, I don't really like it.  There are about 18
lines of shared tcl code, and the rest (225 lines) is runtime-specific tcl
code.  This runtime-specific tcl code sets up kfails for each runtime.

I think it might make more sense to leave buildok.exp alone, and have a
separate dyninst-buildok.exp testcase.  Obviously splitting the testcase makes
it much easier to just test one specific runtime.

Comments welcome.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the Systemtap mailing list