Review: Approve
Sorry I didn't get back to re-review this sooner.
You can't have a single mention for both XQuery and XQueryX tests, because the
test names are not the same. So this change is technically wrong, since you've
removed the XQueryX EXPECTED_FAILURE()s.
However, my guess is that
The proposal to merge lp:~zorba-coders/zorba/update_W3C_testsuites into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/update_W3C_testsuites/+merge/104652
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/update_W3C_testsuites-2012-05-15T07-44-10.725Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/update_W3C_testsuites/+merge/104652
Your team Zorba Coders is subscribed to branch lp:zorba.
Public bug reported:
We rarely test XQTS XQueryX tests using testdriver, meaning that the
EXPECTED_FAILURE()s for them are probably not correct. We need to run
the full set of W3C test suites in XQueryX mode without using
testdriver_mt, and ensure that any failures are marked as
Validation queue job update_W3C_testsuites-2012-05-15T07-44-10.725Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/update_W3C_testsuites/+merge/104652
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/update_W3C_testsuites into
lp:zorba has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/update_W3C_testsuites/+merge/104652
--
William Candillon has proposed merging lp:~zorba-coders/zorba/doxygen into
lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105787
Remove dead links
--
The proposal to merge lp:~zorba-coders/zorba/doxygen into lp:zorba has been
updated.
Status: Needs review = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105787
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105787
Your team
William Candillon has proposed merging lp:~zorba-coders/zorba/doxygen into
lp:zorba.
Requested reviews:
William Candillon (wcandillon)
Matthias Brantner (matthias-brantner)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105796
Remove dead links in the
The proposal to merge lp:~zorba-coders/zorba/doxygen into lp:zorba has been
updated.
Commit Message changed to:
Remove dead links in the documentation.
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105796
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105796
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 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/92171
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/92171
Your team
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/jsoniq/+merge/92171
Your team Zorba Coders is requested to review the proposed merge of
lp:~zorba-coders/zorba/jsoniq
The proposal to merge lp:~zorba-coders/zorba/jsoniq into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/92171
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/92171
Your team
The proposal to merge lp:~zorba-coders/zorba/jsoniq into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/92171
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/92171
Your team
The proposal to merge lp:~zorba-coders/zorba/jsoniq-deactivated into lp:zorba
has been updated.
Commit Message changed to:
Merged the JSONiq branch into the zorba trunk, with JSONiq deactivated.
For more details, see:
The proposal to merge lp:~zorba-coders/zorba/jsoniq-deactivated into lp:zorba
has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-deactivated/+merge/105825
--
The proposal to merge lp:~zorba-coders/zorba/jsoniq-deactivated into lp:zorba
has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-deactivated/+merge/105825
--
The proposal to merge lp:~zorba-coders/zorba/jsoniq-deactivated into lp:zorba
has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-deactivated/+merge/105825
--
The proposal to merge lp:~zorba-coders/zorba/jsoniq-deactivated into lp:zorba
has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-deactivated/+merge/105825
--
Nicolae Brinza has proposed merging lp:~nbrinza/zorba/parse-fragment into
lp:zorba.
Requested reviews:
Nicolae Brinza (nbrinza)
Matthias Brantner (matthias-brantner)
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/105841
Documentation improvements.
--
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Commit Message changed to:
Documentation improvements.
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/105841
--
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/105841
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 attempt to merge lp:~zorba-coders/zorba/jsoniq 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 jsoniq-2012-05-15T14-50-23.022Z is finished. The
final status
Review: Needs Fixing
Why do the build instructions not point to the latest version but to 2.2.0?
http://www.zorba-xquery.com/html/documentation/2.2.0/zorba/build
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105796
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The proposal to merge lp:~zorba-coders/zorba/xquery3.0_funcs into lp:zorba has
been updated.
Commit Message changed to:
Implementation of fn:parse-xml-fragment
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/xquery3.0_funcs/+merge/105840
--
The proposal to merge lp:~zorba-coders/zorba/xquery3.0_funcs into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/xquery3.0_funcs/+merge/105840
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/xquery3.0_funcs-2012-05-15T15-37-15.371Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/xquery3.0_funcs/+merge/105840
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The attempt to merge lp:~zorba-coders/zorba/xquery3.0_funcs 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 xquery3.0_funcs-2012-05-15T15-37-15.371Z is finished.
The proposal to merge lp:~zorba-coders/zorba/xquery3.0_funcs into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/xquery3.0_funcs/+merge/105840
--
William Candillon has proposed merging lp:~zorba-coders/zorba/doxygen into
lp:zorba.
Requested reviews:
William Candillon (wcandillon)
Matthias Brantner (matthias-brantner)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105881
Remove dead links in the
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105881
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 :
Example fixed
Also fixed all Doxygen warnings.
--
https://code.launchpad.net/~zorba-coders/zorba/xqjdoc/+merge/105700
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: Needs Fixing
The link to the C++ documentation also hard-codes 2.2.0.
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105881
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
Should we do them all at once?
On Tue, May 15, 2012 at 10:23 PM, Matthias Brantner
matthias.brant...@28msec.com wrote:
Review: Needs Fixing
The link to the C++ documentation also hard-codes 2.2.0.
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105881
You proposed
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/xqjdoc/+merge/105700
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
The proposal to merge lp:~zorba-coders/zorba/xqjdoc into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/xqjdoc/+merge/105700
--
https://code.launchpad.net/~zorba-coders/zorba/xqjdoc/+merge/105700
Your team
Should we do them all at once?
I don't know. I just saw the ones in the merge proposal (i.e. the ones you
changed). I think that if we change them, we should change them to the latest.
On Tue, May 15, 2012 at 10:23 PM, Matthias Brantner
matthias.brant...@28msec.com wrote:
Review: Needs
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/xqjdoc-2012-05-15T20-33-06.579Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/xqjdoc/+merge/105700
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Review: Needs Fixing
let $i := fn:parse-xml-fragment()
return $i instance of document-node()
should return true
The expression fn:parse-xml-fragment() returns a document node having no
children.
--
https://code.launchpad.net/~zorba-coders/zorba/xquery3.0_funcs/+merge/105840
Your team Zorba
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug905028/+merge/104447
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/bug905028 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug905028/+merge/104447
--
https://code.launchpad.net/~zorba-coders/zorba/bug905028/+merge/104447
Validation queue job xqjdoc-2012-05-15T20-33-06.579Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/xqjdoc/+merge/105700
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in src/runtime/full_text/ft_module_impl.cpp
--
https://code.launchpad.net/~zorba-coders/zorba/bug905028/+merge/104447
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug905028 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug905028/+merge/104447
--
https://code.launchpad.net/~zorba-coders/zorba/bug905028/+merge/104447
The proposal to merge lp:~zorba-coders/zorba/bug905028 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug905028/+merge/104447
--
https://code.launchpad.net/~zorba-coders/zorba/bug905028/+merge/104447
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug905028-2012-05-15T21-16-08.381Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug905028/+merge/104447
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug-986377 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/103908
--
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/103908
Validation queue job bug905028-2012-05-15T21-16-08.381Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug905028/+merge/104447
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/bug-986377-2012-05-15T21-53-24.114Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/103908
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
** Changed in: zorba
Status: In Progress = Fix Committed
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/905028
Title:
Base URI can't be set to undefined
Status in Zorba - The XQuery
The attempt to merge lp:~zorba-coders/zorba/bug-986377 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 bug-986377-2012-05-15T21-53-24.114Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/bug-986377 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/103908
--
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/103908
Matthias Brantner has proposed merging lp:~zorba-coders/zorba/ann-vs-an into
lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/ann-vs-an/+merge/105910
use an instead of ann prefix in the documentation
--
The proposal to merge lp:~zorba-coders/zorba/ann-vs-an into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/ann-vs-an/+merge/105910
--
https://code.launchpad.net/~zorba-coders/zorba/ann-vs-an/+merge/105910
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/ann-vs-an/+merge/105910
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/ann-vs-an/+merge/105910
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 :
Matthias Brantner has proposed merging lp:~zorba-coders/zorba/bug-986377 into
lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
Markos Zaharioudakis (markos-za)
Related bugs:
Bug #986377 in Zorba: do not apply any updates on collection if it is to be
truncated
The proposal to merge lp:~zorba-coders/zorba/bug-986377 into lp:zorba has been
updated.
Commit Message changed to:
- no undo is done for collection truncate
- fix for bug #986377 do not apply any updates on collection if it is to be
truncated
For more details, see:
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/105911
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
Looks good except that the fetch module version should probably only be 2.1,
not 3.0. I'm voting Approve now, so if you don't re-submit the proposal (and
don't change anything else besides the version number) you should be able to
get it merged.
--
The proposal to merge lp:~zorba-coders/zorba/bug-986377 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/105911
--
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/105911
Looks good except that the fetch module version should probably only be 2.1,
not 3.0. I'm voting Approve now, so if you don't re-submit the proposal (and
don't change anything else besides the version number) you should be able to
get it merged.
I have changed the version 3.0 = 2.1 as
Validation queue job ann-vs-an-2012-05-16T00-00-09.672Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/ann-vs-an/+merge/105910
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/ann-vs-an into lp:zorba has been
updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/ann-vs-an/+merge/105910
--
https://code.launchpad.net/~zorba-coders/zorba/ann-vs-an/+merge/105910
Your team
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-986377-2012-05-16T00-35-19.597Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/105911
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Paul J. Lucas has proposed merging lp:~zorba-coders/zorba/feature-ft_module
into lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
Paul J. Lucas (paul-lucas)
Related bugs:
Bug #944795 in Zorba: XQDoc doesn't handle in URLs
https://bugs.launchpad.net/zorba/+bug/944795
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/feature-ft_module/+merge/105913
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 :
Paul J. Lucas has proposed merging lp:~zorba-coders/zorba/feature-ft_module
into lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
Paul J. Lucas (paul-lucas)
Related bugs:
Bug #944795 in Zorba: XQDoc doesn't handle in URLs
https://bugs.launchpad.net/zorba/+bug/944795
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/feature-ft_module/+merge/105914
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 bug-986377-2012-05-16T00-35-19.597Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/105911
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 Approve, 2 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/105911
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug-986377 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/105911
--
https://code.launchpad.net/~zorba-coders/zorba/bug-986377/+merge/105911
William Candillon has proposed merging lp:~zorba-coders/zorba/doxygen into
lp:zorba.
Requested reviews:
William Candillon (wcandillon)
Matthias Brantner (matthias-brantner)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/105928
Remove dead links in the
74 matches
Mail list logo