Opened 23 months ago

Closed 23 months ago

Last modified 23 months ago

#1482 closed enhancement (duplicate)

OS X debugger xterm and Terminal.app

Reported by: akoya Owned by: tome, bennie
Priority: trivial Milestone:
Component: OSX Version: 0.96
Keywords: debugger osx Cc:

Description

I am running universal perll 5.12.4 compiled with macports on OS X 10.5.

On my installation of Padre 0.96 the debugger only works if I start Padre from the terminal and
X11 is properly configured to run.

When I build a package for padre as a standalone bundle with all dependencies and start the debugger, padre will not find the xterm and hang:

29/01/13 2:08:43 [0x0-0x3ab3ab].org.perlide.padre sh: xterm: command not found


Attached is a patch over 0.96 that works on 10.5 for me, not tested on other OS versions. The path for xterm should be the same for other versions...

Maybe there is a better solution that accounts for other system() calls in padre...

This code has changed in trunk but it does not seem to work for me.
In trunk xterm has been substituted by Terminal.app and this should be a much nicer solution.

However the code used to launch Terminal app in trunk requires Applescript which may introduce additional hassle. I have attached another patch to accomplish the same using open and tempfile as a suggestion.

I believe tnis is for tome

Thanks

Attachments (2)

xterm.path.osx.patch (398 bytes) - added by akoya 23 months ago.
open.osx.terminal.app.patch (726 bytes) - added by akoya 23 months ago.

Download all attachments as: .zip

Change History (5)

Changed 23 months ago by akoya

Changed 23 months ago by akoya

comment:1 Changed 23 months ago by bowtie

I think this is fixed in trunk, please check

http://padre.perlide.org/trac/wiki/Development

comment:2 Changed 23 months ago by bowtie

yes i just checked trunk

line 2924 Padre/Wx/Main?.pm

sub run_command {

see - http://padre.perlide.org/trac/browser/trunk/Padre/lib/Padre/Wx/Main.pm

Also please use the Patch Tool in Padre ( Editor - Patch )

your patch files are missing full file names

see example - http://padre.perlide.org/trac/attachment/ticket/1285/diff.2.patch

note full name of file, as we have some files with same name in different directories

please summit all patches against current trunk, thanks

Version 1, edited 23 months ago by bowtie (previous) (next) (diff)

comment:3 Changed 23 months ago by bowtie

  • Priority changed from major to trivial
  • Resolution set to duplicate
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.