Hi,

On Sa, 2007-02-17 at 11:42 +0100, Christian Stimming wrote:
> Am Samstag, 17. Februar 2007 02:20 schrieb Andreas Köhler:
> > src/libs/aqbanking/banking.c::AB_Banking_FindWizard suggests that it
> > fills a GWEN_BUFFER with the complete absolute path to where the wizard
> > should be found.  But it does not seem to care about file name
> > extensions because it returns
> >
> > c:/soft/aqbanking/lib/aqbanking/plugins/16\\wizards\\qt3-wizard
> > instead of
> > c:\\soft\\aqbanking\\lib\\aqbanking\\plugins\\16\\wizards\\qt3-wizard.exe
> >
> > I am sure the forward slashes can be ignored though.
> 
> Thanks for spotting this one. Does the attached patch help?

Yes, but in the end I used attached patch.  Thanks.

-- andi5

Index: src/libs/aqbanking/banking.c
===================================================================
--- src/libs/aqbanking/banking.c	(Revision 1178)
+++ src/libs/aqbanking/banking.c	(Arbeitskopie)
@@ -3831,6 +3831,9 @@
                                  AB_WIZARD_FOLDER
                                  DIRSEP);
         GWEN_Buffer_AppendString(pbuf, name);
+	/* For windows, we need the exe extension as well */
+	if (strlen(EXEEXT) > 0)
+	  GWEN_Buffer_AppendString(pbuf, EXEEXT);
 	GWEN_PluginDescription_List2Iterator_free(pit);
 	GWEN_PluginDescription_List2_freeAll(pl);
 	return 0;
@@ -3881,6 +3884,9 @@
                                    AB_WIZARD_FOLDER
                                    DIRSEP);
           GWEN_Buffer_AppendString(pbuf, name);
+	  /* For windows, we need the exe extension as well */
+	  if (strlen(EXEEXT) > 0)
+	    GWEN_Buffer_AppendString(pbuf, EXEEXT);
           free(s);
 	  GWEN_PluginDescription_List2Iterator_free(pit);
 	  GWEN_PluginDescription_List2_freeAll(pl);
Index: src/libs/aqbanking/Makefile.am
===================================================================
--- src/libs/aqbanking/Makefile.am	(Revision 1178)
+++ src/libs/aqbanking/Makefile.am	(Arbeitskopie)
@@ -11,6 +11,7 @@
 # $(datadir) et al are not available.
 localedir=$(datadir)/locale
 DEFS += \
+ -DEXEEXT=\"$(EXEEXT)\" \
  -DLOCALEDIR=\"$(localedir)\" \
  -DAQBANKING_PLUGINS=\"$(aqbanking_plugindir)\" \
  -DAQBANKING_SYSCONF_DIR=\"$(sysconfdir)\" \
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Aqbanking-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/aqbanking-devel

Reply via email to