Re: [Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-863730 into lp:zorba

2011-10-03 Thread Markos Zaharioudakis
Matthias, I did not look at those tests to see how important they are. But, if they do test something important, rather than changing the expected result, I think you should modify the tests to use an ordered collection, and add one or two new tests for the error condition. --

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

2011-10-03 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874 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:~markos-za/zorba/markos-bugs into lp:zorba

2011-10-03 Thread Markos Zaharioudakis
The proposal to merge lp:~markos-za/zorba/markos-bugs into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874 -- https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874 Your

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

2011-10-03 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/markos-bugs-2011-10-03T08-56-11.421Z/log.html -- https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2011-10-03 Thread Markos Zaharioudakis
Markos Zaharioudakis has proposed merging lp:~markos-za/zorba/markos-bugs into lp:zorba. Requested reviews: Markos Zaharioudakis (markos-za) For more details, see: https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874 --

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

2011-10-03 Thread Zorba Build Bot
Validation queue job markos-bugs-2011-10-03T08-56-11.421Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2011-10-03 Thread noreply
The proposal to merge lp:~markos-za/zorba/markos-bugs into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874 -- https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874 Your team

Re: [Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-863730 into lp:zorba

2011-10-03 Thread Matthias Brantner
According to Launchpad, the change I suggested has not been made - why the new Merge Proposal? (Also confused by Launchpad's saying that the Approved Revision is no longer in the source branch...) Strange, I can see the following changes in the diff: === removed file

Re: [Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-863730 into lp:zorba

2011-10-03 Thread Chris Hillery
Review: Approve Ah, I see the deleted files now. So, I'll approve the diff. However, I'm not sure the remote queue will pick it up with the no longer in the source branch message for Approved Revision. You may need to re-Approve the Merge Proposal. --

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-863730 into lp:zorba

2011-10-03 Thread Matthias Brantner
The proposal to merge lp:~matthias-brantner/zorba/bug-863730 into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~matthias-brantner/zorba/bug-863730/+merge/77784 --

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-863730 into lp:zorba

2011-10-03 Thread Matthias Brantner
The proposal to merge lp:~matthias-brantner/zorba/bug-863730 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~matthias-brantner/zorba/bug-863730/+merge/77784 --

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-863730 into lp:zorba

2011-10-03 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-863730-2011-10-03T20-24-36.523Z/log.html -- https://code.launchpad.net/~matthias-brantner/zorba/bug-863730/+merge/77784 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-863730 into lp:zorba

2011-10-03 Thread Zorba Build Bot
Validation queue job bug-863730-2011-10-03T20-24-36.523Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~matthias-brantner/zorba/bug-863730/+merge/77784 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-863730 into lp:zorba

2011-10-03 Thread noreply
The proposal to merge lp:~matthias-brantner/zorba/bug-863730 into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~matthias-brantner/zorba/bug-863730/+merge/77784 --

[Zorba-coders] [Bug 863730] Re: static delete-node* functions don't raise ZDDY0012

2011-10-03 Thread Zorba Build Bot
** Changed in: zorba Status: New = Fix Committed -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/863730 Title: static delete-node* functions don't raise ZDDY0012 Status in Zorba - The

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-fixing into lp:zorba

2011-10-03 Thread Matthias Brantner
Matthias Brantner has proposed merging lp:~matthias-brantner/zorba/bug-fixing into lp:zorba. Requested reviews: Chris Hillery (ceejatec) For more details, see: https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019 fix if Zorba is built with a different store (name of

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-fixing into lp:zorba

2011-10-03 Thread Matthias Brantner
The proposal to merge lp:~matthias-brantner/zorba/bug-fixing into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019 --

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-fixing into lp:zorba

2011-10-03 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-fixing-2011-10-03T23-12-08.546Z/log.html -- https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-fixing into lp:zorba

2011-10-03 Thread Zorba Build Bot
Validation queue job bug-fixing-2011-10-03T23-12-08.546Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-fixing into lp:zorba

2011-10-03 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 0, Disapprove 1. Got: 1 Pending. -- https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-fixing into lp:zorba

2011-10-03 Thread Chris Hillery
The proposal to merge lp:~matthias-brantner/zorba/bug-fixing into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019 --

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-fixing into lp:zorba

2011-10-03 Thread noreply
The proposal to merge lp:~matthias-brantner/zorba/bug-fixing into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019 --

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

2011-10-03 Thread Paul J. Lucas
** Changed in: zorba Status: New = In Progress -- 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 XQuery