Difference between revisions of "PatchSubmissionGuidelines"

From The Battle for Wesnoth Wiki
m (recategorizing)
m (Mention that adding oneself to about.cfg involves this strange thing called 'alphabetical order'; also, it's 'Miscellaneous Contributors' nowadays)
Line 12: Line 12:
 
* '''Add yourself in about.cfg'''
 
* '''Add yourself in about.cfg'''
 
Including your name and/or nick it makes it easier for me.
 
Including your name and/or nick it makes it easier for me.
New contributors should go in the contributor section of the about file
+
New contributors should add themselves to the ''Miscellaneous Contributors'' section of the about file (data/core/about.cfg). Please keep the alphabetical order while doing it.
  
 
* '''When changing WML, please add a pointer in the wiki to where it should be updated when commited'''
 
* '''When changing WML, please add a pointer in the wiki to where it should be updated when commited'''

Revision as of 21:04, 24 July 2009

Copied from http://www.wesnoth.org/forum/viewtopic.php?f=10&t=9979

  • All patches should be submitted at patches.wesnoth.org

You can post them here too for discussion, but we need to track what is their status and p.w.o helps a lot

  • Patches should be generated using "svn diff"

SVN provides a cool command to generate diff, I usually run "svn diff >mypatch.patch" to create the patch, and its all nice and ready. It even records precisely the commit against which it was generated, it makes things really easy for me...

  • Don't forget to add an entry to the Changelog
  • if you add afile, don't forget to change the Makefiles
  • Add yourself in about.cfg

Including your name and/or nick it makes it easier for me. New contributors should add themselves to the Miscellaneous Contributors section of the about file (data/core/about.cfg). Please keep the alphabetical order while doing it.

  • When changing WML, please add a pointer in the wiki to where it should be updated when commited
  • Be patient, sometime I'm not very responsive
  • Don't be suprised if we discuss the patch a lot

Thus, you should leave us a way to contact you, either a forum nick, an email adress, or submit with a registered gna account.

  • The patch should generate no warnings

Wesnoth has a large number of warnings enabled, all of them are useful. If your code spits warning, please have a look, and ask yourself why you are doing whatever causes the warning.

  • When adding a file, don't forget Makefile.am

It makes things simpler for me, and is often forgotten.

  • Sometimes patches are rejected, don't be suprised if it happens

See also