Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
Review: Needs Fixing Agreed with Matthias's comments. Regarding DECLARE_ZORBA_JAR(), it looks like the commented-out macro is an older version, and the newer version is not commented-out. The older one should just be deleted. Couple more comments: - Do we also need to mention the addition of swig/various.i in NOTICE.xml? It has a comment saying it's from the SWIG package directly. - Wasn't the class name IStream causing a build conflict on Windows? - The call to init_val() in zorbacmdproperties_base.h should be on a separate line. (I know it was cut-and-pasted from above, but ugh.) - Why is the method for retrieving the PropertiesGlobal* called getProperties()? Shouldn't that be getProperiesGlobal()? (Or, IMHO, getGlobalProperties() and rename the class to GlobalProperties.) - Similarly, why is the factory method for PropertiesGlobal Properties::instance()? (The Zorba Properties stuff has always seemed like a mishmash to me; this isn't helping.) - Need to fix the conflict in ExternalModules.conf. - Typo: singelton in zorba.h. -- https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97105 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba has been updated. Commit Message changed to: Add java classpath to zorbacmd and to Zorba API. Fix dynamic libraries loading. Tested it works on Windows with both xsl-fo and xmlbeans modules. All tests are passing. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97105 -- https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97105 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97105 -- https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97105 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
Attempt to merge into lp:zorba failed due to conflicts: text conflict in modules/ExternalModules.conf -- https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97105 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97105 -- https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97105 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba has been updated. Commit Message changed to: Add java classpath to zorbacmd and to Zorba API. Fix dynamic libraries loading. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/96399 -- https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/96399 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp