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 subscrib
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: https://launchpad.ne
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 subscrib
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 l
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 f
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 lp:zorba/oauth-m
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
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_867094/+m
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 : https://
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
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_867094/+m
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.
--
https://code.launchpad.net/~zorba-coder
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
--
https://code.launchpad.net/~zorba-coders
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 : zorba-coders@lists
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: htt
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
--
https://code.launchpad.net/~zorba-coders/zorba/fi
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 l
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: https://launchpad.net/~z
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
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_867094/+merge/1
> 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 i
> 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() met
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
--
https://code.launchpad.net/~zorba-coders/zorba/bug-106
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
U
> 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: https://launchpa
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: https://launch
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
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1067028-n
> > It's because it adds them to the XCode project. Not having them makes full-
> > text search very cumbersome (i.e., headers are not considered).
>
> Isn't that what our own HEADER_GROUP_SUBFOLDER custom macro is for?
Thanks for the pointer. I removed the .h files for now and will address this
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
--
https://code.launchpad.net/~zorba-coders/zorba/xml-in-
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
> - 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 ZD
> - 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 : zorba-coders@lists.laun
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: https://launch
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
--
https://code.launchpad.net/~zorba-coders/zorba/xml-in-
> - 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.
--
https://code.launchpad.net/~zorb
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
--
https://code.launchpad.net/~zorba-code
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 branc
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 list
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 lis
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
--
https://code.launchpad.net/~zorba-code
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 : https://launchpad.net/~zorba-co
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
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/1251
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: h
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: https://launchpad.net/~zorba-c
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
You
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 to
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 : https://launchpad.net/~zorba-code
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
You
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 : https://launchpad.net/~z
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 : https://launchp
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
--
ht
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
--
https://code.launchpad.net/~zorba-coders/zorba/ge
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
You
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 lp:zorba/geo-module.
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: https://l
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
--
https://code.launchpad.net/~zorba-coders/zorba/geo-modu
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
--
https://code.launchpad.net/~zorba-coders/zorba/add-geopr
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.
--
M
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 f
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: https://launchpa
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
--
https://code.launchpad.net/~zorba-coders/zorba/add-geoproj-mod
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 : https://launchp
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
You
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
--
https://code.launchpad.net/~zorba-code
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 branc
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 list
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
--
https://code.launchpad.net/~zorba-coders/zor
69 matches
Mail list logo