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

2012-10-18 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

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

2012-10-18 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 --

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

2012-10-18 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 --

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

2012-10-18 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

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

2012-10-18 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

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

2012-10-18 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

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

2012-10-18 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 --

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

2012-10-18 Thread Chris Hillery
Whoops, right - this merge proposal won't pass until https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602 is merged. Ignore these test failures for now. -- https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/123604 Your team Zorba Coders is

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

2012-10-18 Thread Zorba Build Bot
Validation queue job zorba-xqxq-url-resolver-2012-10-18T06-48-00.823Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-10-18 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/zorba-xqxq-url-resolver/+merge/123602 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

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

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

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

2012-10-18 Thread Sorin Marian Nasoi
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602 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 :

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

2012-10-18 Thread Sorin Marian Nasoi
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/123604 Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

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

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

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

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

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

2012-10-18 Thread Zorba Build Bot
Validation queue job zorba-xqxq-url-resolver-2012-10-18T09-14-46.434Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-10-18 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 2 Approve, 1 Pending. -- https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

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

2012-10-18 Thread Matthias Brantner
Review: Needs Information I would really like to avoid this special case and the string comparison on the function call. What about using a function annotation? -- https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602 Your team Zorba Coders is subscribed to branch

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

2012-10-18 Thread Matthias Brantner
Review: Needs Fixing I don't feel comfortable with the QName solution. It would be great if HOFs could be used for this. As far as I understood earlier discussions, this should have been developed together with Nicolae. Using HOFs would also solve the static context problem. Maybe the HOF

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

2012-10-18 Thread Markos Zaharioudakis
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos-scratch into lp:zorba. Commit message: changed some scripting tests to cmply with the current syntax of the scripting spec. Requested reviews: Markos Zaharioudakis (markos-za) For more details, see:

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

2012-10-18 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/130422 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 :

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

2012-10-18 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/markos-scratch into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/130422 --

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

2012-10-18 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2012-10-18T20-19-00.223Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/130422 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

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

2012-10-18 Thread Zorba Build Bot
Validation queue job markos-scratch-2012-10-18T20-19-00.223Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/130422 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-10-18 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/markos-scratch into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/130422 --

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

2012-10-18 Thread Rodolfo Ochoa
Rodolfo Ochoa has proposed merging lp:~zorba-coders/zorba/bug1068323 into lp:zorba. Requested reviews: Juan Zacarias (juan457) Chris Hillery (ceejatec) Related bugs: Bug #1068323 in Zorba: Zorba don't install zorba_api.jar https://bugs.launchpad.net/zorba/+bug/1068323 For more details,

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

2012-10-18 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1068323 into lp:zorba has been updated. Commit Message changed to: This is working on linux very well, seems that cmake don't like quoted variables or file variables if they doesn't exist yet. For more details, see:

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

2012-10-18 Thread Juan Zacarias
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug1068323/+merge/130434 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 :

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

2012-10-18 Thread Chris Hillery
Chris Hillery has proposed merging lp:~zorba-coders/zorba/xqxq-url-resolver into lp:zorba/xqxq-module. Commit message: Bug 903797: add feature to prepare-main-module() to enable URI mappers and URL resolvers written in XQuery. Requested reviews: Chris Hillery (ceejatec) Matthias Brantner

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

2012-10-18 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/xqxq-url-resolver into lp:zorba/xqxq-module has been updated. Commit Message changed to: Bug 903797: add feature to prepare-main-module() to enable URI mappers and URL resolvers written in XQuery. For more details, see:

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

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

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

2012-10-18 Thread Chris Hillery
Review: Disapprove No longer needed with HOF-based XQXQ implementation. -- https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

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

2012-10-18 Thread Chris Hillery
Review: Approve Now using HOF! -- https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460 Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe

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

2012-10-18 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/130460 --

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

2012-10-18 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/xqxq-url-resolver-2012-10-19T01-22-47.205Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460 Your team Zorba Coders is subscribed to branch

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

2012-10-18 Thread Zorba Build Bot
Validation queue job xqxq-url-resolver-2012-10-19T01-22-47.205Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460 Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-module. -- Mailing list:

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

2012-10-18 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 1 Approve, 2 Pending. -- https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460 Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-module. --

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

2012-10-18 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/130460 --

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

2012-10-18 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460 Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :