Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-08-30 Thread Cezar Andrei
Review: Approve Error for the attached query is now fixed. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/181327 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:~nbrinza/zorba/error-messages into lp:zorba

2013-04-10 Thread Chris Hillery
Nicolae, is this proposal still necessary? If not, please mark it Rejected. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/148300 Your team Zorba Coders is requested to review the proposed merge of lp:~nbrinza/zorba/error-messages into lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-02-13 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 1 Pending. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/148300 Your team Zorba Coders is requested to review the proposed merge of

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-01-29 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/~nbrinza/zorba/error-messages/+merge/144397 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-01-29 Thread William Candillon
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/144397 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:~nbrinza/zorba/error-messages into lp:zorba

2013-01-22 Thread Zorba Build Bot
The attempt to merge lp:~nbrinza/zorba/error-messages into lp:zorba failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message): Validation queue job error-messages-2013-01-22T21-51-44.881Z is finished. The final

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-01-10 Thread William Candillon
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142504 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:~nbrinza/zorba/error-messages into lp:zorba

2013-01-10 Thread William Candillon
Review: Needs Fixing -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142504 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:~nbrinza/zorba/error-messages into lp:zorba

2013-01-10 Thread William Candillon
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142504 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:~nbrinza/zorba/error-messages into lp:zorba

2013-01-10 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142697 Your team Zorba Coders is requested to review the proposed merge of lp:~nbrinza/zorba/error-messages into lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-01-10 Thread Zorba Build Bot
The attempt to merge lp:~nbrinza/zorba/error-messages into lp:zorba failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message): Validation queue job error-messages-2013-01-10T14-11-42.339Z is finished. The final

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-01-10 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/~nbrinza/zorba/error-messages/+merge/142697 Your team Zorba Coders is requested to review the proposed merge of

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-01-10 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142697 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:~nbrinza/zorba/error-messages into lp:zorba

2013-01-09 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 1 Pending. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142504 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-01-08 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 1 Pending. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142317 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-01-07 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171 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:~nbrinza/zorba/error-messages into lp:zorba

2013-01-07 Thread Chris Hillery
Nicolae - something seems to be weird with either your changes or with the RQ in general. As you see it started the queue a dozen times. Each time it timed out running the following test. Could your changes somehow be causing this test to hang? 211: Test command:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-01-07 Thread Zorba Build Bot
The attempt to merge lp:~nbrinza/zorba/error-messages into lp:zorba failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message): Validation queue job error-messages-2013-01-08T01-58-49.912Z is finished. The final

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-01-07 Thread Chris Hillery
I've killed the most recent run manually. Before I did so I noted that testdriver was stuck spinning 100% of the CPU on the RQ machine, so it appears there's an infinite loop. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171 Your team Zorba Coders is subscribed to

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-09-09 Thread Zorba Build Bot
The attempt to merge lp:~nbrinza/zorba/error-messages into lp:zorba failed. Below is the output from the failed tests. CMake Warning (dev) in /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake: A logical block opening on the line

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-09-06 Thread Nicolae Brinza
Matthias, this is a known issue -- all the error mesages in the parser are hard-coded. There is an open bug and even some work done in a branch to fix these. But this current bug needs to be merged for the 2.7 release, while that other one was not scheduled. So I think it should be approved. --

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-09-06 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/122401 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:~nbrinza/zorba/error-messages into lp:zorba

2012-09-06 Thread William Candillon
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/122401 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:~nbrinza/zorba/error-messages into lp:zorba

2012-09-06 Thread Zorba Build Bot
The attempt to merge lp:~nbrinza/zorba/error-messages 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 error-messages-2012-09-06T22-02-02.164Z is finished. The final

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-09-05 Thread Matthias Brantner
Review: Needs Fixing The error message should not be hard-coded. Instead, it should be put into the dictionary in src/diagnostics/diagnostic_en.xml and be taken from there. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/122401 Your team Zorba Coders is subscribed to branch

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-09-01 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/122400 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:~nbrinza/zorba/error-messages into lp:zorba

2012-09-01 Thread Zorba Build Bot
Attempt to merge into lp:zorba failed due to conflicts: text conflict in ChangeLog -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/122400 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-09-01 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/122401 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:~nbrinza/zorba/error-messages into lp:zorba

2012-09-01 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/~nbrinza/zorba/error-messages/+merge/122401 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-07-16 Thread David Graf
Review: Approve Very cool. It's just annoying that zorba needs parameters to execute a file. But it's already nice to be able to write scripts like this: #!./bin/zorba -f -q 1+2 --

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-07-16 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114935 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:~nbrinza/zorba/error-messages into lp:zorba

2012-07-13 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 1 Pending. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114935 Your team Zorba Coders is requested to review the proposed merge of

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-07-11 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/~nbrinza/zorba/error-messages/+merge/114399 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-07-11 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114399 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:~nbrinza/zorba/error-messages into lp:zorba

2012-07-10 Thread Zorba Build Bot
Attempt to merge into lp:zorba failed due to conflicts: text conflict in src/compiler/parser/xquery_scanner.cpp -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114003 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-07-09 Thread Matthias Brantner
Review: Needs Information - What about those tests? All of them are fixed, as the remote queue shows, except for eqname-013, which is waiting on a W3C bug resolution. OK. Maybe I don't understand how the RQ works. I was asking this because those tests are still part of the file

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-07-09 Thread Nicolae Brinza
Is this file automatically maintained by the RQ in case of a progression? I think so. That was the original idea, if I remember correctly. After the testsuite is run the known failures file is updated. I've looked into the testdriver, and there is some code that seems to do that, see

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-07-09 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/112849 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:~nbrinza/zorba/error-messages into lp:zorba

2012-07-09 Thread Zorba Build Bot
Attempt to merge into lp:zorba failed due to conflicts: text conflict in ChangeLog -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/112849 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-07-09 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/113999 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:~nbrinza/zorba/error-messages into lp:zorba

2012-07-09 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114003 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:~nbrinza/zorba/error-messages into lp:zorba

2012-07-09 Thread Nicolae Brinza
I have fixed the Changelog conflict. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114003 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:~nbrinza/zorba/error-messages into lp:zorba

2012-07-09 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114003 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:~nbrinza/zorba/error-messages into lp:zorba

2012-06-30 Thread Nicolae Brinza
- What about those tests? All of them are fixed, as the remote queue shows, except for eqname-013, which is waiting on a W3C bug resolution. - Nope, that query is not supposed to work. The new EQName syntax is Q{URI}localname, there is no colon. --

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-06-29 Thread Zorba Build Bot
Attempt to merge into lp:zorba failed due to conflicts: text conflict in test/rbkt/Queries/CMakeLists.txt -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/112757 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-06-29 Thread Zorba Build Bot
The attempt to merge lp:~nbrinza/zorba/error-messages 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 error-messages-2012-06-29T13-07-06.398Z is finished. The final

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-06-29 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/~nbrinza/zorba/error-messages/+merge/112767 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-06-29 Thread Zorba Build Bot
The attempt to merge lp:~nbrinza/zorba/error-messages 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 error-messages-2012-06-29T14-18-00.292Z is finished. The final

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-06-29 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 1 Pending. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/112849 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-06-29 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/112849 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:~nbrinza/zorba/error-messages into lp:zorba

2012-06-29 Thread Matthias Brantner
Review: Needs Information - what about the following tests from in w3c_known_failures.txt? test/rbkt/w3c_testsuite/XQuery/Basics/EQNames/eqname-013 test/rbkt/w3c_testsuite/XQuery/Basics/EQNames/eqname-006 test/rbkt/w3c_testsuite/XQuery/Basics/EQNames/eqname-012

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-06-25 Thread David Graf
Do we need to approve again? Or is something broken and needs to get fixed? -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/111745 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-06-25 Thread Nicolae Brinza
Yes, David, please approve it again. As you can see above, I've run the remote queue and all the tests succeeded. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/111745 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-06-25 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/111745 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:~nbrinza/zorba/error-messages into lp:zorba

2012-06-25 Thread David Graf
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/111745 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:~nbrinza/zorba/error-messages into lp:zorba

2012-06-24 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 2 Pending. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/111745 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-06-21 Thread David Graf
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/111391 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:~nbrinza/zorba/error-messages into lp:zorba

2012-06-21 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/111391 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:~nbrinza/zorba/error-messages into lp:zorba

2012-06-20 Thread Matthias Brantner
Review: Needs Fixing This branch has some conflicts with the current trunk. Also, if I merge the trunk, I get the following compile error: /home/mbrantner/zorba/sandbox/src/compiler/parser/xquery_parser.y: In member function ‘virtual int zorba::xquery_parser::parse()’:

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2012-05-30 Thread David Graf
Review: Approve Tested it. Very cool. Something that is unrelated: Is it possible to make those things: +#line 87 /home/colea/xquery_bzr/error-messages/src/compiler/parser/xquery_parser.y ... with relative paths? -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/107899 Your