Review: Approve
I initially saw some bad memory hits when I did a build of this branch as well,
but after merging the latest trunk and doing a clean build, it all looks good.
I do not see the memory leaks Sorin noted; I ran the url- tests with valgrind,
and the only memory leaks reported were
The proposal to merge lp:~zorba-coders/zorba/xqxq-url-resolver into
lp:zorba/xqxq-module has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/123604
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/xqxq-url-resolver-2012-10-18T06-21-40.181Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/123604
Your team Zorba Coders is subscribed to branch
The attempt to merge lp:~zorba-coders/zorba/xqxq-url-resolver into
lp:zorba/xqxq-module 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 xqxq-url-resolver-2012-10-18T06-21-40.181Z
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/zorba-xqxq-url-resolver-2012-10-18T06-48-00.823Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
Your team Zorba Coders is subscribed to branch
The proposal to merge lp:~zorba-coders/zorba/xqxq-url-resolver into
lp:zorba/xqxq-module has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/123604
--
Whoops, right - this merge proposal won't pass until
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
is merged. Ignore these test failures for now.
--
https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/123604
Your team Zorba Coders is
Validation queue job zorba-xqxq-url-resolver-2012-10-18T06-48-00.823Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
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, 1 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
The proposal to merge lp:~zorba-coders/zorba/zorba-xqxq-url-resolver into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
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/xqxq-url-resolver/+merge/123604
Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
The proposal to merge lp:~zorba-coders/zorba/zorba-xqxq-url-resolver into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/zorba-xqxq-url-resolver-2012-10-18T09-14-46.434Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
Your team Zorba Coders is subscribed to branch
Validation queue job zorba-xqxq-url-resolver-2012-10-18T09-14-46.434Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
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: 2 Approve, 1 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Review: Needs Information
I would really like to avoid this special case and the string comparison on the
function call. What about using a function annotation?
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
Your team Zorba Coders is subscribed to branch
Review: Needs Fixing
I don't feel comfortable with the QName solution. It would be great if HOFs
could be used for this. As far as I understood earlier discussions, this should
have been developed together with Nicolae. Using HOFs would also solve the
static context problem. Maybe the HOF
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos-scratch
into lp:zorba.
Commit message:
changed some scripting tests to cmply with the current syntax of the scripting
spec.
Requested reviews:
Markos Zaharioudakis (markos-za)
For more details, see:
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/130422
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/markos-scratch into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/130422
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2012-10-18T20-19-00.223Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/130422
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Validation queue job markos-scratch-2012-10-18T20-19-00.223Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/130422
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/markos-scratch into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/130422
--
Rodolfo Ochoa has proposed merging lp:~zorba-coders/zorba/bug1068323 into
lp:zorba.
Requested reviews:
Juan Zacarias (juan457)
Chris Hillery (ceejatec)
Related bugs:
Bug #1068323 in Zorba: Zorba don't install zorba_api.jar
https://bugs.launchpad.net/zorba/+bug/1068323
For more details,
The proposal to merge lp:~zorba-coders/zorba/bug1068323 into lp:zorba has been
updated.
Commit Message changed to:
This is working on linux very well, seems that cmake don't like quoted
variables or file variables if they doesn't exist yet.
For more details, see:
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug1068323/+merge/130434
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 :
Chris Hillery has proposed merging lp:~zorba-coders/zorba/xqxq-url-resolver
into lp:zorba/xqxq-module.
Commit message:
Bug 903797: add feature to prepare-main-module() to enable URI mappers and URL
resolvers written in XQuery.
Requested reviews:
Chris Hillery (ceejatec)
Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/xqxq-url-resolver into
lp:zorba/xqxq-module has been updated.
Commit Message changed to:
Bug 903797: add feature to prepare-main-module() to enable URI mappers and URL
resolvers written in XQuery.
For more details, see:
The proposal to merge lp:~zorba-coders/zorba/zorba-xqxq-url-resolver into
lp:zorba has been updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
--
Review: Disapprove
No longer needed with HOF-based XQXQ implementation.
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-xqxq-url-resolver/+merge/123602
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
Review: Approve
Now using HOF!
--
https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460
Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe
The proposal to merge lp:~zorba-coders/zorba/xqxq-url-resolver into
lp:zorba/xqxq-module has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/xqxq-url-resolver-2012-10-19T01-22-47.205Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460
Your team Zorba Coders is subscribed to branch
Validation queue job xqxq-url-resolver-2012-10-19T01-22-47.205Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460
Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-module.
--
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/xqxq-url-resolver/+merge/130460
Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-module.
--
The proposal to merge lp:~zorba-coders/zorba/xqxq-url-resolver into
lp:zorba/xqxq-module has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/xqxq-url-resolver/+merge/130460
Your team Zorba Coders is subscribed to branch lp:zorba/xqxq-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
38 matches
Mail list logo
