The proposal to merge lp:~nbrinza/zorba/bugs into lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/bugs/+merge/139848
--
https://code.launchpad.net/~nbrinza/zorba/bugs/+merge/139848
Your team Zorba Coders is
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bugs-2012-12-14T08-00-08.241Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/bugs/+merge/139848
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The attempt to merge lp:~nbrinza/zorba/bugs 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 bugs-2012-12-14T08-00-08.241Z is finished. The final
status was:
The proposal to merge lp:~nbrinza/zorba/bugs into lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/bugs/+merge/139848
--
https://code.launchpad.net/~nbrinza/zorba/bugs/+merge/139848
Your team Zorba Coders is
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fix_bug_867094-2012-12-14T08-24-55.154Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_867094/+merge/121405
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
The attempt to merge lp:~zorba-coders/zorba/fix_bug_867094 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 fix_bug_867094-2012-12-14T08-24-55.154Z is finished.
The
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fix-oauth-security-2012-12-14T08-49-34.391Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fix-oauth-security/+merge/117813
Your team Zorba Coders is subscribed to branch
The proposal to merge lp:~zorba-coders/zorba/fix_bug_867094 into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_867094/+merge/121405
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-ctest-fix/+merge/107482
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
The proposal to merge lp:~zorba-coders/zorba/fix_bug_867094 into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_867094/+merge/121405
--
Review: Disapprove
Actually, the changes here conflict with changes on the trunk. Since this
proposal is from 7 months ago and as far as I know there are no outstanding
problems with util-jvm, I'm going to assume this proposal is no longer
desirable.
--
The proposal to merge lp:~zorba-coders/zorba/util-jvm-ctest-fix into
lp:zorba/util-jvm-module has been updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-ctest-fix/+merge/107482
--
Review: Needs Fixing
Also, add a mention to the option changes to the ChangeLog.
--
https://code.launchpad.net/~zorba-coders/zorba/1025564/+merge/125256
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
Validation queue job fix-oauth-security-2012-12-14T08-49-34.391Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fix-oauth-security/+merge/117813
Your team Zorba Coders is subscribed to branch lp:zorba/oauth-module.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/fix-oauth-security into
lp:zorba/oauth-module has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-oauth-security/+merge/117813
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fix_bug_867094-2012-12-14T09-15-51.708Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_867094/+merge/121405
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Validation queue job fix_bug_867094-2012-12-14T09-15-51.708Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_867094/+merge/121405
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/fix_bug_867094 into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_867094/+merge/121405
--
And I have one more question: an XML tree can have a tree ID even
if it is not in a collection. The tree id is used in the construction of
structural references, and maybe in other places that I don't remember right
now. In your merge proposal I see that the tree id is changed when a tree is
In XmlTree, there are now 2 pointers to the root node of the tree: theRootNode
and theTreeInfo-theRoot.
Also, theTreeInfo should be an embeded instance of CollectionTreeInfo, instead
of being allocated every time an XmlTree is constructed.
There is already an Item::isInSubtreeOf() method at
The proposal to merge lp:~zorba-coders/zorba/bug-1067028-no-json into lp:zorba
has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1067028-no-json/+merge/129743
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1067028-no-json-2012-12-14T10-29-02.137Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1067028-no-json/+merge/129743
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Please put a more descriptive commit message for this change.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1026250/+merge/138044
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Why is the isStructuredItem() method a virtual one? I don't think it needs to
be.
Nice catch, I made it non-virtual.
--
https://code.launchpad.net/~zorba-coders/zorba/xml-in-json-indices/+merge/122548
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job bug-1067028-no-json-2012-12-14T10-29-02.137Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1067028-no-json/+merge/129743
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug-1067028-no-json into lp:zorba
has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1067028-no-json/+merge/129743
--
The proposal to merge lp:~zorba-coders/zorba/xml-in-json-indices into lp:zorba
has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/xml-in-json-indices/+merge/122548
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/xml-in-json-indices-2012-12-14T12-50-24.017Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/xml-in-json-indices/+merge/122548
Your team Zorba Coders is subscribed to branch lp:zorba.
--
- I understand why you wanted to rename the function from replace to edit and
your concern makes sense. However, the description of the function still uses
the term replace and I kind of like the explanation as it is.
Sorry about this, this comment was out of date. I put a description that, I
- The comment should mention that you can't replace a node with an object or
vice versa (mentioning inconsistent kind with the ZDDY0040 error is not enough
for the user). Also, that it's only possible to replace root nodes.
I updated this accordingly. Replace root nodes only is handled by
- Could the Item::swap function be implemented on JSONObjects, JSONArrays, and
the base-class for all nodes instead of for Item. It seems like that checking
if a node is the root and swapping the tree in XmlNode::swap would be
sufficient. That seems to be more consistent with the way other
I nevertheless could do some simplification in the code.
--
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
Validation queue job xml-in-json-indices-2012-12-14T12-50-24.017Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/xml-in-json-indices/+merge/122548
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. Got: 4 Needs Fixing, 1 Needs Information.
--
https://code.launchpad.net/~zorba-coders/zorba/xml-in-json-indices/+merge/122548
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The proposal to merge lp:~zorba-coders/zorba/xml-in-json-indices into lp:zorba
has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/xml-in-json-indices/+merge/122548
--
- Is there no more efficient way to find out whether a JSONItem is the root of
a tree or not? Currently, it seems that the searching only works for ordered
collections because it's using findNode.
Yes, this makes sense. I introduced Item::isRoot for this.
--
The proposal to merge lp:~zorba-coders/zorba/replace-nodes-in-collection into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/replace-nodes-in-collection-2012-12-14T13-26-12.575Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
Your team Zorba Coders is subscribed to
Validation queue job replace-nodes-in-collection-2012-12-14T13-26-12.575Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
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 Needs Fixing.
--
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/replace-nodes-in-collection into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
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-1015964 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1015964-2012-12-14T14-28-47.229Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job bug-1015964-2012-12-14T14-28-47.229Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug-1015964 into lp:zorba has been
updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
If this merge is about __llvm__, why is there apparently unrelated stuff about
collectionDecl in it?
--
https://code.launchpad.net/~zorba-coders/zorba/clang-friend/+merge/101518
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/no_disqus/+merge/114793
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/no_disqus into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/no_disqus/+merge/114793
--
https://code.launchpad.net/~zorba-coders/zorba/no_disqus/+merge/114793
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/add-geoproj-module/+merge/139840
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/geo-module-alone/+merge/118626
Your team Zorba Coders is subscribed to branch lp:zorba/geo-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in doc/zorba/xqdoc/templates/www.functx.com_index.html
text conflict in doc/zorba/xqdoc/templates/www.w3.org_2005_xpath-functions.html
text conflict in
doc/zorba/xqdoc/templates/www.w3.org_2005_xpath-functions_math.html
--
The proposal to merge lp:~zorba-coders/zorba/geo-module-alone into
lp:zorba/geo-module has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/geo-module-alone/+merge/118626
--
The proposal to merge lp:~zorba-coders/zorba/no_disqus into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/no_disqus/+merge/114793
--
https://code.launchpad.net/~zorba-coders/zorba/no_disqus/+merge/114793
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/geo-module-alone-2012-12-14T16-00-35.264Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/geo-module-alone/+merge/118626
Your team Zorba Coders is subscribed to branch
Validation queue job geo-module-alone-2012-12-14T16-00-35.264Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/geo-module-alone/+merge/118626
Your team Zorba Coders is subscribed to branch lp:zorba/geo-module.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/geo-module-alone into
lp:zorba/geo-module has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/geo-module-alone/+merge/118626
--
The proposal to merge lp:~zorba-coders/zorba/add-geoproj-module into lp:zorba
has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/add-geoproj-module/+merge/139840
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/add-geoproj-module-2012-12-14T18-17-08.819Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/add-geoproj-module/+merge/139840
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Review: Needs Fixing
I did some testing and the numbers are much better/consistent.
I have started a test to make sure we keep this on the radar and understand if
something regresses.
It's at test/rbkt/Queries/zorba/item/size.xq.
Paul, could you please extend this test with at least one example
Validation queue job add-geoproj-module-2012-12-14T18-17-08.819Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/add-geoproj-module/+merge/139840
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/add-geoproj-module into lp:zorba
has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/add-geoproj-module/+merge/139840
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
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
Ghislain didn't vote on his own proposal, but I'm assuming he'd approve it. So
I'm adding a second Approve vote in his place. If this is OK, Matthias, go
ahead and merge the proposal.
--
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
The proposal to merge lp:~zorba-coders/zorba/replace-nodes-in-collection into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/replace-nodes-in-collection-2012-12-14T21-27-02.231Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
Your team Zorba Coders is subscribed to
Validation queue job replace-nodes-in-collection-2012-12-14T21-27-02.231Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
The proposal to merge lp:~zorba-coders/zorba/replace-nodes-in-collection into
lp:zorba has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/replace-nodes-in-collection/+merge/138196
--
68 matches
Mail list logo