The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has
been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126824
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merg
Validation queue job fread-pdf-trunk-2012-10-02T15-03-54.622Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126824
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-10-02T15-03-54.622Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126824
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
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/126824
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
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/126824
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
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/126824
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-c
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has
been updated.
Commit Message changed to:
Add read-pdf module for getting text and rendered images from pdf documents.
Make doc comments for createBaser64Binary more explicit on what parameters they
expect and what th
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126824
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/~zorb
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126824
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/~zorb
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/fread-pdf-trunk/+merge/126823
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: htt
Validation queue job fread-pdf-trunk-2012-09-27T23-13-49.404Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126824
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/
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/126824
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Review: Approve
Now with tag for read-pdf.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126824
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 : h
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:zorba.
Requested reviews:
Cezar Andrei (cezar-andrei)
Juan Zacarias (juan457)
Matthias Brantner (matthias-brantner)
Related bugs:
Bug #1012417 in Zorba: "PDF to XML data convertor"
https://bugs.launchpad.ne
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/126823
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-09-27T23-13-49.404Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126823
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126823
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/~zorb
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/126823
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Review: Approve
Merged and solved conflict.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126823
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.
Commit Message changed to:
Add read-pdf module for getting text and rendered images from pdf documents.
Make doc comments for createBaser64Binary more explicit on what parameters they
expect and what th
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:zorba.
Commit message:
Add read-pdf module for getting text and rendered images from pdf documents.
Make doc comments for createBaser64Binary more explicit on what parameters they
expect and what they do.
Change, re
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/126822
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in modules/ExternalModules.conf
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126822
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Po
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/126822
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Review: Approve
Merged and solved conflict.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126822
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.
Commit Message changed to:
Add read-pdf module for getting text and rendered images from pdf documents.
Make doc comments for createBaser64Binary more explicit on what parameters they
expect and what th
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:zorba.
Commit message:
Add read-pdf module for getting text and rendered images from pdf documents.
Make doc comments for createBaser64Binary more explicit on what parameters they
expect and what they do.
Change, re
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/126743
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in modules/ExternalModules.conf
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126743
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Po
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126743
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/~zorb
Review: Approve
Fixed fonts on build box and changed tests to be machine independent.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126743
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : 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/126743
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
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/126743
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
The attempt to merge lp:~zorba-coders/zorba/fread-pdf-trunk 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 fread-pdf-trunk-2012-09-27T19-32-56.467Z is finished.
The
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-09-27T19-32-56.467Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126743
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has
been updated.
Description changed to:
Add read-pdf module for getting text and rendered images from pdf documents.
Make doc comments for createBaser64Binary more explicit on what parameters they
expect and what they
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has
been updated.
Commit Message changed to:
Add read-pdf module for getting text and rendered images from pdf documents.
Make doc comments for createBaser64Binary more explicit on what parameters they
expect and what th
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126743
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/~zorb
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/126743
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:zorba.
Requested reviews:
Cezar Andrei (cezar-andrei)
Matthias Brantner (matthias-brantner)
Juan Zacarias (juan457)
Related bugs:
Bug #1012417 in Zorba: "PDF to XML data convertor"
https://bugs.launchpad.ne
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/126568
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
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/126568
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-c
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/126568
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
It's strange that a few tests where failing because pdfbox was trying to use
lob4j. The website clearly says it depends on commons-logging which can use
log4j if available but fall down to java's default logging api.
I added explicit checks for jempbox and commons-lib to read-pdf module. And
al
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/126568
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
The attempt to merge lp:~zorba-coders/zorba/fread-pdf-trunk 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 fread-pdf-trunk-2012-09-27T01-51-44.419Z is finished.
The
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-09-27T01-51-44.419Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126568
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
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/126568
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
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/126568
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
The attempt to merge lp:~zorba-coders/zorba/fread-pdf-trunk 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 fread-pdf-trunk-2012-09-26T23-05-52.544Z is finished.
The
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-09-26T23-05-52.544Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126568
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126568
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/~zorb
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/126568
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has
been updated.
Commit Message changed to:
Changes to trunk for read-pdf module.
Make doc comments for createBaser64Binary more explicit on what parameters they
expect and what they do.
Change, return value to xs_int f
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:zorba.
Commit message:
Changes to trunk for read-pdf module.
Make doc comments for createBaser64Binary more explicit on what parameters they
expect and what they do.
Change, return value to xs_int for getIntValue()
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/125858
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
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/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-c
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/125858
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Review: Approve
I added texts for all functions.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscri
Error message from failed test:
:127,18:
Zorba error [zerr:ZXQP0008]:
"{http://www.zorba-xquery.com/modules/read-pdf}extract-text-internal#2":
function implementation not found
Remember that the RQ runs in release mode.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge
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/125858
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
The attempt to merge lp:~zorba-coders/zorba/fread-pdf-trunk 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 fread-pdf-trunk-2012-09-26T04-11-47.786Z is finished.
The
The read-pdf module was found and built with this latest RQ run.
However, it has no tests. I personally don't think that's acceptable, even for
a beta release.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-09-26T04-11-47.786Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
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/125858
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
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/125858
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
The attempt to merge lp:~zorba-coders/zorba/fread-pdf-trunk 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 fread-pdf-trunk-2012-09-25T23-51-47.915Z is finished.
The
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-09-25T23-51-47.915Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
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/125858
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Review: Approve
I've modified read-pdf's CMakeLists.txt to use ZORBA_FIND_JAR instead of
FIND_FILE.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post
I have re-assigned my review request to Juan, since I will not be around much
longer to vote on this proposal.
Juan, all you need to review is the CMake stuff in the lp:zorba/read-pdf-module
branch, specifically to ensure that it uses FIND_ZORBA_JAR() appropriately.
Once Cezar makes that change
Cezar - The remote queue is still not finding the jars, although they are
installed correctly in /usr/share/java as requested. (This is also probably why
the experimental build isn't including the module.) The problem, I believe, is
that you are using FIND_FILE() to find them, and you only list
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/125858
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
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/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: htt
Validation queue job fread-pdf-trunk-2012-09-25T20-57-55.26Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-09-25T20-57-55.26Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
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/125858
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
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/~zorb
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/125858
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
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/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: htt
Validation queue job fread-pdf-trunk-2012-09-22T21-29-56.025Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-09-22T21-29-56.025Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has
been updated.
Commit Message changed to:
Add read-pdf module, which reads the text from a pdf doc and renders its pages
to images.
Add java.library.path to the jvm in util-jvm module.
Make doc comments for createBase
The proposal to merge lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba has
been updated.
Commit Message changed to:
Add read-pdf module, which reads the text from a pdf doc and renders it's pages
to images.
Add java.library.path to the jvm in util-jvm module.
Make doc comments for createBas
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/125858
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125858
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/~zorb
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/fread-pdf-trunk into
lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
Cezar Andrei (cezar-andrei)
Chris Hillery (ceejatec)
Related bugs:
Bug #1012417 in Zorba: "PDF to XML data convertor"
https://bugs.launchpad.n
See answers inline:
- the error seems to be too general, essentially it always raises
JAVA-EXCEPTION no matter what goes wrong (e.g. it the given input is not a
valid pdf)
I adapted the error msg to be more clear/specific.
- the java stack trace seems to be sent to standard error
Goes to std err
Review: Needs Fixing
The module works pretty decent. I was able to extract text or generate images
for several pdfs without any problems.
There are some minor things that should be discussed and/or fixed:
- the error seems to be too general, essentially it always raises
JAVA-EXCEPTION no matter
Paul,
If you're asking about my last commit msg, it is as it should be just one line
change, by deleting the tag name, it makes trunk use the latest HEAD version of
the module branch. The files in util-jvm were already reviewed and merged see
this merge:
https://code.launchpad.net/~zorba-coder
It doesn't look like the fpdf.* files were added via "bzr add" -- were they
supposed to be?
--
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.
Commit Message changed to:
Make doc comments for createBase64Binary more explicit on what parameters they
expect and what they do.
Change return value to xs_int for getIntValue() method.
For more detai
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
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1,
Needs Fixing < 1, Pending < 1. Got: 1 Approve, 2 Needs Fixing.
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list
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: https://launchpad.net/
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fread-pdf-trunk-2012-09-20T03-03-43.261Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/125338
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
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
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
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.
--
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
the
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
--
https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk
1 - 100 of 112 matches
Mail list logo