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

2012-08-14 Thread Chris Hillery
Review: Needs Information My only concern is that for non-Apple, non-Windows platforms, you add -luuid to the link line, but there's no CMake stuff to search for a uuid library or to ensure that such an option will compile/link successfully. Is it a guarantee that it will always be available,

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

2012-08-14 Thread David Graf
Matthias, can you please review it again? -- https://code.launchpad.net/~davidagraf/zorba/virtual_destr_auditing/+merge/118584 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:~davidagraf/zorba/fn_trace_jsoniq_fix into lp:zorba

2012-08-14 Thread David Graf
David Graf has proposed merging lp:~davidagraf/zorba/fn_trace_jsoniq_fix into lp:zorba. Requested reviews: David Graf (davidagraf) Till Westmann (tillw) For more details, see: https://code.launchpad.net/~davidagraf/zorba/fn_trace_jsoniq_fix/+merge/119478 Because the jsoniq_emitter

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1034582-macports-flex into lp:zorba

2012-08-14 Thread David Graf
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1034582-macports-flex/+merge/119458 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:~davidagraf/zorba/fn_trace_jsoniq_fix into lp:zorba

2012-08-14 Thread David Graf
The proposal to merge lp:~davidagraf/zorba/fn_trace_jsoniq_fix into lp:zorba has been updated. Commit Message changed to: Because the jsoniq_emitter aggregates the xml_emitter, it needs to handover all the parameters. Inclusive aEmitAttributes. For more details, see:

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

2012-08-14 Thread David Graf
Review: Approve -- https://code.launchpad.net/~davidagraf/zorba/fn_trace_jsoniq_fix/+merge/119478 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-1034582-macports-flex into lp:zorba

2012-08-14 Thread David Graf
The proposal to merge lp:~zorba-coders/zorba/bug-1034582-macports-flex into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1034582-macports-flex/+merge/119458 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1034582-macports-flex into lp:zorba

2012-08-14 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1034582-macports-flex-2012-08-14T08-26-05.466Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1034582-macports-flex/+merge/119458 Your team Zorba Coders is subscribed to branch

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

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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1034582-macports-flex into lp:zorba

2012-08-14 Thread Zorba Build Bot
Validation queue job bug-1034582-macports-flex-2012-08-14T08-26-05.466Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1034582-macports-flex/+merge/119458 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1034582-macports-flex into lp:zorba

2012-08-14 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug-1034582-macports-flex into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1034582-macports-flex/+merge/119458 --

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

2012-08-14 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2012-08-14T08-56-03.744Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/119482 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

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

2012-08-14 Thread Zorba Build Bot
Validation queue job markos-scratch-2012-08-14T08-56-03.744Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/119482 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-08-14 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/119482 --

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

2012-08-14 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727 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-899364 into lp:zorba

2012-08-14 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/bug-899364 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727 -- https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727

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

2012-08-14 Thread Zorba Build Bot
Attempt to merge into lp:zorba failed due to conflicts: text conflict in src/compiler/translator/translator.cpp -- https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

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

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

2012-08-14 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/bug-899364 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727 -- https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727

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

2012-08-14 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/bug-899364/+merge/118727 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

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

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

2012-08-14 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/bug-899364 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727 -- https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727

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

2012-08-14 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-899364-2012-08-14T10-22-45.746Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-08-14 Thread Zorba Build Bot
Validation queue job bug-899364-2012-08-14T10-22-45.746Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-08-14 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug-899364 into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727 -- https://code.launchpad.net/~zorba-coders/zorba/bug-899364/+merge/118727 Your

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

2012-08-14 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1024892/+merge/118977 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-1024892 into lp:zorba

2012-08-14 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/bug-1024892 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1024892/+merge/118977 --

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

2012-08-14 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/bug-1024892/+merge/118977 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

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

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

2012-08-14 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/bug-1024892 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1024892/+merge/118977 --

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

2012-08-14 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1024892-2012-08-14T11-38-28.961Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1024892/+merge/118977 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-08-14 Thread Zorba Build Bot
Validation queue job bug-1024892-2012-08-14T11-38-28.961Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1024892/+merge/118977 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-08-14 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug-1024892 into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1024892/+merge/118977 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1024892/+merge/118977

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-library-functions into lp:zorba

2012-08-14 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/jsoniq-library-functions-2012-08-14T13-16-06.204Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347 Your team Zorba Coders is subscribed to branch

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-library-functions into lp:zorba

2012-08-14 Thread Zorba Build Bot
Validation queue job jsoniq-library-functions-2012-08-14T13-16-06.204Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-library-functions into lp:zorba

2012-08-14 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/jsoniq-library-functions into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347 --

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

2012-08-14 Thread David Graf
David Graf has proposed merging lp:~davidagraf/zorba/paging_with_refs into lp:zorba. Requested reviews: Till Westmann (tillw) For more details, see: https://code.launchpad.net/~davidagraf/zorba/paging_with_refs/+merge/119553 --

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

2012-08-14 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~davidagraf/zorba/virtual_destr_auditing/+merge/118584 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:~davidagraf/zorba/virtual_destr_auditing into lp:zorba

2012-08-14 Thread Matthias Brantner
The proposal to merge lp:~davidagraf/zorba/virtual_destr_auditing into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~davidagraf/zorba/virtual_destr_auditing/+merge/118584 --

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

2012-08-14 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/virtual_destr_auditing-2012-08-14T14-50-08.762Z/log.html -- https://code.launchpad.net/~davidagraf/zorba/virtual_destr_auditing/+merge/118584 Your team Zorba Coders is subscribed to branch lp:zorba.

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

2012-08-14 Thread Zorba Build Bot
Validation queue job virtual_destr_auditing-2012-08-14T14-50-08.762Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~davidagraf/zorba/virtual_destr_auditing/+merge/118584 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-08-14 Thread noreply
The proposal to merge lp:~davidagraf/zorba/virtual_destr_auditing into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~davidagraf/zorba/virtual_destr_auditing/+merge/118584 --

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

2012-08-14 Thread Till Westmann
Review: Needs Fixing I'm trying to run: zorba -q 'trace(attribute { a } { 1 }, a)' and that still results in a serialization error [err:SENR0001]: 1: can not serialize attribute node Does this work for you? -- https://code.launchpad.net/~davidagraf/zorba/fn_trace_jsoniq_fix/+merge/119478

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-library-functions into lp:zorba

2012-08-14 Thread Till Westmann
Review: Approve Looks good and works for me. -- https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/memory-leak-array-append into lp:zorba

2012-08-14 Thread Till Westmann
Review: Approve Change looks good. Validated (using valgrind) that the memory that was lost when running test/rbkt/Queries/zorba/jsoniq/arr_append_01.xq is not lost anymore after applying this patch. -- https://code.launchpad.net/~zorba-coders/zorba/memory-leak-array-append/+merge/118960 Your