The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
--
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/new-jsoniq-2013-05-13T11-36-28.819Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The attempt to merge lp:~zorba-coders/zorba/new-jsoniq 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 new-jsoniq-2013-05-13T11-36-28.819Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
--
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
--
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/new-jsoniq-2013-05-13T12-21-59.605Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job new-jsoniq-2013-05-13T12-21-59.605Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1,
Needs Fixing 1, Pending 1, Needs Information 1, Resubmit 1. Got: 1
Approve, 2 Needs Fixing.
--
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
Your team Zorba Coders is subscribed to branch
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
--
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
Matthias, your suggestions for two improved warnings do not change the
messages: you just switched the order of the words. The warnings are built
around the template:
feature not supported by the common language grammar{: $1}
So it is not easy to make this swap. I could probably do it,
Why would you need a qualified value if the option itself is already
qualified?
I guess because the option name and the option value may not live in the same
namespace.
--
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
Your team Zorba Coders is subscribed to branch
Why would you need a qualified value if the option itself is already
qualified?
I guess because the option name and the option value may not live in the same
namespace.
Why not? It's not as if you do schema validation on the options and their
values. The option needs to be qualified to
Review: Needs Fixing
Thanks for fixing the warnings. I successfully checked, except for two things:
1.
\ (XQuery parser) or (JSONiq parser) leads to a warning about disallowed
character entity references/JSON character escape sequence. I think the warning
could be made more general and say
(by the way, character entity references do not exist in XML. There are
character references (#...;) and entity references (...;), but not both at
the same time if I am correct :-) )
Indeed you're right. I trusted Wikipedia which uses the term character entity
references and says ``The XML
Fixed all the raised issues.
--
https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375
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 :
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba.
Commit message:
Fixes stupid null-pointer dereference.
Requested reviews:
Paul J. Lucas (paul-lucas)
For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/163627
Fixes stupid
Review: Approve
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/163627
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/~zorba-coders
Review: Approve
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/163627
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/~zorba-coders
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/163627
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/163627
Your team
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/pjl-misc-2013-05-14T02-51-46.488Z/log.html
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/163627
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job pjl-misc-2013-05-14T02-51-46.488Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/163627
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been
updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/163627
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/163627
Your team Zorba
Note: even though FOTS test orderBy52a now fails, I think it worked before by
accident. The code is more consistent now, so orderBy52a now fails in the
same way that the other FOTS orderBy tests fail.
--
https://code.launchpad.net/~paul-lucas/zorba/bug-1090089/+merge/163085
Your team Zorba
The proposal to merge lp:~paul-lucas/zorba/bug-1090089 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/bug-1090089/+merge/163085
--
https://code.launchpad.net/~paul-lucas/zorba/bug-1090089/+merge/163085
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1090089-2013-05-14T05-24-54.459Z/log.html
--
https://code.launchpad.net/~paul-lucas/zorba/bug-1090089/+merge/163085
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The attempt to merge lp:~paul-lucas/zorba/bug-1090089 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 bug-1090089-2013-05-14T05-24-54.459Z is finished. The
final
The proposal to merge lp:~paul-lucas/zorba/bug-1090089 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/bug-1090089/+merge/163085
--
https://code.launchpad.net/~paul-lucas/zorba/bug-1090089/+merge/163085
27 matches
Mail list logo
