Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug1082740_fn_subsequence/+merge/157755
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 :
The proposal to merge lp:~zorba-coders/zorba/bug1082740_fn_subsequence into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1082740_fn_subsequence/+merge/157755
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug867259/+merge/160515
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 :
Looks great!
--
https://code.launchpad.net/~zorba-coders/zorba/bug867259/+merge/160515
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
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug1082740_fn_subsequence-2013-04-24T06-13-54.41Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug1082740_fn_subsequence/+merge/157755
Your team Zorba Coders is subscribed to branch
The proposal to merge lp:~zorba-coders/zorba/bug1082740_fn_subsequence into
lp:zorba has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1082740_fn_subsequence/+merge/157755
--
Validation queue job bug1082740_fn_subsequence-2013-04-24T06-13-54.41Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug1082740_fn_subsequence/+merge/157755
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
I will include the 3 different errors for uri:serialize(), Should I leave
uri:parse() without any error code?
I'm sure there are some reasonable errors for uri:parse(), but I'm not totally
sure what they might be. It's possible that URI.cpp will already throw some
exceptions if you try to
Review: Needs Fixing
Unfortunately, I don't think the proposed change is safe. istream::unget() is
not guaranteed to work, and in particular it probably won't work if the stream
is coming via HTTP.
However, there is a function StreamResource::isStreamSeekable(). Perhaps you
could check that
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug867259/+merge/160515
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 :
The proposal to merge lp:~zorba-coders/zorba/bug867259 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug867259/+merge/160515
--
https://code.launchpad.net/~zorba-coders/zorba/bug867259/+merge/160515
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug867259-2013-04-24T11-56-52.859Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug867259/+merge/160515
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job bug867259-2013-04-24T11-56-52.859Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug867259/+merge/160515
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug867259 into lp:zorba has been
updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug867259/+merge/160515
--
https://code.launchpad.net/~zorba-coders/zorba/bug867259/+merge/160515
Your team
Review: Needs Fixing
Please see https://bugs.launchpad.net/zorba/+bug/1147563/comments/3
--
https://code.launchpad.net/~zorba-coders/zorba/bug1147563/+merge/153485
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
So, should I remove the xml dependency from the function? or should I
remove
the whole function?
If we remove the function the user should be able to create the same
function
by themselves using xqxq:get-type-variable
I think we should remove the entire function. It leaves
Paul J. Lucas has proposed merging lp:~zorba-coders/zorba/bug-1169908 into
lp:zorba.
Commit message:
char_length() and utf8::read() now throw an exception upon an invalid UTF-8.
Requested reviews:
Paul J. Lucas (paul-lucas)
Related bugs:
Bug #1169908 in Zorba: Zorba hangs with invalid utf-8
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
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 :
The proposal to merge lp:~zorba-coders/zorba/bug-1169908 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1169908-2013-04-24T15-53-40.871Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The attempt to merge lp:~zorba-coders/zorba/bug-1169908 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-1169908-2013-04-24T15-53-40.871Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/bug-1169908 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
--
Review: Approve
Looks good.
It will most likely not pass in the current remote queue setup because it's
still running libxercesc 2.8). It has been tested with libxerces 3.1.1 and is
running correctly, should pass when Chris will update remote queue to use the
newer xerces.
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug1100471/+merge/159239
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 :
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug1100471/+merge/159239
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 :
The proposal to merge lp:~zorba-coders/zorba/bug1100471 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1100471/+merge/159239
--
https://code.launchpad.net/~zorba-coders/zorba/bug1100471/+merge/159239
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug1100471-2013-04-24T17-04-59.855Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug1100471/+merge/159239
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
So, should I remove the xml dependency from the function? or should I
remove
the whole function?
If we remove the function the user should be able to create the same
function
by themselves using xqxq:get-type-variable
I think we should remove the entire function. It
Validation queue job bug1100471-2013-04-24T17-04-59.855Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug1100471/+merge/159239
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/160731
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 :
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/160731
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2013-04-24T18-05-01.675Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/160731
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Validation queue job markos-scratch-2013-04-24T18-05-01.675Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/160731
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
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/160731
--
So, should I remove the xml dependency from the function? or should
I
remove
the whole function?
If we remove the function the user should be able to create the same
function
by themselves using xqxq:get-type-variable
I think we should remove the entire
The proposal to merge lp:~zorba-coders/zorba/bug-1169908 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
--
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-1169908/+merge/160692
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug1103819-catchAll into lp:zorba
has been updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1103819-catchAll/+merge/159053
--
The proposal to merge lp:~zorba-coders/zorba/bug-1169908 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1169908-2013-04-24T21-32-43.53Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The attempt to merge lp:~zorba-coders/zorba/bug-1169908 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-1169908-2013-04-24T21-32-43.53Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/bug-1169908 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
--
In trunk at least the following tests are passing: cbcl-schema-element-1
2,4,5,8. Which makes them a regression.
--
https://code.launchpad.net/~zorba-coders/zorba/bug1147563/+merge/153485
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug1147563 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1147563/+merge/153485
--
https://code.launchpad.net/~zorba-coders/zorba/bug1147563/+merge/153485
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug1147563-2013-04-25T00-35-04.776Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug1147563/+merge/153485
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Chris Hillery has proposed merging lp:~zorba-coders/zorba/rq-xerces-311 into
lp:zorba.
Commit message:
Any test changes necessary due to upgrading the RQ to Xerces 3.1.1.
Requested reviews:
Zorba Coders (zorba-coders)
For more details, see:
The proposal to merge lp:~zorba-coders/zorba/rq-xerces-311 into lp:zorba has
been updated.
Commit Message changed to:
Any test changes necessary due to upgrading the RQ to Xerces 3.1.1.
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
--
Guys, FYI, this RQ run is running with Xerces 3.1.1. Look for any test
progressions/regressions.
--
https://code.launchpad.net/~zorba-coders/zorba/bug1147563/+merge/153485
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to
The proposal to merge lp:~zorba-coders/zorba/rq-xerces-311 into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
--
The proposal to merge lp:~zorba-coders/zorba/rq-xerces-311 into lp:zorba has
been updated.
Status: Approved = Rejected
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
--
The proposal to merge lp:~zorba-coders/zorba/rq-xerces-311 into lp:zorba has
been updated.
Status: Rejected = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
--
The proposal to merge lp:~zorba-coders/zorba/rq-xerces-311 into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
--
The proposal to merge lp:~zorba-coders/zorba/rq-xerces-311 into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
--
The proposal to merge lp:~zorba-coders/zorba/rq-xerces-311 into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
--
The attempt to merge lp:~zorba-coders/zorba/bug1147563 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 bug1147563-2013-04-25T00-35-04.776Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/bug1147563 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1147563/+merge/153485
--
https://code.launchpad.net/~zorba-coders/zorba/bug1147563/+merge/153485
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/rq-xerces-311-2013-04-25T01-02-41.386Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
Your team Zorba Coders is requested to review the proposed merge of
The attempt to merge lp:~zorba-coders/zorba/rq-xerces-311 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 rq-xerces-311-2013-04-25T01-02-41.386Z is finished.
The
The proposal to merge lp:~zorba-coders/zorba/rq-xerces-311 into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
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 :
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
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 :
The proposal to merge lp:~zorba-coders/zorba/rq-xerces-311 into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/rq-xerces-311-2013-04-25T01-33-39.33Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Validation queue job rq-xerces-311-2013-04-25T01-33-39.33Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/rq-xerces-311 into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/rq-xerces-311/+merge/160781
--
The proposal to merge lp:~zorba-coders/zorba/bug-1169908 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1169908-2013-04-25T02-10-50.587Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job bug-1169908-2013-04-25T02-10-50.587Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
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.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The proposal to merge lp:~zorba-coders/zorba/bug-1169908 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
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 :
The proposal to merge lp:~zorba-coders/zorba/bug-1169908 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1169908-2013-04-25T02-57-44.597Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
So, should I remove the xml dependency from the function? or
should
I
remove
the whole function?
If we remove the function the user should be able to create the
same
function
by themselves using xqxq:get-type-variable
I think we should remove the
Validation queue job bug-1169908-2013-04-25T02-57-44.597Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug-1169908 into lp:zorba has been
updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1169908/+merge/160692
76 matches
Mail list logo
