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

2011-10-20 Thread Federico Cavalieri
> > Connection dropped... > > > > Do you know why document nodes have a component less than any other kind of > > node when they are the root of their tree? > > > > I finally found out what you meant (the code is in the OrdPathNode ctor, not > the OrdPath ctor). > I think the reason why element no

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

2011-10-20 Thread Markos Zaharioudakis
Carlos, let's try to finis this today and commit to the main trunk. So, please add a test for w3c collections, and also update the ChangeLog file with a one-line description of the change. -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 Your team Zorba Coders i

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

2011-10-20 Thread Matthias Brantner
One question: Why is the iterator not generated using the runtime code generation? -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba

[Zorba-coders] [Bug 855715] Re: Invalid escaped characters in regex not caught

2011-10-20 Thread Paul J. Lucas
** Changed in: zorba Status: In Progress => Fix Released -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/855715 Title: Invalid escaped characters in regex not caught Status in Zorba - The

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

2011-10-20 Thread Federico Cavalieri
> Federico, I think the documentation in node-position.xq needs some more work. > I can do this, but first we must agree on what is exactly the functionality > and the assumptions about node positions that you rely on. For example, in the > documentation of the np:node-position() function, you writ

Re: [Zorba-coders] [Merge] lp:~juan457/zorba/xqxq-api-changes into lp:zorba

2011-10-20 Thread Juan Zacarias
Done I changed the getExternalVariables to the staticContext. -- https://code.launchpad.net/~juan457/zorba/xqxq-api-changes/+merge/79589 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Un

[Zorba-coders] [Merge] lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module

2011-10-20 Thread Juan Zacarias
Juan Zacarias has proposed merging lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module. Requested reviews: Chris Hillery (ceejatec) For more details, see: https://code.launchpad.net/~juan457/zorba/email-module-bugfixes/+merge/79984 Added missing dependency files. -- https://co

[Zorba-coders] [Bug 862091] Re: Files missing from email module

2011-10-20 Thread Juan Zacarias
Just requested a merge to add those 2 files to the email module -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/862091 Title: Files missing from email module Status in Zorba - The XQuery Process

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

2011-10-20 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 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/~z

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

2011-10-20 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 -- https://code.launchpad.net/~zorba-coders/zorba/collecti

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

2011-10-20 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/collection-counting/+merge/79765 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://lau

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

2011-10-20 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 -- https://code.launchpad.net/~zorba-coders/zorba/collecti

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

2011-10-20 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 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/~z

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

2011-10-20 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 -- https://code.launchpad.net/~zorba-coders/zorba/collecti

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

2011-10-20 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/collection-counting/+merge/79765 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorb

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

2011-10-20 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 -- https://code.launchpad.net/~zorba-coders/zorba/collecti

Re: [Zorba-coders] [Merge] lp:~juan457/zorba/xqxq-api-changes into lp:zorba

2011-10-20 Thread Markos Zaharioudakis
Ok, it looks better now, but we are not done yet First, the comment in include/zorba/static_context.h for the getExternalVariables() method is wrong. The method returns the qnames of all the external variables that are in-scope within that static context. It's not about whether the variable

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

2011-10-20 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 -- https://code.launchpad.net/~zorba-coders/zorba/collecti

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

2011-10-20 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/collection-counting/+merge/79765 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://lau

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

2011-10-20 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 -- https://code.launchpad.net/~zorba-coders/zorba/collecti

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

2011-10-20 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 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/~z

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

2011-10-20 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 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/~z

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

2011-10-20 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 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/~z

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

2011-10-20 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 -- https://code.launchpad.net/~zorba-coders/zorba/collecti

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

2011-10-20 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/collection-counting-2011-10-20T23-16-14.372Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 Your team Zorba Coders is subscribed to branch lp:zorba. --

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

2011-10-20 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 -- https://code.launchpad.net/~zorba-coders/zorba/collecti

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

2011-10-20 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:272 (message): Validation queue job collection-counting-2011-10-20T23-16-14.372Z is finis

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

2011-10-20 Thread Markos Zaharioudakis
> > Federico, I think the documentation in node-position.xq needs some more > work. > > I can do this, but first we must agree on what is exactly the functionality > > and the assumptions about node positions that you rely on. For example, in > the > > documentation of the np:node-position() functi

[Zorba-coders] [Bug 862091] Re: Files missing from email module

2011-10-20 Thread Chris Hillery
I'll review the merge proposal for the email module, thanks. Juan, please do a quick check of the other modules for similar problems - eg., is FindImageMagick.cmake in the right place? It might also be worth taking a quick look at what FindFoo.cmake scripts are still in Zorba core and deciding whe

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

2011-10-20 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 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/~z

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

2011-10-20 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 -- https://code.launchpad.net/~zorba-coders/zorba/collecti

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

2011-10-20 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/collection-counting-2011-10-21T00-49-13.645Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 Your team Zorba Coders is subscribed to branch lp:zorba. --

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

2011-10-20 Thread Zorba Build Bot
Validation queue job collection-counting-2011-10-21T00-49-13.645Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchp

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

2011-10-20 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 -- https://code.launchpad.net/~zorba-coders/zorba/collection-cou

Re: [Zorba-coders] [Merge] lp:~juan457/zorba/xqxq-api-changes into lp:zorba

2011-10-20 Thread Juan Zacarias
> Second, to get the external variables from all the modules that participate in > a query, we also need a getExternalVariables() at the XQueryImpl level. This > method will go through all the static context objs in > theCompilerCB->theSctxMap and compute the union of the external variables > retur

Re: [Zorba-coders] [Merge] lp:~juan457/zorba/xqxq-api-changes into lp:zorba

2011-10-20 Thread Markos Zaharioudakis
> > Second, to get the external variables from all the modules that participate > in > > a query, we also need a getExternalVariables() at the XQueryImpl level. This > > method will go through all the static context objs in > > theCompilerCB->theSctxMap and compute the union of the external variabl