patchFreeciv - Patches: patch #3502, New /topic command

 
 
Show feedback again

patch #3502: New /topic command

Submitted by:  Davide Baldini <davide_at_debian>
Submitted on:  Wed 15 Aug 2012 12:19:59 PM UTC  
 
Category: generalPriority: 5 - Normal
Status: Wont DoPrivacy: Public
Assigned to: Marko Lindqvist <cazfi>Open/Closed: Closed
Planned Release: 

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 19 Apr 2013 09:36:52 AM UTC, comment #4:

Current metaserver development version has already dropped topic, and uses metamessage in main page too. Main metaserver at meta.freeciv.org just hasn't been updated yet.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Tue 28 Aug 2012 10:38:48 PM UTC, comment #3:

I'm going to change metaserver as soon as I get freeciv-web's hackish use of topic replaced by some cleaner solution.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Sun 19 Aug 2012 12:35:36 AM UTC, comment #2:

I think that having both of them is redundant and not completely useful since "topic" is not shown in client's window.

Anyway it would be useful to have the /topic command until the metaserver software is adjusted to display the "metamessage" in the main web page.

Schedule or drop as you believe.

Davide Baldini <davide_at_debian>
Sat 18 Aug 2012 11:50:16 PM UTC, comment #1:

Do you really need separate metamessage and topic? For a long time I have considered dropping topic from metaserver too, and display metamessage instead on web interface main page too.

Marko Lindqvist <cazfi>
Project AdministratorIn charge of this item.
Wed 15 Aug 2012 12:19:59 PM UTC, original submission:

This patch adds a "topic" server command which can set the relative metamessage string on the metaserver webpage at runtime.

Before this patch, we used to have a Cazfi's patch that allowed to set a topic message only by cmdline. Anyway, that patch got obsoleted and is now incompatible with current sources.

Patch has been tested on 2.3.2 .

Davide Baldini <davide_at_debian>

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by cazfi (Posted a comment)
  • -unavailable- added by davide_at_debian (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 19 Apr 2013 09:36:52 AM UTCcazfiStatusNone=>Wont Do
      Assigned toNone=>cazfi
      Open/ClosedOpen=>Closed
    Wed 15 Aug 2012 12:19:59 PM UTCdavide_at_debianAttached File-=>Added topic_cmd-2.3.2.patch, #16395
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup