Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Debugger] Windows port for geany's "debugger" plugin. #83

Closed
wants to merge 1 commit into from
Closed

[Debugger] Windows port for geany's "debugger" plugin. #83

wants to merge 1 commit into from

Conversation

vkochan
Copy link
Contributor

@vkochan vkochan commented Jan 31, 2013

Hi,
This is my 1'st little testable version

@frlan
Copy link
Member

frlan commented Feb 10, 2013

Was anybody able to test? Don't have some suitable Windows-setup for testing.

@frlan
Copy link
Member

frlan commented Mar 29, 2014

As nobody was able to test it so far, this will be postponed to post 1.24

@frlan
Copy link
Member

frlan commented Oct 11, 2014

Any news on this?

@eht16
Copy link
Member

eht16 commented Oct 19, 2014

I had a quick look.

First, I made the following changes to the Waf build system to properly build and install on Windows:

diff --git a/debugger/wscript_build b/debugger/wscript_build
index 0be873f..25d3126 100644
--- a/debugger/wscript_build
+++ b/debugger/wscript_build
@@ -20,12 +20,12 @@
 #
 # $Id: wscript_build 1735 2010-11-09 17:03:40Z eht16 $

-from build.wafutils import build_plugin
+from build.wafutils import build_plugin, target_is_win32
 from waflib.Utils import subst_vars

 name = 'Debugger'

-includes = ['debugger/src', 'debugger/src/cell_renderers', 
+includes = ['debugger/src', 'debugger/src/cell_renderers',
             'debugger/src/xpm']

 libraries = ['VTE', 'UTIL']
@@ -37,6 +37,9 @@ defines=[ subst_vars('DBGPLUG_DATA_DIR="' + plugin_datadir + '"', bld.env) ]
 build_plugin(bld, name, includes=includes, libraries=libraries, defines=defines)

 # Icons
+prefix = '${G_PREFIX}/' if target_is_win32(bld) else ''
+icon_dest = '%s${PKGDATADIR}/debugger' % prefix
+
 start_dir = bld.path.find_dir('img')
-bld.install_files(plugin_datadir, start_dir.ant_glob('*.png'), cwd=start_dir)
-bld.install_files(plugin_datadir, start_dir.ant_glob('*.gif'), cwd=start_dir)
+bld.install_files(icon_dest, start_dir.ant_glob('*.png'), cwd=start_dir)
+bld.install_files(icon_dest, start_dir.ant_glob('*.gif'), cwd=start_dir)
diff --git a/debugger/wscript_configure b/debugger/wscript_configure
index 2d64484..451a176 100644
--- a/debugger/wscript_configure
+++ b/debugger/wscript_configure
@@ -20,13 +20,13 @@
 #
 # $Id: wscript_configure 1735 2010-11-09 17:03:40Z eht16 $

-from build.wafutils import check_cfg_cached
+from build.wafutils import check_cfg_cached, target_is_win32

+if not target_is_win32(conf):
+   check_cfg_cached(conf,
+                    package='vte',
+                    uselib_store='VTE',
+                    args='--cflags --libs')

-check_cfg_cached(conf,
-                 package='vte',
-                 uselib_store='VTE',
-                 args='--cflags --libs')
-
-conf.check_cc(function_name='poll', header_name='poll.h')
-conf.check_cc(function_name='openpty', header_name='pty.h', lib='util', uselib_store='UTIL')
+   conf.check_cc(function_name='poll', header_name='poll.h')
+   conf.check_cc(function_name='openpty', header_name='pty.h', lib='util', uselib_store='UTIL')

After these changes, the plugin built and loaded fine.

Opening a target and setting breakpoints works fine. However, executing the target in the debugger doesn't work. It seems like some command is exectued because a command line window is popping up but it closes very quickly. Except the following messages, nothing is logged:

14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_string_free: assertion `string != NULL' failed
14:39:19: GLib CRITICAL : g_hash_table_destroy: assertion `hash_table != NULL' failed

@frlan
Copy link
Member

frlan commented Mar 10, 2015

@vkochan Can you please review Enrico's suggestions and maybe update your PR?

@vkochan
Copy link
Contributor Author

vkochan commented Mar 11, 2015

Meanwhile I don't have a setup with windows machine, I dropped using windows by default, so I am not
sure if I can help with supporting of this patch.

@frlan
Copy link
Member

frlan commented Mar 13, 2015

Am 11.03.2015 um 08:37 schrieb Vadim Kochan:

Meanwhile I don't have a setup with windows machine, I dropped using
windows by default, so I am not
sure if I can help with supporting of this patch.

OK. I keep it open and maybe somebody is able to adopt it.

b4n added a commit to b4n/geany-plugins that referenced this pull request Feb 9, 2016
@b4n b4n mentioned this pull request Feb 9, 2016
@frlan
Copy link
Member

frlan commented Apr 14, 2024

I will close the MR yet as it's quiet ancient already and nobody seemed to take over.

@frlan frlan closed this Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants