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

2012-03-16 Thread William Candillon
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 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/~z

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

2012-03-16 Thread Zorba Build Bot
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/fix_warnings_fedora/+merge/97705 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2012-03-15 Thread Sorin Marian Nasoi
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 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/~z

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

2012-03-15 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 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/~z

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

2012-03-15 Thread Sorin Marian Nasoi
> Works great for me. It still needs one more Approve to be merged :). -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists

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

2012-03-15 Thread William Candillon
Works great for me. -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 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/

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

2012-03-15 Thread Matthias Brantner
The PROJECT_NAMES need to be quoted because they contain spaces. That is, "Zorba C-API". -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 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/fix_warnings_fedora into lp:zorba

2012-03-15 Thread Sorin Marian Nasoi
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 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/~z

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

2012-03-14 Thread Juan Zacarias
Is working on Windows -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97394 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.ne

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

2012-03-14 Thread Matthias Brantner
> > > > > - why were the spaces for the project names removed (e.g. ZorbaC-API > => > > > Zorba > > > > C-API) > > > Also some details would help me understand your issues. > > > > > > FYI: the doxy.config.in were *not* updated by hand. > > > I used doxygen -u doxy.config.in for every file in order

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

2012-03-14 Thread Sorin Marian Nasoi
> > > - The changes in src/runtime/collections/collections_impl.cpp trigger > > another > > > warning on other systems about a return value that is not used. > > Can you please give me more details to see if I can fix both issues? > > Otherwise I will just revert that change. > Just revert the chan

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

2012-03-14 Thread Matthias Brantner
> > - The changes in src/runtime/collections/collections_impl.cpp trigger > another > > warning on other systems about a return value that is not used. > Can you please give me more details to see if I can fix both issues? > Otherwise I will just revert that change. Just revert the change and add a

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

2012-03-14 Thread Sorin Marian Nasoi
> - The changes in src/runtime/collections/collections_impl.cpp trigger another > warning on other systems about a return value that is not used. Can you please give me more details to see if I can fix both issues? Otherwise I will just revert that change. > - why were the spaces for the project n

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

2012-03-14 Thread Matthias Brantner
Review: Needs Fixing - The changes in src/runtime/collections/collections_impl.cpp trigger another warning on other systems about a return value that is not used. - why were the spaces for the project names removed (e.g. ZorbaC-API => Zorba C-API) -- https://code.launchpad.net/~zorba-coders/zo

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

2012-03-14 Thread Sorin Marian Nasoi
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97394 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/~z