Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug938320 into lp:zorba

2012-08-28 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug938320/+merge/118194 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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug938320b into lp:zorba

2012-09-10 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug938320b/+merge/123356 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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-1046415 into lp:zorba

2012-09-13 Thread Cezar Andrei
I understand, it make sense in this way but it's very dangerous, in this way I don't think it helps the users on contrary can leave bad code falsely working. Cezar On Thu, Sep 13, 2012 at 6:38 AM, Ghislain Fourny ghislain.fou...@28msec.com wrote: Hi Cezar, In general, an object constructor

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-zorbaxquerycom into lp:zorba

2012-09-16 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix-zorbaxquerycom/+merge/124540 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq_demo into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
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 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/zorba-2.6 into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
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 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/zorba-2.6 into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Cezar Andrei
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 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Cezar Andrei
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:

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-utiljvm-module into lp:zorba/util-jvm-module

2012-09-19 Thread Cezar Andrei
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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
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. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
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 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
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 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-19 Thread Cezar Andrei
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 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
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. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-19 Thread Cezar Andrei
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 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/zorba-2.6 into lp:~zorba-coders/zorba/zorba-experimental

2012-09-20 Thread Cezar Andrei
This is the sandbox for the live demo page. Since you added libgeos to the build box, I want to trigger a new build. I don't know what this error is. Where do you get this error? -- https://code.launchpad.net/~zorba-coders/zorba/zorba-2.6/+merge/120278 Your team Zorba Coders is subscribed to

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-21 Thread Cezar Andrei
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:

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-22 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-22 Thread Cezar Andrei
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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-22 Thread Cezar Andrei
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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-22 Thread Cezar Andrei
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 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-22 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-25 Thread Cezar Andrei
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 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-26 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-26 Thread Cezar Andrei
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 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-26 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-26 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-26 Thread Cezar Andrei
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 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-26 Thread Cezar Andrei
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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-26 Thread Cezar Andrei
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 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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.net

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-27 Thread Cezar Andrei
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 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-27 Thread Cezar Andrei
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/126757 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-27 Thread Cezar Andrei
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/126757 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:~zorba-coders/zorba/zorba-experimental

2012-09-27 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fread-pdf-trunk/+merge/126757 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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-09-27 Thread Cezar Andrei
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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/schema-tools-module-doc into lp:zorba/schema-tools-module

2012-09-28 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/schema-tools-module-doc/+merge/126944 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/data-formatting-doc into lp:zorba/process-module

2012-09-28 Thread Cezar Andrei
Review: Needs Fixing William, I think you chose the wrong branch to merge into. Pls check again. -- https://code.launchpad.net/~zorba-coders/zorba/data-formatting-doc/+merge/126953 Your team Zorba Coders is subscribed to branch lp:zorba/process-module. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/xslt-doc into lp:zorba/languages-module

2012-09-28 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/xslt-doc/+merge/126969 Your team Zorba Coders is subscribed to branch lp:zorba/languages-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-10-02 Thread Cezar Andrei
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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fread-pdf-trunk into lp:zorba

2012-10-02 Thread Cezar Andrei
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 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/process_doc into lp:zorba/process-module

2012-10-03 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/process_doc/+merge/126738 Your team Zorba Coders is subscribed to branch lp:zorba/process-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2012-11-08 Thread Cezar Andrei
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba. Commit message: Add new nosql-db module. Requested reviews: Juan Zacarias (juan457) Matthias Brantner (matthias-brantner) Related bugs: Bug #1056953 in Zorba: NOSQL DB module https

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2012-11-08 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba has been updated. Commit Message changed to: Add new nosql-db module. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/133585 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2012-11-08 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/133585 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2012-11-08 Thread Cezar Andrei
Please review the code in https://code.launchpad.net/~zorba-coders/zorba/f-nosqldb-module for the new nosql-db module. -- https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/133585 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fots_driver into lp:zorba

2012-11-14 Thread Cezar Andrei
Review: Approve One suggestion would be to add one more switch to output a certain test case into a file to be really easy to debug it quickly. -- https://code.launchpad.net/~zorba-coders/zorba/fots_driver/+merge/133988 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/data-formatting-doc into lp:zorba/data-formatting-module

2012-11-26 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/data-formatting-doc/+merge/127666 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/data-formatting-doc into lp:zorba/data-formatting-module

2012-11-26 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/data-formatting-doc into lp:zorba/data-formatting-module has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/data-formatting-doc/+merge/127666 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2012-11-26 Thread Cezar Andrei
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba. Requested reviews: Juan Zacarias (juan457) Matthias Brantner (matthias-brantner) Related bugs: Bug #1056953 in Zorba: NOSQL DB module https://bugs.launchpad.net/zorba/+bug/1056953 For more details

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2012-11-26 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba has been updated. Commit Message changed to: Add new oracle-nosql-db module. This has been renamed and it also contains multi-get and multi-delete functions. For more details, see:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2012-11-26 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/136215 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1073964-valInternalError into lp:zorba

2012-11-26 Thread Cezar Andrei
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/bug1073964-valInternalError into lp:zorba. Commit message: Fix internal error. Requested reviews: Cezar Andrei (cezar-andrei) Sorin Marian Nasoi (sorin.marian.nasoi) Related bugs: Bug #1073964 in Zorba: FOTS: validate returns

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1073964-valInternalError into lp:zorba

2012-11-26 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/bug1073964-valInternalError into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1073964-valInternalError/+merge/136295 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1073964-valInternalError into lp:zorba

2012-11-26 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/bug1073964-valInternalError into lp:zorba has been updated. Commit Message changed to: Fix internal error. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1073964-valInternalError/+merge/136295 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1073964-valInternalError into lp:zorba

2012-11-26 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug1073964-valInternalError/+merge/136295 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-04 Thread Cezar Andrei
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module. Commit message: Fix for bug 1086323. Requested reviews: Sorin Marian Nasoi (sorin.marian.nasoi) Paul J. Lucas (paul-lucas) Related bugs: Bug #1086323 in Zorba: Process module

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-04 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1086323-processExitCode/+merge/137966 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-04 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module has been updated. Commit Message changed to: Fix for bug 1086323. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1086323-processExitCode/+merge/137966 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-04 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module has been updated. Description changed to: Fix for bug 1086323. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1086323-processExitCode/+merge/137966 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-04 Thread Cezar Andrei
Interesting Ubuntu and Fedora don't implement this the same way. On Ubuntu running a bash script or the process xq script which calls a program that segFaults, the exit code is 139. The macro WIFEXITED(stat) returns true and WEXITSTATUS(stat) returns 139. Which btw means acording to the link

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-04 Thread Cezar Andrei
it's executing a shell with the commands, this way one can have access to shell commands. -- https://code.launchpad.net/~zorba-coders/zorba/bug1086323-processExitCode/+merge/137966 Your team Zorba Coders is subscribed to branch lp:zorba/process-module. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-05 Thread Cezar Andrei
Cezar Andrei has proposed merging lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module. Requested reviews: Sorin Marian Nasoi (sorin.marian.nasoi) Paul J. Lucas (paul-lucas) Chris Hillery (ceejatec) Related bugs: Bug #1086323 in Zorba: Process module: exit code

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-05 Thread Cezar Andrei
Using the usual exit codes convention, as per our todays' talk. -- https://code.launchpad.net/~zorba-coders/zorba/bug1086323-processExitCode/+merge/138266 Your team Zorba Coders is subscribed to branch lp:zorba/process-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-05 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module has been updated. Commit Message changed to: Fix for bug 1086323, align Fedora and Ubuntu when process module executes a command that crashes. For more details, see:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-05 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1086323-processExitCode/+merge/138266 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1086323-processExitCode into lp:zorba/process-module

2012-12-05 Thread Cezar Andrei
Fixed xqdoc. Also I tested it on Ubuntu, segfault comes with exit code 139. To test it one needs to call process:exec on a program that ends with a segmentation fault. -- https://code.launchpad.net/~zorba-coders/zorba/bug1086323-processExitCode/+merge/138266 Your team Zorba Coders is

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/llvm into lp:zorba

2012-12-26 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/llvm/+merge/141233 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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1049378-jvm into lp:zorba/util-jvm-module

2013-01-04 Thread Cezar Andrei
Review: Needs Fixing Must fix the else branch. -- https://code.launchpad.net/~zorba-coders/zorba/bug1049378-jvm/+merge/141692 Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2013-01-04 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/136215 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2013-01-08 Thread Cezar Andrei
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 Fixed. - the

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2013-01-08 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/136215 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2013-01-08 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/136215 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/couchbase-trunk into lp:zorba

2013-01-09 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/couchbase-trunk/+merge/135779 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2013-01-09 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/136215 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

2013-01-09 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/f-nosql-db-trunk/+merge/136215 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/couchbase-trunk into lp:zorba

2013-01-09 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/couchbase-trunk into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/couchbase-trunk/+merge/135779 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/2.8-release-prep into lp:zorba

2013-01-14 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/2.8-release-prep into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/2.8-release-prep/+merge/142856 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/2.8-release-prep into lp:zorba

2013-01-14 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/2.8-release-prep/+merge/142856 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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1100380 into lp:zorba/archive-module

2013-01-16 Thread Cezar Andrei
Review: Approve Works fine with Macports now. -- https://code.launchpad.net/~zorba-coders/zorba/bug1100380/+merge/143607 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

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1100380 into lp:zorba/archive-module

2013-01-16 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/bug1100380 into lp:zorba/archive-module has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1100380/+merge/143607 --

<    1   2   3   4   5   >