From: Joseph B. <jb...@um...> - 2005-09-24 03:33:08
|
I've applied your suggestions and committed them to CVS. Thanks again! On Fri, 23 Sep 2005, Francesco Montorsi wrote: > Hi, > > > Joseph Blough wrote: >> The bakefile is databaselayer.bkl at >> http://cvs.sourceforge.net/viewcvs.py/wxcode/wxCode/components/databaselayer/build/databaselayer.bkl > yes; sorry I forgot to specify the -d flag in cvs update... now I see the > files > > >> Now that I think about it more, maybe the tests_firebird.cpp file should be >> added to CVS (which would eliminate the issue completely). The main reason >> it isn't in CVS is that everything I've read indicates that it's best to >> leave script generated files out of the source control system. > generally I agree, even if I do an exception for my generated makefiles which > I do keep in CVS... but that's not required. > > >> Unfortunately, it this situation is requires the user to have cxxtest >> installed. > In this case assing tests_firebird.cpp to the CVS seems to be a good choice > since AFAIK cxxtest is not very common on user's workstations... > > > >> Thanks for looking at this! > I have two little advices looking at your BKL: > > 1) many settings are repetead in the <lib> and <dll> targets of the same > database source; this means that when you want to change something you need > to do it twice... a better approach could be declaring a <template> from > which your <lib> and <dll> derive. > > 2) instead of the current definition of MSVC6_MERGED_TARGETS I would use: > > <set var="MSVC6PRJ_MERGED_TARGETS"> > databaselayer_postgresql=databaselayer_postgresql_lib+databaselayer_postgresql_dll > databaselayer_mysql=databaselayer_mysql_lib+databaselayer_mysql_dll > databaselayer_firebird=databaselayer_firebird_lib+databaselayer_firebird_dll > databaselayer_sqlite=databaselayer_sqlite_lib+databaselayer_sqlite_dll > </set> > > which make msvc6prj nicer ;) > > > >> I'll try to get wxspellchecker and shortcutpanel under bakefiles also after >> the next databaselayer release. > Good ! > I think it will help users who want to try out wxcode components. > Please let me know if you find something is missing from > wxcode\build\bakefiles\readme.txt and from > wxcode\build\bakefiles\empty.bkl.template... > > > Francesco Montorsi > > > > >> >> >> On Thu, 22 Sep 2005, Francesco Montorsi wrote: >> >>> Hi, >>> >>> Joseph Blough wrote: >>> >>>> I've almost got the databaselayer component working with Bakefile, but >>>> I'm having problems running the perl script that generates the unit >>>> tests. The line that needs to be run is: >>>> >>>> cxxtestgen.pl -o tests_firebird.cpp MyTest.h >>>> >>>> and I tried wrapping it in an action module like >>>> >>>> <action id="tests_firebird.cpp"> >>>> <command>cxxtestgen.pl -o tests_firebird.cpp MyTest.h</command> >>>> </action> >>> >>> yes, this is how to do it; the only suspect thing is the id of the >>> <action> tag: in this way it would create a target in makefiles named >>> "tests_firebird.cpp"; I think this could give some problem to bakefile... >>> see below >>> >>> >>>> but when I try to call the target through >>>> >>>> <depends>tests_firebird.cpp</depends> >>> >>> I'm not sure to understand what do you mean by "I try to call the target >>> through": where did you put that <depends> tag ? >>> I suggest you to use the <depends-on-file> tag instead. >>> >>> I cannot see any bakefile in the "databaselayer" folder of my wxCode >>> repository; if you commit them to the CVS I can give you more details... >>> >>> >>> >>>> the following is spit out by GNU make >>>> >>>> No rule to make target `tests_firebird.cpp' >>> >>> how does your GNU makefile looks like ? >>> I think the problem is that <action> tag creates a target which is not >>> exactly named "tests_firebird.cpp" (probably looks like >>> "sub_tests_firebird.cpp"). >>> >>> Francesco Montorsi >>> >>> >>> ------------------------------------------------------- >>> SF.Net email is sponsored by: >>> Tame your development challenges with Apache's Geronimo App Server. >>> Download >>> it for free - -and be entered to win a 42" plasma tv or your very own >>> Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php >>> _______________________________________________ >>> wxCode-users mailing list >>> wxC...@li... >>> https://lists.sourceforge.net/lists/listinfo/wxcode-users >>> >>> >>> >> >> >> ------------------------------------------------------- >> SF.Net email is sponsored by: >> Tame your development challenges with Apache's Geronimo App Server. >> Download >> it for free - -and be entered to win a 42" plasma tv or your very own >> Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php >> _______________________________________________ >> wxCode-users mailing list >> wxC...@li... >> https://lists.sourceforge.net/lists/listinfo/wxcode-users >> > > > > ------------------------------------------------------- > SF.Net email is sponsored by: > Tame your development challenges with Apache's Geronimo App Server. Download > it for free - -and be entered to win a 42" plasma tv or your very own > Sony(tm)PSP. Click here to play: http://sourceforge.net/geronimo.php > _______________________________________________ > wxCode-users mailing list > wxC...@li... > https://lists.sourceforge.net/lists/listinfo/wxcode-users > > > |