The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug923686-2012-09-19T06-06-26.347Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The attempt to merge lp:~zorba-coders/zorba/bug923686 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 bug923686-2012-09-19T06-06-26.347Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug923686-2012-09-19T06-33-57.964Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The attempt to merge lp:~zorba-coders/zorba/bug923686 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 bug923686-2012-09-19T06-33-57.964Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
Stupid Launchpad problem with the email module. Trying again.
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug923686-2012-09-19T08-21-15.404Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The attempt to merge lp:~zorba-coders/zorba/bug923686 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 bug923686-2012-09-19T08-21-15.404Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/bug923686 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
Chris Hillery has proposed merging lp:~zorba-coders/zorba/bug-1015964 into
lp:zorba.
Requested reviews:
Chris Hillery (ceejatec)
Related bugs:
Bug #1015964 in Zorba: Add warning re: SWIG 2.0.7 for PHP binding
https://bugs.launchpad.net/zorba/+bug/1015964
For more details, see:
The proposal to merge lp:~zorba-coders/zorba/bug-1015964 into lp:zorba has been
updated.
Commit Message changed to:
Add note to doc regarding bug in Swig on some platforms.
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
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/bug-1015964 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1015964-2012-09-19T09-18-54.055Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job bug-1015964-2012-09-19T09-18-54.055Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
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.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug-1015964 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1015964/+merge/125137
--
Review: Needs Fixing
--
https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816
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 :
Why is the processPragma method defined only for
static_collections_dml_apply_insert_nodes_last. Should it be defined for the
other collection-insert functions as well?
ChangeLog needs updating
Tiny optimization: if you want to erase the last N entries in a vector, you can
just use resize().
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/hashmap into
lp:zorba.
Requested reviews:
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/hashmap/+merge/125187
HashMap optimizations
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/hashmap/+merge/125187
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/bug1052242/+merge/124823
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_demo into
lp:~zorba-coders/zorba/zorba-experimental has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq_demo/+merge/114238
--
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been
updated.
Commit Message changed to:
Removed fn:parse-xml#2
Fix for bug 950621
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
--
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
--
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
--
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
--
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
--
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-950621-2012-09-19T15-33-07.413Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The attempt to merge lp:~zorba-coders/zorba/bug-950621 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-950621-2012-09-19T15-33-07.413Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
--
https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823
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/bug1052242 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823
--
https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug1052242-2012-09-19T16-02-04.637Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/1025564 into lp:zorba has been
updated.
Status: Merged = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/1025564/+merge/120024
--
https://code.launchpad.net/~zorba-coders/zorba/1025564/+merge/120024
Your team
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-module-bug1006311/+merge/118647
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
Validation queue job bug1052242-2012-09-19T16-02-04.637Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/bug1052242 into lp:zorba has been
updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823
--
https://code.launchpad.net/~zorba-coders/zorba/bug1052242/+merge/124823
Your
The proposal to merge lp:~zorba-coders/zorba/zorba-2.6 into
lp:~zorba-coders/zorba/zorba-experimental has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/zorba-2.6/+merge/120278
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/zorba-2.6/+merge/120278
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/zorba-2.6.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
Markos, can you please review?
--
https://code.launchpad.net/~zorba-coders/zorba/XUDY00021/+merge/93100
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/bug-1044260 into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1044260/+merge/125297
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1044260-2012-09-19T18-38-42.91Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1044260/+merge/125297
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1044260/+merge/125297
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/bug-1044260 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-1044260-2012-09-19T18-38-42.91Z is finished. The
final
The proposal to merge lp:~zorba-coders/zorba/bug-1044260 into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1044260/+merge/125297
--
Why is the processPragma method defined only for
static_collections_dml_apply_insert_nodes_last. Should it be defined for the
other collection-insert functions as well?
ChangeLog needs updating
Tiny optimization: if you want to erase the last N entries in a vector, you
can just use
The proposal to merge lp:~zorba-coders/zorba/feature-pragma_no_copy into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816
--
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/feature-pragma_no_copy/+merge/112816
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/feature-pragma_no_copy into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816
--
I am going to approve, but it would be nice if you added an iter test to make
sure that the optimization is applied and we won't loose it somehow in the
future. To do so, you must first make PrinterVisitor print theNeedToCpy data
member of the various ZorbaInsertNodesXXX iterators. This will
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816
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 :
Rodolfo, you will need to fix all the iter tests that fail. The tests fail
because the var ids assigned to variables have changed.
--
https://code.launchpad.net/~zorba-coders/zorba/bug923686/+merge/124531
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Cezar Andrei has proposed merging
lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module.
Requested reviews:
Chris Hillery (ceejatec)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
Add java.library.path
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into
lp:zorba/util-jvm-module has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
--
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into
lp:zorba/util-jvm-module has been updated.
Commit Message changed to:
Add java.library.path property when starting the jvm
For more details, see:
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-utiljvm-module-2012-09-19T20-32-43.622Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
Your team Zorba Coders is subscribed to branch
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:zorba.
Requested reviews:
Chris Hillery (ceejatec)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Make doc comments for createBaser64Binary more explicit on
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has
been updated.
Commit Message changed to:
Make doc comments for createBaser64Binary more explicit on what parameters they
expect and what they do.
Change, return value to xs_int for getIntValue() method.
For more
Review: Needs Fixing
You changed the URIs of the test modules back to www.zorba-xquery.com, instead
of zorba-tests.28msec.us. That will cause test failures. Make sure you've
merged from the trunk recently.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Your
(Other than that the changes look fine)
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
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 fread-pdf-utiljvm-module-2012-09-19T20-32-43.622Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.
--
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/fread-pdf-utiljvm-module/+merge/125332
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.
--
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into
lp:zorba/util-jvm-module has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
--
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into
lp:zorba/util-jvm-module has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-utiljvm-module-2012-09-19T21-04-41.287Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
Your team Zorba Coders is subscribed to branch
You still haven't corrected the URIs of the test modules, though
(test/rbkt/modules/CMakeLists.txt).
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Review: Needs Fixing
Let me add the new module.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Validation queue job fread-pdf-utiljvm-module-2012-09-19T21-04-41.287Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.
--
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into
lp:zorba/util-jvm-module has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-utiljvm-module/+merge/125332
--
The proposal to merge lp:~zorba-coders/zorba/feature-pragma_no_copy into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816
--
I am going to approve, but it would be nice if you added an iter test to
make sure that the optimization is applied and we won't loose it somehow in
the future. To do so, you must first make PrinterVisitor print theNeedToCpy
data member of the various ZorbaInsertNodesXXX iterators. This will
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/feature-pragma_no_copy-2012-09-19T21-42-49.059Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816
Your team Zorba Coders is subscribed to branch
The ref to the new module is in, please review it too: lp:zorba/read-pdf-module
at
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-the-module .
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Validation queue job feature-pragma_no_copy-2012-09-19T21-42-49.059Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:~zorba-coders/zorba/zorba-experimental has been updated.
Commit Message changed to:
Testing doc for new read-pdf module.
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356
--
The proposal to merge lp:~zorba-coders/zorba/feature-pragma_no_copy into
lp:zorba has been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/feature-pragma_no_copy/+merge/112816
--
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:~zorba-coders/zorba/zorba-experimental has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356
--
Back to zorba-tests.28msec.us in test/rbkt/modules/CMakeLists.txt and mention
in Changelog.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to
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/fread-pdf-trunk/+merge/125356
Your team Zorba Coders is requested to review the proposed merge of
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:~zorba-coders/zorba/zorba-experimental has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356
--
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356
Your team Zorba Coders is subscribed to branch
lp:~zorba-coders/zorba/fread-pdf-trunk.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:~zorba-coders/zorba/zorba-experimental has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356
--
Review: Approve
Great, thanks.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
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/fread-pdf-trunk into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
--
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in ChangeLog
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
--
Review: Needs Fixing
It looks nice (without being able to testing it ;-)
- The module should follow XQuery coding conventions. For example, no camel-case
but function and variable names using dashes.
- The read-pdf:renderToImages-internal function is not documented. This will
probably cause
Review: Needs Fixing
Works great. Some minor issues:
- the --help documentation should mention that the argument is deprecated
- also, the --help documentation should mention the syntax to pass the file
- please respect coding guidelines. For example,
if(qBeforeF()){ = if (qBeforeF())
{
- remove
I fixed all the comments with the exception of the optional occurrence
indicator for internal functions. I tested it with empty sequence and works
fine.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
--
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/fread-pdf-trunk/+merge/125356
Your team Zorba Coders is subscribed to branch
lp:~zorba-coders/zorba/fread-pdf-trunk.
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:~zorba-coders/zorba/zorba-experimental has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125356
--
Validation queue job fread-pdf-trunk-2012-09-20T03-03-43.261Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
1 - 100 of 108 matches
Mail list logo