- Given the documentation, I was not able to figure out why the collection
function takes the $start and $skip parameters. Is one of the ignored if
the other one is given? Does skipping start at the item identified by
$start? If so, would it make sense to separate the two ways of skipping
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/121884
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:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/121884
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/121884
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/parse-fragment-2012-08-30T08-02-09.972Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/121884
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The changes look OK to me, but the branch doesn't compile on my machine
(DynamicContextImpl is pure abstract, but xqueryimpl.cpp line 1016 tries to
instantiate one). I'm kicking off the queue to see if the same problem occurs
there.
--
Review: Needs Fixing
However, there are two things to fix regardless:
1. A number of files have been committed with Windows newlines - all the new
.h, .i, and .java files under swig/, I believe. You need to have the
appropriate rules file in your Bazaar home directory, as described in the
The proposal to merge lp:~zorba-coders/zorba/bug938320 into lp:zorba has been
updated.
Commit Message changed to:
Test commit message to make the queue run
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug938320/+merge/118194
--
Validation queue job parse-fragment-2012-08-30T08-02-09.972Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/121884
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/121884
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/121884
Your team
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug938320-2012-08-30T08-33-45.881Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug938320/+merge/118194
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The attempt to merge lp:~zorba-coders/zorba/bug938320 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 bug938320-2012-08-30T08-33-45.881Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/bug938320 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug938320/+merge/118194
--
https://code.launchpad.net/~zorba-coders/zorba/bug938320/+merge/118194
The proposal to merge lp:~zorba-coders/zorba/bug938320 into lp:zorba has been
updated.
Commit Message changed to:
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug938320/+merge/118194
--
https://code.launchpad.net/~zorba-coders/zorba/bug938320/+merge/118194
Your team
Yep, same compilation error:
[ 42%] Building CXX object
src/CMakeFiles/zorba_simplestore.dir/api/dynamiccontextimpl.cpp.o
../../src/api/xqueryimpl.cpp: In member function 'virtual
zorba::DynamicContext* zorba::XQueryImpl::getDynamicContext() const':
../../src/api/xqueryimpl.cpp:1040: error:
Ghislain Fourny has proposed merging
lp:~zorba-coders/zorba/removing-store-naive-in-includes into lp:zorba.
Requested reviews:
Till Westmann (tillw)
Matthias Brantner (matthias-brantner)
For more details, see:
The proposal to merge lp:~zorba-coders/zorba/removing-store-naive-in-includes
into lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/removing-store-naive-in-includes/+merge/122051
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/removing-store-naive-in-includes-2012-08-30T12-28-57.303Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/removing-store-naive-in-includes/+merge/122051
Your team Zorba Coders is
Ghislain Fourny has proposed merging lp:~zorba-coders/zorba/treeid-fixes into
lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/treeid-fixes/+merge/122054
Some fixes in
Validation queue job removing-store-naive-in-includes-2012-08-30T12-28-57.303Z
is finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/removing-store-naive-in-includes/+merge/122051
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1,
Needs Fixing 1, Pending 1. Got: 2 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/removing-store-naive-in-includes/+merge/122051
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/removing-store-naive-in-includes
into lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/removing-store-naive-in-includes/+merge/122051
--
The proposal to merge lp:~zorba-coders/zorba/treeid-fixes into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/treeid-fixes/+merge/122054
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/treeid-fixes-2012-08-30T13-45-59.158Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/treeid-fixes/+merge/122054
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos-scratch
into lp:zorba.
Requested reviews:
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/122068
rchandle cleanup
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/122068
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 :
Validation queue job treeid-fixes-2012-08-30T13-45-59.158Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/treeid-fixes/+merge/122054
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/jsoniq-library-functions into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347
--
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1,
Needs Fixing 1, Pending 1. Got: 3 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/treeid-fixes/+merge/122054
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/treeid-fixes into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/treeid-fixes/+merge/122054
--
Hi Matthias,
Thanks! I added the @error tag.
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
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/122068
--
Validation queue job jsoniq-library-functions-2012-08-30T14-13-07.942Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2012-08-30T14-43-30.885Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/122068
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1,
Needs Fixing 1, Pending 1. Got: 1 Approve, 1 Needs Fixing, 1 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347
Your team Zorba Coders is subscribed to branch lp:zorba.
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/removing-store-naive-in-includes/+merge/122051
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/jsoniq-library-functions into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347
--
Validation queue job markos-scratch-2012-08-30T14-43-30.885Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/122068
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/122068
--
Review: Approve
I still think the definitions of some of these functions are weird, but this
proposal does match the spec as it stands at the moment.
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347
Your team Zorba Coders is subscribed to branch lp:zorba.
Fixed.
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/121978
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
More help
Replaced by better solution in
lp:~zorba-coders/zorba/bug-932884-xhtml-serialization .
--
https://code.launchpad.net/~zorba-coders/zorba/iframe/+merge/91202
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/iframe into lp:zorba has been
updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/iframe/+merge/91202
--
https://code.launchpad.net/~zorba-coders/zorba/iframe/+merge/91202
Your team
Chris Hillery has proposed merging
lp:~zorba-coders/zorba/bug-932884-xhtml-serialization into lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
Related bugs:
Bug #932884 in Zorba: empty tags in XHTML serialization
https://bugs.launchpad.net/zorba/+bug/932884
For more
The proposal to merge lp:~zorba-coders/zorba/bug-932884-xhtml-serialization
into lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-932884-xhtml-serialization/+merge/122156
--
Matthias - you initially raised the issue about iframe serialization. If you
could confirm that this change fixes that, and ideally doesn't break anything
else, that'd be great.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-932884-xhtml-serialization/+merge/122156
Your team Zorba Coders
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug-932884-xhtml-serialization/+merge/122156
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 :
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-932884-xhtml-serialization-2012-08-30T21-42-53.199Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-932884-xhtml-serialization/+merge/122156
Your team Zorba Coders is subscribed
Validation queue job bug-932884-xhtml-serialization-2012-08-30T21-42-53.199Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-932884-xhtml-serialization/+merge/122156
Your team Zorba Coders is subscribed to branch lp:zorba.
--
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/~zorba-coders/zorba/bug-932884-xhtml-serialization/+merge/122156
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The proposal to merge lp:~zorba-coders/zorba/bug-932884-xhtml-serialization
into lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-932884-xhtml-serialization/+merge/122156
--
Review: Needs Fixing
Ghislain, could you please add the new module to the ChangeLog?
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-library-functions/+merge/119347
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to
Review: Approve
I still don't love the additional dependency, but I think it probably makes
sense not to roll our own UUID libs, so I'll approve this. I wonder if the RQ
machine will have it?...
--
https://code.launchpad.net/~zorba-coders/zorba/feature-uuid/+merge/118268
Your team Zorba Coders
The proposal to merge lp:~zorba-coders/zorba/feature-uuid into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/feature-uuid/+merge/118268
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/feature-uuid-2012-08-31T01-04-21.504Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/feature-uuid/+merge/118268
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Validation queue job feature-uuid-2012-08-31T01-04-21.504Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/feature-uuid/+merge/118268
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/feature-uuid into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/feature-uuid/+merge/118268
--
https://code.launchpad.net/~zorba-coders/zorba/feature-uuid/+merge/118268
56 matches
Mail list logo