[Freeciv-Dev] [bug #20551] failed to compile luasql: not compatible with Lua 5.2

2013-03-04 Thread Matthias Pfafferodt
Follow-up Comment #11, bug #20551 (project freeciv):

file #17357 fixes the bug for me; see also svn 2.5 at
https://build.opensuse.org/project/monitor?project=home%3Asyntron%3Afreeciv

___

Reply to this item at:

  http://gna.org/bugs/?20551

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20551] failed to compile luasql: not compatible with Lua 5.2

2013-03-04 Thread Marko Lindqvist
Update of bug #20551 (project freeciv):

  Status:  Ready For Test = Fixed  
 Assigned to:None = cazfi  
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/bugs/?20551

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20551] failed to compile luasql: not compatible with Lua 5.2

2013-02-26 Thread Jacob Nevins
Follow-up Comment #8, bug #20551 (project freeciv):

Applying the patch (file #17336) I get


cc1: warnings being treated as errors
ls_mysql.c: In function ‘cur_nullify’:
ls_mysql.c:193: error: no return statement in function returning non-void


And indeed that appears to be true. The whole function cur_nullify() is new in
the patch.

(My normal build has SQLite and MySQL backends enabled, but not PostgreSQL).

___

Reply to this item at:

  http://gna.org/bugs/?20551

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20551] failed to compile luasql: not compatible with Lua 5.2

2013-02-26 Thread Jacob Nevins
Follow-up Comment #9, bug #20551 (project freeciv):

(Erm, I should have said that's in dependencies/luasql-2.1.1/src/ls_mysql.c.)

___

Reply to this item at:

  http://gna.org/bugs/?20551

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20551] failed to compile luasql: not compatible with Lua 5.2

2013-02-26 Thread Marko Lindqvist
Follow-up Comment #10, bug #20551 (project freeciv):

No caller expects return value, so making the return value void.

I'm more worried about the fact that this didn't turn out as error in my
build. Despite -Werror it shows it just as an warning.

(file #17357)
___

Additional Item Attachment:

File name: LuasqlEffect230-2.patch.bz2Size:8 KB


___

Reply to this item at:

  http://gna.org/bugs/?20551

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20551] failed to compile luasql: not compatible with Lua 5.2

2013-02-25 Thread Marko Lindqvist
Follow-up Comment #5, bug #20551 (project freeciv):

So, they have no tarballs at all. README's (in our luasql-2.1.1 directory, and
in their latest tree) refer itself as luasql-2.1, homepage says 2.2 is the
latest, git has no 2.2 tag but has 2.3 tag...


___

Reply to this item at:

  http://gna.org/bugs/?20551

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20551] failed to compile luasql: not compatible with Lua 5.2

2013-02-25 Thread Marko Lindqvist
Update of bug #20551 (project freeciv):

  Status:None = Ready For Test 
 Planned Release: = 2.5.0  

___

Follow-up Comment #6:

Attached patch updates our copy of luasql parts to ones found from v2.3.0 git
tag. For simplicity of this patch I've left directory rename to future ticket.
File Version has been added to describe what the version actually is. 

(file #17336)
___

Additional Item Attachment:

File name: LuasqlEffect230.patch.bz2  Size:8 KB


___

Reply to this item at:

  http://gna.org/bugs/?20551

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20551] failed to compile luasql: not compatible with Lua 5.2

2013-02-25 Thread Jacob Nevins
Follow-up Comment #7, bug #20551 (project freeciv):

...issue 7 https://github.com/keplerproject/luasql/issues/7 claims that
2.4.0 exists (haven't checked)...

___

Reply to this item at:

  http://gna.org/bugs/?20551

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20551] failed to compile luasql: not compatible with Lua 5.2

2013-02-24 Thread Jacob Nevins
Update of bug #20551 (project freeciv):

 Summary: failed to compile luasql = failed to compile
luasql: not compatible with Lua 5.2


___

Reply to this item at:

  http://gna.org/bugs/?20551

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev