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

2012-09-19 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531

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

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug923686-2012-09-19T06-06-26.347Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug923686 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 bug923686-2012-09-19T06-06-26.347Z is finished. The final

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

2012-09-19 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531

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

2012-09-19 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531

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

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug923686-2012-09-19T06-33-57.964Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug923686 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 bug923686-2012-09-19T06-33-57.964Z is finished. The final

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

2012-09-19 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531

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

2012-09-19 Thread Chris Hillery
Stupid Launchpad problem with the email module. Trying again. -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net

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

2012-09-19 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531

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

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug923686-2012-09-19T08-21-15.404Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug923686 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 bug923686-2012-09-19T08-21-15.404Z is finished. The final

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

2012-09-19 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531

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

2012-09-19 Thread Chris Hillery
Chris Hillery has proposed merging lp:~zorba-coders/zorba/bug-1015964 into lp:zorba. Requested reviews: Chris Hillery (ceejatec) Related bugs: Bug #1015964 in Zorba: Add warning re: SWIG 2.0.7 for PHP binding https://bugs.launchpad.net/zorba/+bug/1015964 For more details, see:

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

2012-09-19 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug-1015964 into lp:zorba has been updated. Commit Message changed to: Add note to doc regarding bug in Swig on some platforms. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137 --

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

2012-09-19 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137 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/bug-1015964 into lp:zorba

2012-09-19 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug-1015964 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137 --

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

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1015964-2012-09-19T09-18-54.055Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread Zorba Build Bot
Validation queue job bug-1015964-2012-09-19T09-18-54.055Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 1 Approve. -- https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug-1015964 into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137 --

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

2012-09-19 Thread Markos Zaharioudakis
Review: Needs Fixing -- https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816 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/feature-pragma_no_copy into lp:zorba

2012-09-19 Thread Markos Zaharioudakis
Why is the processPragma method defined only for static_collections_dml_apply_insert_nodes_last. Should it be defined for the other collection-insert functions as well? ChangeLog needs updating Tiny optimization: if you want to erase the last N entries in a vector, you can just use resize().

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

2012-09-19 Thread Markos Zaharioudakis
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/hashmap into lp:zorba. Requested reviews: Markos Zaharioudakis (markos-za) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/hashmap/+merge/125187 HashMap optimizations --

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

2012-09-19 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/hashmap/+merge/125187 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/bug1052242 into lp:zorba

2012-09-19 Thread Juan Zacarias
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823 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/jsoniq_demo into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/jsoniq_demo into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/jsoniq_demo/+merge/114238 --

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

2012-09-19 Thread Juan Zacarias
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been updated. Commit Message changed to: Removed fn:parse-xml#2 Fix for bug 950621 For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655 --

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

2012-09-19 Thread Juan Zacarias
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655 -- https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655

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

2012-09-19 Thread Juan Zacarias
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655 -- https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655

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

2012-09-19 Thread Juan Zacarias
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655 -- https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655

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

2012-09-19 Thread Juan Zacarias
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655 -- https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655

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

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-950621-2012-09-19T15-33-07.413Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug-950621 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 bug-950621-2012-09-19T15-33-07.413Z is finished. The final

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

2012-09-19 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655 -- https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655

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

2012-09-19 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823 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/bug1052242 into lp:zorba

2012-09-19 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug1052242 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823 -- https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823

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

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug1052242-2012-09-19T16-02-04.637Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/1025564 into lp:zorba has been updated. Status: Merged = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/1025564/+merge/120024 -- https://code.launchpad.net/~zorba-coders/zorba/1025564/+merge/120024 Your team

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

2012-09-19 Thread Chris Hillery
Chris Hillery has proposed merging lp:~zorba-coders/zorba/1025564 into lp:zorba. Requested reviews: Chris Hillery (ceejatec) Matthias Brantner (matthias-brantner) Related bugs: Bug #1025564 in Zorba: Deprecate -f argument to zorbacmd https://bugs.launchpad.net/zorba/+bug/1025564 For more

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/util-jvm-module-bug1006311 into lp:zorba/util-jvm-module

2012-09-19 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/util-jvm-module-bug1006311/+merge/118647 Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

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

2012-09-19 Thread Zorba Build Bot
Validation queue job bug1052242-2012-09-19T16-02-04.637Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug1052242 into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823 -- https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823 Your

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

2012-09-19 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/zorba-2.6 into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/zorba-2.6/+merge/120278 --

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

2012-09-19 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/zorba-2.6/+merge/120278 Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/zorba-2.6. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

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

2012-09-19 Thread Matthias Brantner
Markos, can you please review? -- https://code.launchpad.net/~zorba-coders/zorba/XUDY00021/+merge/93100 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/bug-1044260 into lp:zorba

2012-09-19 Thread Paul J. Lucas
The proposal to merge lp:~zorba-coders/zorba/bug-1044260 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1044260/+merge/125297 --

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

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1044260-2012-09-19T18-38-42.91Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1044260/+merge/125297 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread Paul J. Lucas
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1044260/+merge/125297 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/bug-1044260 into lp:zorba

2012-09-19 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug-1044260 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 bug-1044260-2012-09-19T18-38-42.91Z is finished. The final

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

2012-09-19 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug-1044260 into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1044260/+merge/125297 --

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

2012-09-19 Thread Matthias Brantner
Why is the processPragma method defined only for static_collections_dml_apply_insert_nodes_last. Should it be defined for the other collection-insert functions as well? ChangeLog needs updating Tiny optimization: if you want to erase the last N entries in a vector, you can just use

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

2012-09-19 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/feature-pragma_no_copy into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816 --

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

2012-09-19 Thread Zorba Build Bot
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/feature-pragma_no_copy/+merge/112816 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-09-19 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/feature-pragma_no_copy into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816 --

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

2012-09-19 Thread Markos Zaharioudakis
I am going to approve, but it would be nice if you added an iter test to make sure that the optimization is applied and we won't loose it somehow in the future. To do so, you must first make PrinterVisitor print theNeedToCpy data member of the various ZorbaInsertNodesXXX iterators. This will

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

2012-09-19 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816 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/bug923686 into lp:zorba

2012-09-19 Thread Markos Zaharioudakis
Rodolfo, you will need to fix all the iter tests that fail. The tests fail because the var ids assigned to variables have changed. -- https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Cezar Andrei
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module. Requested reviews: Chris Hillery (ceejatec) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 Add java.library.path

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module has been updated. Commit Message changed to: Add java.library.path property when starting the jvm For more details, see:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-utiljvm-module-2012-09-19T20-32-43.622Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 Your team Zorba Coders is subscribed to branch

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba. Requested reviews: Chris Hillery (ceejatec) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 Make doc comments for createBaser64Binary more explicit on

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has been updated. Commit Message changed to: Make doc comments for createBaser64Binary more explicit on what parameters they expect and what they do. Change, return value to xs_int for getIntValue() method. For more

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Chris Hillery
Review: Needs Fixing You changed the URIs of the test modules back to www.zorba-xquery.com, instead of zorba-tests.28msec.us. That will cause test failures. Make sure you've merged from the trunk recently. -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 Your

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Chris Hillery
(Other than that the changes look fine) -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 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/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Zorba Build Bot
Validation queue job fread-pdf-utiljvm-module-2012-09-19T20-32-43.622Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module. --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Zorba Build Bot
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/fread-pdf-utiljvm-module/+merge/125332 Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-utiljvm-module-2012-09-19T21-04-41.287Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 Your team Zorba Coders is subscribed to branch

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Chris Hillery
You still haven't corrected the URIs of the test modules, though (test/rbkt/modules/CMakeLists.txt). -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
Review: Needs Fixing Let me add the new module. -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Zorba Build Bot
Validation queue job fread-pdf-utiljvm-module-2012-09-19T21-04-41.287Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332 --

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

2012-09-19 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/feature-pragma_no_copy into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816 --

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

2012-09-19 Thread Matthias Brantner
I am going to approve, but it would be nice if you added an iter test to make sure that the optimization is applied and we won't loose it somehow in the future. To do so, you must first make PrinterVisitor print theNeedToCpy data member of the various ZorbaInsertNodesXXX iterators. This will

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

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/feature-pragma_no_copy-2012-09-19T21-42-49.059Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816 Your team Zorba Coders is subscribed to branch

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
The ref to the new module is in, please review it too: lp:zorba/read-pdf-module at https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-the-module . -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 Your team Zorba Coders is subscribed to branch lp:zorba. --

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

2012-09-19 Thread Zorba Build Bot
Validation queue job feature-pragma_no_copy-2012-09-19T21-42-49.059Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental has been updated. Commit Message changed to: Testing doc for new read-pdf module. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356 --

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

2012-09-19 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/feature-pragma_no_copy into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
Back to zorba-tests.28msec.us in test/rbkt/modules/CMakeLists.txt and mention in Changelog. -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 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/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Zorba Build Bot
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/fread-pdf-trunk/+merge/125356 Your team Zorba Coders is requested to review the proposed merge of

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356 Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/fread-pdf-trunk. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Chris Hillery
Review: Approve Great, thanks. -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 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/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Zorba Build Bot
Attempt to merge into lp:zorba failed due to conflicts: text conflict in ChangeLog -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Matthias Brantner
Review: Needs Fixing It looks nice (without being able to testing it ;-) - The module should follow XQuery coding conventions. For example, no camel-case but function and variable names using dashes. - The read-pdf:renderToImages-internal function is not documented. This will probably cause

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

2012-09-19 Thread Matthias Brantner
Review: Needs Fixing Works great. Some minor issues: - the --help documentation should mention that the argument is deprecated - also, the --help documentation should mention the syntax to pass the file - please respect coding guidelines. For example, if(qBeforeF()){ = if (qBeforeF()) { - remove

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
I fixed all the comments with the exception of the optional occurrence indicator for internal functions. I tested it with empty sequence and works fine. -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 Your team Zorba Coders is subscribed to branch lp:zorba. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-09-20T03-03-43.261Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338 Your team Zorba Coders is subscribed to branch lp:zorba. --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Zorba Build Bot
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/fread-pdf-trunk/+merge/125356 Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/fread-pdf-trunk.

  1   2   >