The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/142005
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/142005
Your team
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/~paul-lucas/zorba/pjl-misc/+merge/142005
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Review: Needs Fixing
Hi Matthias,
It looks good! I have the following comments:
- 104 : JSON reference - did you mean JSON array?
- Why return an error if ref:dereference is called with an unrecognized URI,
while returning the empty sequence if it is recognized but unbound? Can we
unify and
I am not sure about switching back to xs:string instead of xs:anyURI - somehow,
I find it semantically weird. Perhaps this deserves an offline discussion?
--
https://code.launchpad.net/~zorba-coders/zorba/feature-reference_module/+merge/138345
Your team Zorba Coders is subscribed to branch
The proposal to merge lp:~nbrinza/zorba/error-messages into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
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 starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/error-messages-2013-01-07T18-19-01.443Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug-1053113 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/error-messages-2013-01-07T19-50-48.489Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Review: Needs Fixing
The following query causes a segmentation fault:
import module namespace dt = http://www.zorba-xquery.com/modules/datetime;;
dt:parse-date( 23, %h, de-DE )
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
Your team Zorba Coders is subscribed to
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/error-messages-2013-01-07T20-36-47.472Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
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/error-messages-2013-01-07T21-22-48.853Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
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/error-messages-2013-01-07T23-40-46.807Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
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/error-messages-2013-01-08T00-26-54.704Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~nbrinza/zorba/error-messages into lp:zorba has been
updated.
Description changed to:
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
Your team
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/error-messages-2013-01-08T01-12-53.098Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The following query causes a segmentation fault:
import module namespace dt = http://www.zorba-xquery.com/modules/datetime;;
dt:parse-date( 23, %h, de-DE )
This is now fixed.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
Your team Zorba Coders is subscribed
The proposal to merge lp:~zorba-coders/zorba/bug-1053113 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
--
The proposal to merge lp:~zorba-coders/zorba/bug-1053113 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
--
Review: Approve
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/142005
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:~paul-lucas/zorba/pjl-misc into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/142005
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/142005
Your team
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/error-messages-2013-01-08T01-58-49.912Z/log.html
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~nbrinza/zorba/error-messages into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/pjl-misc-2013-01-08T02-10-51.679Z/log.html
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/142005
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Nicolae - something seems to be weird with either your changes or with the RQ
in general. As you see it started the queue a dozen times. Each time it timed
out running the following test. Could your changes somehow be causing this test
to hang?
211: Test command:
The attempt to merge lp:~nbrinza/zorba/error-messages 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 error-messages-2013-01-08T01-58-49.912Z is finished.
The final
I've killed the most recent run manually. Before I did so I noted that
testdriver was stuck spinning 100% of the CPU on the RQ machine, so it
appears there's an infinite loop.
--
https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/142171
Your team Zorba Coders is subscribed to
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been
updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/142005
--
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/142005
Your team Zorba
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/bug-1053113/+merge/141618
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug-1053113 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
--
The proposal to merge lp:~zorba-coders/zorba/bug-1053113 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1053113-2013-01-08T03-14-01.158Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job bug-1053113-2013-01-08T03-14-01.158Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
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 Needs Fixing.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug-1053113 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1053113/+merge/141618
--
Review: Needs Fixing
I like the module a lot. There are only a couple of minor things that need
fixing:
- There are a bunch of typos in the documentation. Could you please run it
through a spell-checker?
For example,
s/corespond/correspond
s/initiaized/initialized
s/corectly/correctly
- the
Hi Matthias,
It looks good! I have the following comments:
- 104 : JSON reference - did you mean JSON array?
fixed
- Why return an error if ref:dereference is called with an unrecognized URI,
while returning the empty sequence if it is recognized but unbound? Can we
unify and always
I am not sure about switching back to xs:string instead of xs:anyURI -
somehow, I find it semantically weird. Perhaps this deserves an offline
discussion?
Yes. Both have pros and cons. Having xs:string is consistent with fn:doc.
Also, it makes it easier for the user. For instance, the user
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/1039576/+merge/123170
Your team Zorba Coders is subscribed to branch lp:zorba/archive-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
The proposal to merge lp:~zorba-coders/zorba/1039576 into
lp:zorba/archive-module has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/1039576/+merge/123170
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/1039576-2013-01-08T05-24-49.124Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/1039576/+merge/123170
Your team Zorba Coders is subscribed to branch lp:zorba/archive-module.
--
Mailing
Validation queue job 1039576-2013-01-08T05-24-49.124Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/1039576/+merge/123170
Your team Zorba Coders is subscribed to branch lp:zorba/archive-module.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/1039576 into
lp:zorba/archive-module has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/1039576/+merge/123170
--
https://code.launchpad.net/~zorba-coders/zorba/1039576/+merge/123170
43 matches
Mail list logo
