[Freeciv-Dev] [patch #3698] Corrections to docs/README.delta

2013-02-13 Thread Sveinung Kvilhaugsvik
URL:
  http://gna.org/patch/?3698

 Summary: Corrections to docs/README.delta
 Project: Freeciv
Submitted by: sveinung
Submitted on: Wed 13 Feb 2013 12:24:59 PM GMT
Category: None
Priority: 5 - Normal
  Status: None
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 

___

Details:

Freeciv's packet header size is 4 bytes now. Remove a the the at the same
time.



___

File Attachments:


---
Date: Wed 13 Feb 2013 12:24:59 PM GMT  Name: README.delta_corrections.patch 
Size: 795B   By: sveinung

http://gna.org/patch/download.php?file_id=17192

___

Reply to this item at:

  http://gna.org/patch/?3698

___
  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] [patch #3698] Corrections to docs/README.delta

2013-02-13 Thread Sveinung Kvilhaugsvik
Follow-up Comment #1, patch #3698 (project freeciv):

The values of the unchanged fields will be filled in from an old version at
the receiving side. could use a clarification. I had a look at the generated
code when delta is enabled. If it is as simple as it appears, and keys etc
aren't taken into account when selecting the old packet, I suggest changing it
to: The values of the unchanged fields will be filled in from the previous
packet of the same kind received at the receiving side.

___

Reply to this item at:

  http://gna.org/patch/?3698

___
  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] [patch #3698] Corrections to docs/README.delta

2013-02-13 Thread pepeto
Follow-up Comment #2, patch #3698 (project freeciv):

 Freeciv's packet header size is 4 bytes now.

Maybe further clarifications would be welcome to determine how those bytes are
used. Also, I proposed a change in the initial network protocol (in bug
#19943) that could make this path out of date.

 and keys etc aren't taken into account when selecting the old
 packet

Keys *are* taken in account in this case (that's why they are keys).


___

Reply to this item at:

  http://gna.org/patch/?3698

___
  Message posté via/par Gna!
  http://gna.org/


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


[Freeciv-Dev] [bug #20493] utype_by_number() returns invalid pointer

2013-02-13 Thread pepeto
Update of bug #20493 (project freeciv):

 Assigned to:None = pepeto 
 Planned Release: = 2.1.10, 2.2.8, 2.3.4,
2.4.0, 2.5.0


___

Reply to this item at:

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

___
  Message posté via/par Gna!
  http://gna.org/


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


[Freeciv-Dev] [bug #20492] PACKET_UNIT_INFO unused fields

2013-02-13 Thread pepeto
Update of bug #20492 (project freeciv):

 Assigned to:None = pepeto 


___

Reply to this item at:

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

___
  Message posté via/par Gna!
  http://gna.org/


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


[Freeciv-Dev] [patch #3685] dio_get_xxx() returned value

2013-02-13 Thread pepeto
Update of patch #3685 (project freeciv):

 Assigned to:None = pepeto 


___

Reply to this item at:

  http://gna.org/patch/?3685

___
  Message posté via/par Gna!
  http://gna.org/


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


[Freeciv-Dev] [patch #3686] dio_put|get_float()

2013-02-13 Thread pepeto
Update of patch #3686 (project freeciv):

 Assigned to:None = pepeto 
 Planned Release: = 2.5.0  


___

Reply to this item at:

  http://gna.org/patch/?3686

___
  Message posté via/par Gna!
  http://gna.org/


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


[Freeciv-Dev] [patch #3687] Ensure all packets fields have been correctly read

2013-02-13 Thread pepeto
Update of patch #3687 (project freeciv):

 Assigned to:None = pepeto 
 Planned Release: = 2.5.0  


___

Reply to this item at:

  http://gna.org/patch/?3687

___
  Message posté via/par Gna!
  http://gna.org/


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


[Freeciv-Dev] [patch #3688] dio_put_xxx() cleanup

2013-02-13 Thread pepeto
Update of patch #3688 (project freeciv):

 Assigned to:None = pepeto 
 Planned Release: = 2.5.0  


___

Reply to this item at:

  http://gna.org/patch/?3688

___
  Message posté via/par Gna!
  http://gna.org/


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


[Freeciv-Dev] [patch #3684] Effect Enemy_Citizen_Unhappy_Pct

2013-02-13 Thread Marko Lindqvist
Follow-up Comment #1, patch #3684 (project freeciv):

- Fixed effect values in experimental ruleset. Was adding hundreds of percents
(multiplying) instead of subtracting 50/67 (dividing)

(file #17194)
___

Additional Item Attachment:

File name: EftEnemyCitizenUnhappyPct-2.patch Size:4 KB


___

Reply to this item at:

  http://gna.org/patch/?3684

___
  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 #19841] Fundamentalism fails in classic ruleset

2013-02-13 Thread Marko Lindqvist
Update of bug #19841 (project freeciv):

Category:rulesets = general
  Status:None = In Progress
 Assigned to:None = cazfi  
 Planned Release: = 2.5.0  


___

Reply to this item at:

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

___
  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 #17887] Tech prerequisites misdisplayed in help if root_req set

2013-02-13 Thread Marko Lindqvist
Follow-up Comment #2, bug #17887 (project freeciv):

- Cannot set tech goal to anything past immediately reachable tech
- Cannot give upper level techs in editor. This may not be a bug (editor
disallows other setups that are impossible to reach in game) but it would be
nice to be override that rule in the editor, and for testing purposes (I
wanted to test Communistic government)

___

Reply to this item at:

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

___
  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 #19841] Fundamentalism fails in classic ruleset

2013-02-13 Thread Marko Lindqvist
Update of bug #19841 (project freeciv):

  Status: In Progress = Ready For Test 

___

Follow-up Comment #4:

Patch with the compatibility.ignore_govs idea
- Do not warn about Fundamentalism in classic or experimental ruleset
- Turned log message to log_error in other cases

(file #17195)
___

Additional Item Attachment:

File name: IgnoreGovs.patch   Size:6 KB


___

Reply to this item at:

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

___
  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 #20507] Barbarian nations in unknown group in civ2 ruleset

2013-02-13 Thread Marko Lindqvist
URL:
  http://gna.org/bugs/?20507

 Summary: Barbarian nations in unknown group in civ2 ruleset
 Project: Freeciv
Submitted by: cazfi
Submitted on: Thu 14 Feb 2013 03:22:15 AM EET
Category: None
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: None
 Planned Release: 

___

Details:

Loading civ2 ruleset with -d 3:
3: Nation Barbarian: Unknown group Core.
3: Nation Pirate: Unknown group Core.

I'm not sure if this has potential to cause any real harm in game when
barbarians are supposed to appear.




___

Reply to this item at:

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

___
  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 #20508] Nations functional typo fixes

2013-02-13 Thread Marko Lindqvist
URL:
  http://gna.org/bugs/?20508

 Summary: Nations functional typo fixes
 Project: Freeciv
Submitted by: cazfi
Submitted on: Thu 14 Feb 2013 03:31:29 AM EET
Category: rulesets
Severity: 3 - Normal
Priority: 5 - Normal
  Status: Ready For Test
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: None
 Planned Release: 2.5.0

___

Details:

There's a couple of typos in nations files causign functional problems (i.e.,
not in legend). Seen by running server with -d 3. Fix attached.
Moscovite - Muscovite
Afghan - Afghani




___

File Attachments:


---
Date: Thu 14 Feb 2013 03:31:29 AM EET  Name: NationTypos.patch  Size: 960B  
By: cazfi

http://gna.org/bugs/download.php?file_id=17196

___

Reply to this item at:

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

___
  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 #20509] City mini-map issues

2013-02-13 Thread anonymous
URL:
  http://gna.org/bugs/?20509

 Summary: City mini-map issues
 Project: Freeciv
Submitted by: None
Submitted on: Thu Feb 14 06:33:31 2013
Category: client
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: None
 Planned Release: 

___

Details:

Mini-maps on the city overview are (always) shown in landscape format (width 
height), this results in unused (black background) areas at the left and right
sides of the city overview mini-map.

In managed cities the mini-map is greyed out, and the tiles are hard to see
for some users.  It would be better to use a grey background with normally
visible tiles to indicate an enabled governor.

In managed cities clicking on tiles shown in the mini-map has no effect, the
governor has to be disabled before.  It would be better to always allow manual
micro-management by clicking on tiles, just disable the governor switching
back to a black mini-map background.



___

File Attachments:


---
Date: Thu Feb 14 06:33:31 2013  Name: living-in-a -cloud.png  Size: 44kB   By:
None
Screenshot of a greyed out mini-map (German UI)
http://gna.org/bugs/download.php?file_id=17197

___

Reply to this item at:

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

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


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