Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
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
1. still refers to old xqxq namespace
2. this makes me wonder how the test case is working...
--
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
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 :
Voting criteria failed for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
:
Votes: {'Approve': 2, 'Needs commit message': 1}
--
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
Your team
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue
--
Validation queue result for
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
Stage CommitZorba failed.
Check console output at http://jenkins.lambda.nu/job/CommitZorba/196/console to
view the results.
--
The proposal to merge lp:~zorba-coders/zorba/bug1133806_xqxq_base_uri into
lp:zorba has been updated.
Commit Message changed to:
Set the base URI for queries run by ZQ to something valid (the URI of the ZQ
module itself).
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba
The proposal to merge lp:~zorba-coders/zorba/bug1133806_xqxq_base_uri into
lp:zorba has been updated.
Status: Needs review = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
--
https://code.launchpad.net/~zorba-coders/zorba
Validation queue succeeded - proposal merged!
--
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Review: Needs Fixing
As discussed - we likely won't make a more drastic API change anytime soon, and
so we may as well merge this workaround fix. This branch will need to be
updated for the new ZQ module namespace. It would also be really nice if there
was a test case for this behaviour.
--
-coders/zorba/bug1133806_xqxq_base_uri into lp:zorba.
=== modified file 'modules/xqxq/xqxq.xq.src/xqxq.cpp'
--- modules/xqxq/xqxq.xq.src/xqxq.cpp 2013-05-28 00:58:27 +
+++ modules/xqxq/xqxq.xq.src/xqxq.cpp 2013-06-07 11:18:32 +
@@ -469,8 +469,9 @@
String lQueryString = getOneStringArgument
The proposal to merge lp:~zorba-coders/zorba/bug1133806_xqxq_base_uri into
lp:zorba has been updated.
Description changed to:
Just want to see the diff here.
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
--
https
(as being discussed in
email), that would be the better choice.
--
https://code.launchpad.net/~zorba-coders/zorba/bug1133806_xqxq_base_uri/+merge/168020
Your team Zorba Coders is requested to review the proposed merge of
lp:~zorba-coders/zorba/bug1133806_xqxq_base_uri into lp:zorba.
--
Mailing list
14 matches
Mail list logo