Follow-up Comment #6, bug #13709 (project freeciv):
Perhaps you could add the patches to this external program code in an extra
directory. This way they are available if a new version has to be patched.
___
Reply to this item at:
Follow-up Comment #7, bug #13709 (project freeciv):
That's a good idea. I think we can be without the first patch for
tolua_outside, see bug #15841.
The second patch for tolua_notnil is still interesting since there is a lot
less duplicate code/duplicate checks needed with it. But we don't need
'char *str' by 'const char *str'
___
File Attachments:
---
Date: Sonntag 11.04.2010 um 15:44 Name:
20100411-02-trunk-cleanup-stdinhand.c-handle_stdin_input.patch Size: 4kB
By: syntron
somethinglike this as it is a security risk. I'm open
for discussion.
___
File Attachments:
---
Date: Sonntag 11.04.2010 um 15:45 Name:
20100411-03-trunk-execute-freeciv-commands
:
___
File Attachments:
---
Date: Sonntag 11.04.2010 um 15:46 Name:
20100411-04-trunk-cleanup-player-name-check.patch Size: 4kB By: syntron
http://gna.org/patch/download.php?file_id=8911
: 20100411-05-trunk-add-a-player-to-a-running-game.patch Size:16 KB
___
Reply to this item at:
http://gna.org/patch/?1574
___
Nachricht geschickt von/durch Gna!
http://gna.org
Sverdrup englabenny (see patch #1574)
___
File Attachments:
---
Date: Sonntag 11.04.2010 um 15:47 Name:
20100411-06-trunk-add-create_player-lua-function.patch Size: 7kB By:
syntron
Update of patch #1616 (project freeciv):
Depends on: = patch #1574
___
Reply to this item at:
http://gna.org/patch/?1616
___
Nachricht
Update of patch #1574 (project freeciv):
Depends on: = patch #1615
___
Reply to this item at:
http://gna.org/patch/?1574
___
Nachricht
Update of patch #1574 (project freeciv):
Depends on: = patch #1613
___
Reply to this item at:
http://gna.org/patch/?1574
___
Nachricht
Update of patch #1614 (project freeciv):
Depends on: = bugs #15709
___
Reply to this item at:
http://gna.org/patch/?1614
___
Nachricht
Update of bug #15825 (project freeciv):
Assigned to:None = jdorje
___
Reply to this item at:
http://gna.org/bugs/?15825
___
Message sent
Update of bug #15830 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #15832 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #1236 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #15839 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #1, patch #1614 (project freeciv):
I guess if caller is handled the same way as for other commands so that
access controls still apply, it will be safe. What happens if caller is
NULL?
*This particular* instance of lua scripting in freeciv is about the game; not
the server,
Follow-up Comment #4, patch #1582 (project freeciv):
Truncation is fundamentally unavoidable unless we strdup-printf every string
we make everywhere, which isn't going to happen. We can cut down on it by
making buffers sufficiently long.
ATM there is an issue with MAX_LEN_NAME. This is
Follow-up Comment #1, bug #15844 (project freeciv):
I forgot to say that this is with S2_2 branch.
___
Reply to this item at:
http://gna.org/bugs/?15844
___
Message sent via/by Gna!
___
Details:
___
File Attachments:
---
Date: Sonntag 11.04.2010 um 22:11 Name:
20100411-01-trunk-remove-newlines-in-log-messages.patch Size: 4kB
, file #8917)
___
Additional Item Attachment:
File name: 20100411-01-S2_2-check-if-ai-is-initialised.patch Size:1 KB
File name: 20100411-02-trunk-check-if-ai-is-initialised-trunk.patch Size:2 KB
Update of bug #15844 (project freeciv):
Category: client-gtk-2.0 = client
___
Reply to this item at:
http://gna.org/bugs/?15844
___
Nachricht
Follow-up Comment #3, bug #15844 (project freeciv):
Thanks. I saw consistent crashes every time a city was destroyed (trunk)
(either real city destruction in the game or when the client quits). I
didn't have time yet to make a clean checkout to testreport it yet. This
seems to be it, the patch
Update of bug #15844 (project freeciv):
Planned Release: 2.3.0 = 2.2.1,2.3.0
___
Follow-up Comment #4:
- if (pcity-owner-ai-funcs.city_close) {
+ if (pcity-owner-ai
Follow-up Comment #5, bug #15844 (project freeciv):
Yes, it seems to fix the problem for me too.
___
Reply to this item at:
http://gna.org/bugs/?15844
___
Message sent via/by Gna!
URL:
http://gna.org/patch/?1617
Summary: Release script objects for removed units and cities
Project: Freeciv
Submitted by: englabenny
Submitted on: söndag 2010-04-11 den 22:10
Category: general
Priority:
Update of patch #1617 (project freeciv):
Depends on: = bugs #13709
___
Reply to this item at:
http://gna.org/patch/?1617
___
Meddelandet
Update of bug #15725 (project freeciv):
Depends on: = patch #1617
___
Reply to this item at:
http://gna.org/bugs/?15725
___
Meddelandet
Follow-up Comment #9, patch #1574 (project freeciv):
I wonder if it should be possible to choose the nation for the player to be
created. It would make sense for scenario scripts. nil/NULL would be random
nation (that part works very well by the way, seems to pick nations of the
same category as
URL:
http://gna.org/patch/?1618
Summary: Scripting/API: Use specific argument error
messages
Project: Freeciv
Submitted by: englabenny
Submitted on: söndag 2010-04-11 den 23:55
Category: None
Priority: 5
Update of patch #1618 (project freeciv):
Depends on: = patch #1601
___
Reply to this item at:
http://gna.org/patch/?1618
___
Meddelandet
31 matches
Mail list logo