The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/95152
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/95152
Your
The attempt to merge lp:~nbrinza/zorba/parse-fragment 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 parse-fragment-2012-02-29T15-41-14.921Z is finished.
The final
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/parse-fragment-2012-02-29T15-41-14.921Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/95152
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: h
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/95152
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/95152
Your
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/95152
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-coder
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Commit Message changed to:
Fully streaming module for parsing XML. Parsing can also be configured with
several options such as skip-text-only-whitespace.
For more details, see:
https://code.launchpad.net/~nb
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/95152
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-coder
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/95150
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-coder
I have fixed the regressions and the conflicts. Resubmitting.
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsu
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in ChangeLog
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@l
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/94419
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
Your
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
Your
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
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-coder
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
Your
The attempt to merge lp:~nbrinza/zorba/parse-fragment 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 parse-fragment-2012-02-23T17-50-09.456Z is finished.
The final
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
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-coder
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/parse-fragment-2012-02-23T17-50-09.456Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: h
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/94419
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
Your
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
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-coder
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Commit Message changed to:
Fully streaming parse-xml module.
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/94419
--
https://code.launchpad.net/~nbrinza/zorba/parse-fr
I've fixed the conflict in the Changelog. Resubmitting for merging into the
trunk.
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/91657
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@l
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/91657
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-coder
Adding David to the review, I think he found a bug.
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/91657
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
I tried it and it looks like just the best.
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/91657
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsub
All pending issues have been fixed. Resubmitting the proposal for merging.
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/89038
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launc
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/89038
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-coder
Review: Needs Fixing
- The branch doesn't build without schema support (i.e.
ZORBA_NO_XMLSCHEMA:BOOL=ON)
[ 1%] Building CXX object
src/CMakeFiles/zorba_simplestore.dir/runtime/parsing_and_serializing/parse_fragment_impl.cpp.o
/home/mbrantner/zorba/sandbox/src/runtime/parsing_and_serializing/pa
It certainly is the closest allowed alternative...
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/89038
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 : h
Would xmldoc be a good prefix?
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/89038
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
Review: Needs Information
I tried the forecast data with the parse function: very impressive!!
For documentation purposes, I would rename the parse-xml prefix to xml, this is
more consistent right?
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/89038
Your team Zorba Coders i
The Elsevier copyright came from the xqdoc.xsd, which I think I used as a base
for the options file. I fixed it together with the other issues Matthias has
raised. I've also fixed a couple of other cases where malformed XMLs where
handled incorrectly. Resubmitting for merging.
--
https://code.l
Review: Needs Information
Why is there an Elsevier copyright for
modules/com/zorba-xquery/www/modules/xml-options.xsd ?
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/88273
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorb
Review: Needs Information
Also, the branch doesn't build without schema support (i.e.
ZORBA_NO_XMLSCHEMA:BOOL=ON)
[ 1%] Building CXX object
src/CMakeFiles/zorba_simplestore.dir/runtime/parsing_and_serializing/parse_fragment_impl.cpp.o
/home/mbrantner/zorba/sandbox/src/runtime/parsing_and_seria
Review: Needs Fixing
WOW, that's really cool. I just tried parsing a huge file and memory
consumption was constant at a low level. Also, skipping root nodes worked like
a charm.
I have some question/remarks:
- In the schema, there is a typo skip-top-Level-text-nodes =>
skip-top-level-text-node
Yes, indeed, this is a strange corruption that I sometimes get after a merge.
I've fixed the issue, resubmitting.
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/88238
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-code
Review: Needs Fixing
It seems to be a problem with the diff:
- bool isLangAttr(const store::Item_t& aAttr) const;
- bool matchesReqLang(const store::Item_t& aLangAttr, const zstring&
aRequestLang) const;
+ bool isLangAttr(const store::Item_t& aAttr) const;
+ bool matchesReqLang(const store::I
The branch doesn't compile on my machine:
[ 0%] Building CXX object
src/CMakeFiles/zorba_simplestore.dir/functions/func_nodes.cpp.o
In file included from
/Users/wcandillon/tmp/parse-fragment/build/src/functions/func_nodes.cpp:26:
/Users/wcandillon/tmp/parse-fragment/build/src/runtime/nodes/nodes
201 - 237 of 237 matches
Mail list logo
