public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
* [Bug testsuite/6821] New: incorrect or missing interpreter command in some of the examples
@ 2008-08-06 12:51 eugen at debian dot org
  2008-08-06 12:54 ` [Bug testsuite/6821] " eugen at debian dot org
  2008-08-11 10:29 ` mwielaard at redhat dot com
  0 siblings, 2 replies; 3+ messages in thread
From: eugen at debian dot org @ 2008-08-06 12:51 UTC (permalink / raw)
  To: systemtap

Some of the examples and samples do not specify interpreter command or contain
"#! stap" that is not valid command.

-- 
           Summary: incorrect or missing interpreter command in some of the
                    examples
           Product: systemtap
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: testsuite
        AssignedTo: systemtap at sources dot redhat dot com
        ReportedBy: eugen at debian dot org


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

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug testsuite/6821] incorrect or missing interpreter command in some of the examples
  2008-08-06 12:51 [Bug testsuite/6821] New: incorrect or missing interpreter command in some of the examples eugen at debian dot org
@ 2008-08-06 12:54 ` eugen at debian dot org
  2008-08-11 10:29 ` mwielaard at redhat dot com
  1 sibling, 0 replies; 3+ messages in thread
From: eugen at debian dot org @ 2008-08-06 12:54 UTC (permalink / raw)
  To: systemtap


------- Additional Comments From eugen at debian dot org  2008-08-06 12:52 -------
Created an attachment (id=2895)
 --> (http://sourceware.org/bugzilla/attachment.cgi?id=2895&action=view)
patch

This patch adds/fixes interpreter command for some samples and examples. It
does nothing for files that require additional parameters for stap (like -g).

-- 


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

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug testsuite/6821] incorrect or missing interpreter command in some of the examples
  2008-08-06 12:51 [Bug testsuite/6821] New: incorrect or missing interpreter command in some of the examples eugen at debian dot org
  2008-08-06 12:54 ` [Bug testsuite/6821] " eugen at debian dot org
@ 2008-08-11 10:29 ` mwielaard at redhat dot com
  1 sibling, 0 replies; 3+ messages in thread
From: mwielaard at redhat dot com @ 2008-08-11 10:29 UTC (permalink / raw)
  To: systemtap


------- Additional Comments From mwielaard at redhat dot com  2008-08-11 10:28 -------
Thanks for the report. You are right, those scripts should be marked executable
and have a proper interpreter command. Some of the examples have been moved
around. But I did a new scan and added the fix wherever appropriate. Also fixed
a small install mistake, where the scripts would get stripped of their
executable bit again on install.

commit a81c61ac1947cb5c13d3a758fc793a119b88b4e0
Author: Mark Wielaard <mwielaard@redhat.com>
Date:   Mon Aug 11 12:24:47 2008 +0200

    PR2895. Add proper #! /usr/bin/env stap line. Make example scripts executable.

commit 5b8561dccfd6d79184c044cec6bd7d01ef415acc
Author: Mark Wielaard <mwielaard@redhat.com>
Date:   Mon Aug 11 12:11:26 2008 +0200

    Use INSTALL_PROGRAM, not INSTALL_DATA for executable .stp scripts.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


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

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-08-11 10:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-08-06 12:51 [Bug testsuite/6821] New: incorrect or missing interpreter command in some of the examples eugen at debian dot org
2008-08-06 12:54 ` [Bug testsuite/6821] " eugen at debian dot org
2008-08-11 10:29 ` mwielaard at redhat dot com

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