Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 315227

Summary: Conflicting keybinding for Ctrl-p
Product: [Tools] Linux Tools Reporter: Andrew Overholt <overholt>
Component: SystemtapAssignee: anithra <anithra>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: pmuldoon, sgehwolf
Version: unspecified   
Target Milestone: 0.6   
Hardware: PC   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
Change keybindings to Ctrl+Shift F5/F6 instead of Ctrl+P none

Description Andrew Overholt CLA 2010-06-01 10:34:36 EDT
Ctrl-p is print but we're trying to use it for "Modify Parsing Expression".  We shouldn't do this.  Please choose another keybinding or just don't have one.
Comment 1 Severin Gehwolf CLA 2010-06-01 15:44:22 EDT
Created attachment 170686 [details]
Change keybindings to Ctrl+Shift F5/F6 instead of Ctrl+P
Comment 2 Severin Gehwolf CLA 2010-06-01 15:45:21 EDT
BTW. this is a project level patch as you might have figured ;-)
Comment 3 Severin Gehwolf CLA 2010-06-01 17:36:08 EDT
Silly me. It's a project level patch on org.eclipse.linuxtools.systemtap.ui.graphicalrun.
Comment 4 Andrew Overholt CLA 2010-06-02 09:35:57 EDT
Anithra, what do you think of the proposed new bindings?
Comment 5 anithra CLA 2010-06-02 13:43:07 EDT
(In reply to comment #4)
> Anithra, what do you think of the proposed new bindings?

That will work, Thanks!
Comment 6 Roland Grunberg CLA 2010-06-02 13:54:17 EDT
Fixed in Revision 24944. A non-submitted patch was available to use.
Comment 7 Andrew Overholt CLA 2010-06-02 14:06:30 EDT
This *just* missed our RC3 submission but we can verify with a nightly build from later today or tomorrow.
Comment 8 Andrew Overholt CLA 2010-06-04 17:33:26 EDT
Verified with our N201006041201 build.
Comment 9 Andrew Overholt CLA 2010-06-04 17:33:34 EDT
Verifying.