Bug 4341 - tolua++ is newer in MDV 2010.2 (contrib) updates than Mageia 1
Summary: tolua++ is newer in MDV 2010.2 (contrib) updates than Mageia 1
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 1
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard:
Keywords: validated_update
Depends on:
Blocks:
 
Reported: 2012-01-29 21:59 CET by David Walser
Modified: 2012-03-07 10:58 CET (History)
5 users (show)

See Also:
Source RPM: tolua++-1.0.93-1.mga1.src.rpm
CVE:
Status comment:


Attachments

Description David Walser 2012-01-29 21:59:49 CET
The respective versions are 1.0.93-3mdv2010.2 and 1.0.93-1.mga1

This should be updated so that upgrading from MDV 2010.2 works as expected.

Also, the bugs fixed in the MDV package probably affect ours.

Changelog of 3mdv2010.2:
- Fix tolua++ symbol lookup error: tolua_open
Comment 1 Manuel Hiebel 2012-01-30 01:07:12 CET
Hi, thanks for reporting this bug.
As there is no maintainer for this package I added the committers in CC.

(Please set the status to 'assigned' if you are working on it)

CC: (none) => cazzaniga.sandro

David Walser 2012-01-31 02:59:37 CET

CC: (none) => lists.jjorge

Comment 2 David Walser 2012-01-31 03:00:53 CET
José, I think you recently built this package in Cauldron.  Would you mind looking at this for Mageia 1?
Comment 3 Manuel Hiebel 2012-03-02 17:52:22 CET
The version in cauldron needs also a rebuild
Comment 4 José Jorge 2012-03-02 19:52:12 CET
Ok, tolua++-1.0.93-4.mga1 in updates_testing.

Status: NEW => ASSIGNED

José Jorge 2012-03-02 19:52:45 CET

Assignee: bugsquad => qa-bugs

Comment 5 Dave Hodgins 2012-03-02 21:50:56 CET
I'm trying the example from
http://www.gamedev.net/topic/462040-tolua-tutorial/

I've created the character.cpp file, but I'm getting ...
tolua++ -o tolua_character.cpp -H tolua_character.h -n character character.cpp
tolua++: symbol lookup error: tolua++: undefined symbol: tolua_open

Am I missing something, or is there a problem in the package?

CC: (none) => davidwhodgins

Comment 6 David Walser 2012-03-03 00:17:29 CET
(In reply to comment #5)
> I'm trying the example from
> http://www.gamedev.net/topic/462040-tolua-tutorial/
> 
> I've created the character.cpp file, but I'm getting ...
> tolua++ -o tolua_character.cpp -H tolua_character.h -n character character.cpp
> tolua++: symbol lookup error: tolua++: undefined symbol: tolua_open
> 
> Am I missing something, or is there a problem in the package?

This is the exact error that was fixed in the Mandriva package, according to the changelog entry I cited in the initial bug report.  The changes in the Mandriva package need to be done to our package as well.  The diff is here:
http://svn.mandriva.com/viewvc/packages/cooker/tolua++/current/SPECS/tolua++.spec?r1=615234&r2=769594
Comment 7 Dave Hodgins 2012-03-03 05:13:07 CET
Using tolua++-1.0.93-4.mga1.src.rpm and a slightly modified command

$ tolua++ -o tolua_character.cpp -H tolua++.h -n character character.cpp
[dave@hodgins source]$ gcc -o tolua_character tolua_character.cpp
In file included from tolua_character.cpp:11:0:
tolua++.h:7:1: error: âTOLUA_APIâ does not name a type
followed by a bunch of errors that appear to be caused by this error.

$ cat tolua++.h
/*
** Lua binding: character
** Generated automatically by tolua++-1.0.92 on Fri Mar  2 23:04:50 2012.
*/

/* Exported function */
TOLUA_API int  tolua_character_open (lua_State* tolua_S);

Looks to me like the type is clearly integer, so I don't
know what the error really is. c++ is not in my programming
experience.
Comment 8 José Jorge 2012-03-04 17:29:04 CET
(In reply to comment #6)
> This is the exact error that was fixed in the Mandriva package, according to
> the changelog entry I cited in the initial bug report.  The changes in the
> Mandriva package need to be done to our package as well.  The diff is here:
> http://svn.mandriva.com/viewvc/packages/cooker/tolua++/current/SPECS/tolua++.spec?r1=615234&r2=769594

This changes are done : 
http://svnweb.mageia.org/packages/updates/1/tolua%2B%2B/current/SPECS/tolua%2B%2B.spec?r1=101464&r2=216889
Comment 9 claire robinson 2012-03-04 18:03:58 CET
Testing x86_64

Before
------
# urpmi --media Release tolua++
To satisfy dependencies, the following packages are going to be installed:
   Package                        Version      Release       Arch   
(medium "Core Release")
  lib64tolua++5.1                1.0.93       1.mga1        x86_64  
  tolua++                        1.0.93       1.mga1        x86_64  
163KB of additional disk space will be used.
42KB of packages will be retrieved.
Proceed with the installation of the 2 packages? (Y/n) y

Using Daves example..

created character.cpp with the following contents

 // your real class definition
class Character {
public:
   Character();
   void setPosition(int x, int y);
   int  getXPosition() const;
   int  getYPosition() const;

private:
   int _x, _y;
};
// tolua class prototype
class Character
{
   Character();
   void setPosition(int x, int y);
   int  getXPosition() const;
   int  getYPosition() const;
};

# tolua++ -o tolua_character.cpp -H tolua_character.h -n character character.cpp
tolua++: symbol lookup error: tolua++: undefined symbol: tolua_open


After
-----
# urpmi tolua++
To satisfy dependencies, the following packages are going to be installed:
   Package                        Version      Release       Arch
(medium "Core Updates Testing")
  lib64tolua++5.1                1.0.93       4.mga1        x86_64
  tolua++                        1.0.93       4.mga1        x86_64
432B of additional disk space will be used.
42KB of packages will be retrieved.
Proceed with the installation of the 2 packages? (Y/n) y

# tolua++ -o tolua_character.cpp -H tolua_character.h -n character character.cpp

# ll
total 20
-rw-r--r-- 1 root root   353 Mar  4 16:59 character.cpp
-rw-r--r-- 1 root root 12059 Mar  4 17:00 tolua_character.cpp
-rw-r--r-- 1 root root   190 Mar  4 17:00 tolua_character.h


Testing complete x86_64
Comment 10 David Walser 2012-03-04 18:07:14 CET
(In reply to comment #8)
> (In reply to comment #6)
> > This is the exact error that was fixed in the Mandriva package, according to
> > the changelog entry I cited in the initial bug report.  The changes in the
> > Mandriva package need to be done to our package as well.  The diff is here:
> > http://svn.mandriva.com/viewvc/packages/cooker/tolua++/current/SPECS/tolua++.spec?r1=615234&r2=769594
> 
> This changes are done : 
> http://svnweb.mageia.org/packages/updates/1/tolua%2B%2B/current/SPECS/tolua%2B%2B.spec?r1=101464&r2=216889

Thanks.  Make sure you commit this to Cauldron too.
Comment 11 claire robinson 2012-03-06 15:21:18 CET
Same procedure i586 all Ok.

Validating.

Advisory
----------
This update enables upgrade from Mandriva 2010.2.

It also provides a fix for a symbol lookup error:-

tolua++: symbol lookup error: tolua++: undefined symbol: tolua_open
----------

SRPM: tolua++-1.0.93-4.mga1.src.rpm


Could sysadmin please push from core/updates_testing to core/updates

Thankyou!

Keywords: (none) => validated_update
CC: (none) => sysadmin-bugs
Hardware: i586 => All

Comment 12 Thomas Backlund 2012-03-07 10:58:35 CET
update pushed

Status: ASSIGNED => RESOLVED
CC: (none) => tmb
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.