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

2013-04-17 Thread Chris Hillery
FYI, the ruby test at least failed because it tried to open this file:

/home/ceej/zo/testing/zorbatest/tester/remotequeue/bug1132937-2013-04-17T01-00-27.847Z/bzr/zorba/build/swig/ruby/../tests/satellites.json

but that file doesn't exist. However, if you remove the ../ from the path, it 
exists. So the path is being computed incorrectly somewhere.

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Chris Hillery
php_test10 failed because the .php file uses a heredoc, and that syntax was 
evidently introduced in PHP 5.3. The RQ box has PHP 5.2.9. The test will need 
to be changed.


php2 failed with the following message:

Fatal error: No matching function for overloaded 'ItemFactory_createInteger' in 
/home/ceej/zo/testing/zorbatest/tester/remotequeue/bug1132937-2013-04-17T01-00-27.847Z/bzr/zorba/build/swig/php/Zorba/zorba_api_wrapper.php
 on line 2392

so that seems like maybe it's a real bug.
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Zorba Build Bot
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/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug1132937-2013-04-17T17-05-15.761Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba failed. 
Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 
(message):
  Validation queue job bug1132937-2013-04-17T17-05-15.761Z is finished.  The
  final status was:

  

  2 tests did not succeed - changes not commited.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug1132937-2013-04-17T17-56-14.606Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread Zorba Build Bot
Validation queue job bug1132937-2013-04-17T17-56-14.606Z is finished. The final 
status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-17 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Approved = Merged

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Matthias Brantner
Review: Needs Fixing

- include/zorba/item_factory.h
  - createDateTimeStamp is not correctly indented and should go to the end of 
the file
  - the type of the timezone argument should be a long and the value indicate 
seconds (see also bug #1103115)
- Why do you remove the catch clause in src/store/naive/simple_pul.cpp?
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Rodolfo Ochoa
- include/zorba/item_factory.h
  - createDateTimeStamp is not correctly indented and should go to the end of 
the file
DONE

  - the type of the timezone argument should be a long and the value indicate 
seconds (see also bug #1103115)
This is a big change, it involves the way the TimeZone is managed internally, 
this is not the scope of this bug, this will require much more time and I think 
the changes for this bug need to be released.
The changes on DateTimeStamp where merged from bug #924756, so, perhaps this 
new behavior can be required on bug #1103115 or on a new bug.

- Why do you remove the catch clause in src/store/naive/simple_pul.cpp?
Because the next clause does exactly the same.

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Rodolfo Ochoa
about [src/store/naive/simple_pul.cpp] was just a windows warning removal.
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Matthias Brantner
   - the type of the timezone argument should be a long and the value indicate
 seconds (see also bug #1103115)
 This is a big change, it involves the way the TimeZone is managed internally,
 this is not the scope of this bug, this will require much more time and I
 think the changes for this bug need to be released.
 The changes on DateTimeStamp where merged from bug #924756, so, perhaps this
 new behavior can be required on bug #1103115 or on a new bug.
I leave the decision to Chris whether we want to extend the already broken API 
or do it once the bug is fixed.

 
 - Why do you remove the catch clause in src/store/naive/simple_pul.cpp?
 Because the next clause does exactly the same.
OK. I hope Markos doesn't use this clause regularly.

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug1132937-2013-04-17T00-13-20.738Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba failed. 
Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 
(message):
  Validation queue job bug1132937-2013-04-17T00-13-20.738Z is finished.  The
  final status was:

  

  No tests were run - build or configure step must have failed.

  Not commiting changes.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug1132937-2013-04-17T00-27-35.962Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba failed. 
Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 
(message):
  Validation queue job bug1132937-2013-04-17T00-27-35.962Z is finished.  The
  final status was:

  

  No tests were run - build or configure step must have failed.

  Not commiting changes.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug1132937-2013-04-17T01-00-27.847Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba failed. 
Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 
(message):
  Validation queue job bug1132937-2013-04-17T01-00-27.847Z is finished.  The
  final status was:

  

  3 tests did not succeed - changes not commited.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-04-16 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-27 Thread Chris Hillery
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-27 Thread Chris Hillery
If xs:dateTimeStamp can be added quickly, do it as part of this merge. If it'll 
take more than a few hours, file a separate bug and propose the change 
separately.
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-27 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Commit Message changed to:

Fixes for API
- Check for JSONiq usage from all languages API
- Added JSON items to ItemFactory
- Using generated zorba_api.php
- Fix for std::vector usage in Ruby
- Added diagnostic handler on some tests
- Fix on C++ API that prevents to compile to any using the API
- General small-bugfixing and CMAKE script improvement
- Added dateTimeStamp to the API
- Fix for DocumentType in TypeIdentifier Class

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-27 Thread Cezar Andrei
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Description changed to:

Fixes for API

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Commit Message changed to:

Fixes for API

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug1132937-2013-03-26T12-57-14.27Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba failed. 
Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 
(message):
  Validation queue job bug1132937-2013-03-26T12-57-14.27Z is finished.  The
  final status was:

  

  No tests were run - build or configure step must have failed.

  Not commiting changes.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug1132937-2013-03-26T23-29-14.147Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba failed. 
Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 
(message):
  Validation queue job bug1132937-2013-03-26T23-29-14.147Z is finished.  The
  final status was:

  

  14 tests did not succeed - changes not commited.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155478
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Commit Message changed to:

Fixes for API
- Check for JSONiq usage within API
- Added JSON items to ItemFactory
- Using generated zorba_api.php
- Fix for std::vector usage in Ruby
- Added diagnostic handler on some tests
- Fix on C++ API that prevents to compile to any using the API
- General small-bugfixing and CMAKE script improvement

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Commit Message changed to:

Fixes for API
- Check for JSONiq usage from all languages API
- Added JSON items to ItemFactory
- Using generated zorba_api.php
- Fix for std::vector usage in Ruby
- Added diagnostic handler on some tests
- Fix on C++ API that prevents to compile to any using the API
- General small-bugfixing and CMAKE script improvement

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Rodolfo Ochoa
The proposal to merge lp:~zorba-coders/zorba/bug1132937 into lp:zorba has been 
updated.

Description changed to:

Fixes for API
- Check for JSONiq usage from all languages API
- Added JSON items to ItemFactory
- Using generated zorba_api.php
- Fix for std::vector usage in Ruby
- Added diagnostic handler on some tests
- Fix on C++ API that prevents to compile to any using the API
- General small-bugfixing and CMAKE script improvement

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Cezar Andrei
Review: Needs Fixing

Please also add xs:dateTimeStamp to the API.

I see you removed zorba_api_wrapper.php and zorba_api.php and php tests 3, 12, 
13, 14. Why?
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Rodolfo Ochoa
Every time swig generates a PHP api, it generates zorba_api.php, but this file 
is bugged, so, since a while (don't know when) we used a fixed version, which 
is called zorba_api_wrapper.php, this obviously needs to be updated from time 
to time so it can include recent changes from the api, zorba_api.php was just 
garbage from one attempt trying to have an updated version.
Now, I added some post-compiling code that fixes these swig error after 
zorba_api.php is generated and it copies this files to zorba_api_wrapper.php so 
the users and installer can use this file as before.
It's fixed!
Test 3 has very PHP particular errors according to my testing:
- on windows try-catch errors from Zorba cause a segfault error, a workaround 
is the DiagnosticHandler
- on linux, the DiagnosticHandler causes a segfault error, but try-catch 
without it work fine
so, for now a put test 3 with .disabled extension, so it is now ignored.
Since the purpose of this bug is not to fix this particular problem, I just 
disabled this test.
I rearranged all tests, disabling 3 and erasing 12,13,14 which where garbage.

adding xs:dateTimeStamp, please wait for an update on this.
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Rodolfo Ochoa
 English typo
I rearranged all tests, disabling 3 and erasing 12,13,14 which were garbage.
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp


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

2013-03-26 Thread Rodolfo Ochoa
xs:dateTimeStamp is not yet in C++ API, 
do you want me to add this in current bug?
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
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
More help   : https://help.launchpad.net/ListHelp