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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in cmake_modules/ZorbaModule.cmake
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-18T07-13-04.737Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The attempt to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274
(message):
Validation queue job module-schema-tools-2012-04-18T07-13-04.737Z
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-18T08-39-07.752Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The attempt to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274
(message):
Validation queue job module-schema-tools-2012-04-18T08-39-07.752Z
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-18T09-15-58.572Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
Your team Zorba Coders is subscribed to branch 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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
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 :
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
Review: Approve
Think I finally got the dependency tracking working. It's ugly but at least
works.
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Attempt to merge into lp:zorba failed due to conflicts:
contents conflict in NOTICE.txt
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-18T15-58-07.886Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Validation queue job module-schema-tools-2012-04-18T15-58-07.886Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Hooray!
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
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 :
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema
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/102001
--
https://code.launchpad.net/~zorba-coders/zorba/module
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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 :
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-17T15-15-59.439Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The attempt to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274
(message):
Validation queue job module-schema-tools-2012-04-17T15-15-59.439Z
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/102001
--
https://code.launchpad.net/~zorba-coders/zorba/module
Chris, can you check the cmake build dependencies for util-jvm module. I see
the same error on my box when I use make -j12.
[ 95%] Built target modlib69_xqxq.xq
[ 95%] Built target modlib56_hmac.xq
Scanning dependencies of target modlib70_util-jvm.xq
[ 95%] Scanning dependencies of target
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/102001
--
https://code.launchpad.net/~zorba-coders/zorba/module
There are additional revisions which have not been approved in review. Please
seek review and approval of these new revisions.
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
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
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/102001
--
https://code.launchpad.net/~zorba-coders/zorba/module
Review: Approve
Fixed erroneous merging of swig files.
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-17T19-20-05.192Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The attempt to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274
(message):
Validation queue job module-schema-tools-2012-04-17T19-20-05.192Z
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/102377
--
https://code.launchpad.net/~zorba-coders/zorba/module
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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 :
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in ChangeLog
text conflict in src/api/staticcontextimpl.h
text conflict in src/precompiled/stdafx.h
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101796
Your team Zorba Coders is subscribed to
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/101796
--
https://code.launchpad.net/~zorba-coders/zorba/module
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.
All modules with Java dependency should use util-jvm module to the way
schema-tools
Review: Approve
I merged from the trunk, but I wasn't totally sure that I did the right thing
with stdafx.h. So I re-submitted the proposal so Matthias and Cezar could have
a chance to re-review.
As far as I can tell, stdafx.h was never explicitly changed on the
module-schema-tools branch;
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/102001
--
https://code.launchpad.net/~zorba-coders/zorba/module
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-14T07-27-03.74Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
Your team Zorba Coders is subscribed to branch 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/102001
--
https://code.launchpad.net/~zorba-coders/zorba/module
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/102001
--
https://code.launchpad.net/~zorba-coders/zorba/module
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-14T08-25-03.504Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
Your team Zorba Coders is subscribed to branch lp:zorba.
--
There is a feature in CMake that DECLARE_ZORBA_JAR(..TARGET..) depends on
which, it turns out, was only implemented correctly as of CMake 2.8.4. (See
http://public.kitware.com/Bug/view.php?id=11332 .) The remote queue was running
CMake 2.8.2, which is why the build failed.
I have upgraded the
The attempt to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274
(message):
Validation queue job module-schema-tools-2012-04-14T08-25-03.504Z
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/102001
--
https://code.launchpad.net/~zorba-coders/zorba/module
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/102001
--
https://code.launchpad.net/~zorba-coders/zorba/module
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-14T08-42-58.716Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The attempt to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274
(message):
Validation queue job module-schema-tools-2012-04-14T08-42-58.716Z
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/102001
--
https://code.launchpad.net/~zorba-coders/zorba/module
Grumble. Sometimes I hate CMake. This stuff works just fine on my machine, but
for whatever reason the dependencies just aren't working out right on the
remote queue machine. I've spent the last hour trying to make it work right and
have so far failed. I'll have to take another crack at it
Unless when you updated cmake you updated swig too, no other idea comes to mind.
I have cmake 2.8.5 and swig 1.3.40 on my box and works fine.
That doesn't have anything to do with my CMake
changes, and will have to be solved as well. Any ideas?
--
Review: Needs Information
Should the classpath in the static context be serialized? I forgot our
decision, is this member needed at all?
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101796
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
It looks like the classpath in static_context is orphaned; nothing sets it or
references it. I'll take that out.
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101796
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Removed. Matthias, if that was your only comment, go ahead and vote Approve.
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101796
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101796
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 :
I'll need Cezar to set the commit message; I'm not sure if he just copied
the Description or not. Unfortunately if you go back to the previous
revision of the Proposal it doesn't show the commit message from that
version...
--
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.
All modules with Java dependency should use util-jvm module to the way
schema-tools
I've set the commit message, but build is not triggered, do I need to resubmit
the proposal?
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101796
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to
There are additional revisions which have not been approved in review. Please
seek review and approval of these new revisions.
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101796
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
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/101796
--
https://code.launchpad.net/~zorba-coders/zorba/module
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/101796
--
https://code.launchpad.net/~zorba-coders/zorba/module
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/101796
--
https://code.launchpad.net/~zorba-coders/zorba/module
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101796
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 :
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101796
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 :
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/101639
--
https://code.launchpad.net/~zorba-coders/zorba/module
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101639
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 :
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/101639
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 :
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
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/98485
--
https://code.launchpad.net/~zorba-coders/zorba/module
Removed IStream swig class and references.
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/98485
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
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-03-20T19-01-23.283Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/98485
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1,
Needs Fixing 1, Pending 1. Got: 3 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/98485
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
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/98485
--
https://code.launchpad.net/~zorba-coders/zorba/module
Validation queue job module-schema-tools-2012-03-20T19-01-23.283Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/98485
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
If you take a look at the file there are many places where this strange
indentation occurs. Check out the top of the file: THIS IS A GENERATED FILE.
DO NOT EDIT!
My understanding is that this isn't true anymore, though. Markos removed the
script to generate this file, and said that in future
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-03-19T16-20-06.136Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/98224
Your team Zorba Coders is subscribed to branch 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/98224
--
https://code.launchpad.net/~zorba-coders/zorba/module
I didn't know Markos removed the scripts after I made the changes. But I
removed the comment and I fixed the indentation of init_val lines.
I also fixed the ExternalModules.config wrong merge, and checked
src/system/zorba_properties.h contains changes only related to CLASSPATH and
indentation.
Validation queue job module-schema-tools-2012-03-19T16-20-06.136Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/98224
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
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/98224
--
https://code.launchpad.net/~zorba-coders/zorba/module
I fixed all of the comments, execept:
- the ones related to swig:
- the changes are not required, right now but they required for optimizing
XQJ, to make swig play nice with streams it's quite a chlange, would be a petty
to loose them
- I don't think various.i should be specified in
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/97976
--
https://code.launchpad.net/~zorba-coders/zorba/module
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
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba
has been updated.
Commit Message changed to:
Merge in branch implementing jvm classpath and fix managing dynamic libraries.
Add java classpath to zorbacmd and to Zorba API.
Fix dynamic libraries loading.
Tested
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/97978
--
https://code.launchpad.net/~zorba-coders/zorba/module
Validation queue job module-schema-tools-2012-03-16T20-15-10.551Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97978
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1,
Needs Fixing 1, Pending 1. Got: 3 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97976
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
There are additional revisions which have not been approved in review. Please
seek review and approval of these new revisions.
--
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/97978
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
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/97978
--
https://code.launchpad.net/~zorba-coders/zorba/module
Review: Needs Fixing
- I think you reverted the changes that have been made in
modules/ExternalModules.conf.
- The file src/system/zorba_properties.h also contains changes that have been
reverted. For example,
1058---no-copy-optim\napply the no-copy optimization (1=enabled (default),
Review: Needs Fixing
It looks like you didn't fix the indentation of calls to init_val() in
zorbacmdproperties_base.h, and in fact it looks like you moved a number of
other instances of calls to that method onto the ends of earlier lines of code
in zorba_properties.h. It's just not good
Review: Needs Fixing
- In DynamicLoader::loadModule the code to create the Module is redundant and
should be factorized
- Why is DECLARE_ZORBA_JAR in cmake_modules/ZorbaModule.cmake commented out?
Should it be removed?
- Why did the file src/store/naive/store_properties.h change? I think Markos
1 - 100 of 106 matches
Mail list logo