[Freeciv-Dev] [patch #5970] JSON protocol: support sending and receiving a requirement.

2015-03-30 Thread Sveinung Kvilhaugsvik
URL:
  

 Summary: JSON protocol: support sending and receiving a
requirement.
 Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 30 Mar 2015 07:47:15 PM UTC
Category: freeciv-web
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: sveinung
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 3.0.0

___

Details:





___

File Attachments:


---
Date: Mon 30 Mar 2015 07:47:15 PM UTC  Name:
0001-JSON-protocol-support-sending-and-receiving-a-requir.patch  Size: 4kB  
By: sveinung



___

Reply to this item at:

  

___
  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 #5970] JSON protocol: support sending and receiving a requirement.

2015-03-30 Thread Sveinung Kvilhaugsvik
Follow-up Comment #1, patch #5970 (project freeciv):

Version 2: Receive the individual fields of the requirements as if they were
fields of a packet. This makes the code shorter. The cost is a worse error
message. Since no C JSON client currently exists the error message is a
theoretical problem.

(file #24166)
___

Additional Item Attachment:

File name: JSON-sending-and-receiving-a-req-careless.patch Size:3 KB


___

Reply to this item at:

  

___
  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 #5970] JSON protocol: support sending and receiving a requirement.

2015-03-30 Thread Sveinung Kvilhaugsvik
Follow-up Comment #2, patch #5970 (project freeciv):

Version 3: Improve the error message at receiving a requirement a bit.

(file #24167)
___

Additional Item Attachment:

File name: JSON-sending-and-receiving-a-req.patch Size:3 KB


___

Reply to this item at:

  

___
  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 #5970] JSON protocol: support sending and receiving a requirement.

2015-03-30 Thread Marko Lindqvist
Follow-up Comment #3, patch #5970 (project freeciv):

> Since no C JSON client currently exists

Building one is as easy as
#define FREECIV_JSON_CONNECTION 1

In the configure code in the repository that's still tied together with
--enable-web, but it can be used separately (I've not yet submitted configure
option for that as server <-> client communication is not completely working
yet)

I'm not saying this is a blocker for this patch, but we're close to having
freeciv server and client to be able to communicate using json (something you
should be vary from breaking more).

___

Reply to this item at:

  

___
  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 #5970] JSON protocol: support sending and receiving a requirement.

2015-03-30 Thread Sveinung Kvilhaugsvik
Follow-up Comment #4, patch #5970 (project freeciv):

_Building one is as easy as
#define FREECIV_JSON_CONNECTION 1_
Thank you for the tip, Marko. Looks like an enormous time saver.

___

Reply to this item at:

  

___
  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 #5970] JSON protocol: support sending and receiving a requirement.

2015-04-01 Thread Sveinung Kvilhaugsvik
Update of patch #5970 (project freeciv):

  Status:  Ready For Test => Done   
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

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


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