[Zorba-coders] [Merge] lp:~zorba-coders/zorba/xqxq-url-resolver into lp:zorba/xqxq-module

2012-10-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/xqxq-url-resolver into lp:zorba/xqxq-module has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/123604 -- https://code.launchpad.net/~zorba-coders/zorba

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/zorba-xqxq-url-resolver into lp:zorba

2012-10-17 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/zorba-xqxq-url-resolver-2012-10-18T06-48-00.823Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602 Your team Zorba Coders is subscribed to branch lp:zor

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/xqxq-url-resolver into lp:zorba/xqxq-module

2012-10-17 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/xqxq-url-resolver into lp:zorba/xqxq-module 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 xqxq-url-resolver-2012-10-18T06-21-40.181Z is

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/xqxq-url-resolver into lp:zorba/xqxq-module

2012-10-17 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/xqxq-url-resolver-2012-10-18T06-21-40.181Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/123604 Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-modu

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/zorba-xqxq-url-resolver into lp:zorba

2012-10-17 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/zorba-xqxq-url-resolver into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602 -- https://code.launchpad.net/~zorba-coders/zorba

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/xqxq-url-resolver into lp:zorba/xqxq-module

2012-10-17 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/xqxq-url-resolver into lp:zorba/xqxq-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/123604 -- https://code.launchpad.net/~zorba-coders/zorba

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/xqxq-url-resolver into lp:zorba/xqxq-module

2012-10-17 Thread Chris Hillery
Review: Approve I initially saw some bad memory hits when I did a build of this branch as well, but after merging the latest trunk and doing a clean build, it all looks good. I do not see the memory leaks Sorin noted; I ran the url- tests with valgrind, and the only memory leaks reported were t

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1001474 into lp:zorba

2012-10-17 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug1001474 into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1001474/+merge/126116 -- https://code.launchpad.net/~zorba-coders/zorba/bug1001474/+merge/126116 Your t

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1001474 into lp:zorba

2012-10-17 Thread Zorba Build Bot
Validation queue job bug1001474-2012-10-17T23-38-54.575Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug1001474/+merge/126116 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-cod

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1058231 into lp:zorba

2012-10-17 Thread Chris Hillery
Ah, OK, thanks for the explanation. As for whether this should be done cross-platform: 1. There is no way that we have discovered to determine the path to zorba_simplestore (or zorba itself) on Linux or Mac, so there is no equivalent problem to be solved there (yet?). 2. It would be best if th

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1058231 into lp:zorba

2012-10-17 Thread Rodolfo Ochoa
1- Because Zorba Processor is not zorba.exe, is zorba_simplestore.dll, so, when you make a call from Java, the actual process is c:\...\java.exe calling zorba_simplestore among other dll's, so the paths come from java.exe. 2- Same principle will apply here, when Zorba is called from other langua

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/zorba-xqxq-url-resolver into lp:zorba

2012-10-17 Thread Chris Hillery
Review: Approve It's an ugly hack, for sure, but I don't see a better alternative without backwards-incompatible API changes. I'll file a new bug for a better solution in 3.0. -- https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602 Your team Zorba Coders is subsc

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1001474 into lp:zorba

2012-10-17 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug1001474-2012-10-17T23-38-54.575Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug1001474/+merge/126116 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: htt

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1001474 into lp:zorba

2012-10-17 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1001474 into lp:zorba has been updated. Description changed to: - Fix for PHP looking library - Also fix for python_d.lib error For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1001474/+merge/126116 -- https://code.launchp

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1001474 into lp:zorba

2012-10-17 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1001474 into lp:zorba has been updated. Commit Message changed to: - Fix for PHP looking library - Also fix for python_d.lib error For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1001474/+merge/126116 -- https://code.laun

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug867693 into lp:zorba/image-module

2012-10-17 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug867693 into lp:zorba/image-module has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug867693/+merge/126578 -- https://code.launchpad.net/~zorba-coders/zorba/bug867693/+merge/126

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug867693 into lp:zorba/image-module

2012-10-17 Thread Zorba Build Bot
Validation queue job bug867693-2012-10-17T23-00-39.162Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug867693/+merge/126578 Your team Zorba Coders is subscribed to branch lp:zorba/image-module. -- Mailing list: https://launchpad.net

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1001474 into lp:zorba

2012-10-17 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug1001474 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1001474/+merge/126116 -- https://code.launchpad.net/~zorba-coders/zorba/bug1001474/+merge/126116

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1001474 into lp:zorba

2012-10-17 Thread Chris Hillery
Review: Approve I would prefer this to be two separate commits, because it contains two completely unrelated changes (the DownloadModules overwrite change and the python fix). However, they're both small, so if you set the commit message to clearly mention both of them I guess it's OK. -- http

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1058231 into lp:zorba

2012-10-17 Thread Chris Hillery
Review: Needs Information I'm assuming that the main fix here works. However I'm a bit confused why it matters whether we find the path to zorba.exe or zorba_simplestore.dll, because as far as I know they should always be in the same directory. So to satisfy my curiosity, why is this change nec

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug867693 into lp:zorba/image-module

2012-10-17 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug867693-2012-10-17T23-00-39.162Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug867693/+merge/126578 Your team Zorba Coders is subscribed to branch lp:zorba/image-module. -- Mailin

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug867693 into lp:zorba/image-module

2012-10-17 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug867693 into lp:zorba/image-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug867693/+merge/126578 -- https://code.launchpad.net/~zorba-coders/zorba/bug867693/+mer

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug867693 into lp:zorba/image-module

2012-10-17 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug867693/+merge/126578 Your team Zorba Coders is subscribed to branch lp:zorba/image-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.ne

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread David Graf
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/coll-iter-state-init/+merge/130055 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] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/coll-iter-state-init/+merge/130055 -- https://code.launchpad.net/~zorba-coders/zorba/coll-

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 1 Approve, 1 Pending. -- https://code.launchpad.net/~zorba-coders/zorba/coll-iter-state-init/+merge/130055 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread Zorba Build Bot
Validation queue job coll-iter-state-init-2012-10-17T18-29-41.949Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/coll-iter-state-init/+merge/130055 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://laun

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/coll-iter-state-init/+merge/130055 -- https://code.launchpad.net/~zorba-coders/zorba/coll-

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/coll-iter-state-init-2012-10-17T18-29-41.949Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/coll-iter-state-init/+merge/130055 Your team Zorba Coders is subscribed to branch lp:zorba. -

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/coll-iter-state-init/+merge/130055 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/

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1001474 into lp:zorba

2012-10-17 Thread Rodolfo Ochoa
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug1001474/+merge/126116 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-cod

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread David Graf
The proposal to merge lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba has been updated. Commit Message changed to: Init ZorbaCollectionIteratorState::theIteratorOpened properly. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/coll-iter-state-init/+merge/130055 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread David Graf
David Graf has proposed merging lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba. Commit message: Init ZorbaCollectionIteratorState::theIteratorOpened properly. Requested reviews: David Graf (davidagraf) Matthias Brantner (matthias-brantner) For more details, see: https://code.laun

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread David Graf
The proposal to merge lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba has been updated. Commit Message changed to: Init ZorbaCollectionIteratorState::theIteratorOpened properly. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/coll-iter-state-init/+merge/130040 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba

2012-10-17 Thread David Graf
David Graf has proposed merging lp:~zorba-coders/zorba/coll-iter-state-init into lp:zorba. Commit message: Init ZorbaCollectionIteratorState::theIteratorOpened properly. Requested reviews: David Graf (davidagraf) Matthias Brantner (matthias-brantner) For more details, see: https://code.laun