Difference between revisions of "DeveloperGuide"

From The Battle for Wesnoth Wiki
(fix syntax)
(allacrost link didn't belong here)
Line 13: Line 13:
 
* register to commit mailing list: https://mail.gna.org/listinfo/wesnoth-commits
 
* register to commit mailing list: https://mail.gna.org/listinfo/wesnoth-commits
 
** or get the list moderator to approve commit messages from you otherwise
 
** or get the list moderator to approve commit messages from you otherwise
http://www.allacrost.org
 
 
* don't forget http://www.wesnoth.org/wiki/EasyCoding if you're ever out of ideas :)
 
* don't forget http://www.wesnoth.org/wiki/EasyCoding if you're ever out of ideas :)
  

Revision as of 02:49, 4 July 2007

  • setting up developer SVN access: https://gna.org/svn/?group=wesnoth
  • Keep the commit easy to revert if need be
  • no huge commit (hard to review), so when possible maybe split it in working part with info about where you are going
  • be on #wesnoth-dev IRC channel and coordinate with other developers
  • code should compile and work after commit
  • Don't mess with the art or music without approval from one of the art and music people.
  • clear commit message (can be several lines)
  • mention "bug #1234" in the commit message for automatic cc to that gna bug number
  • don't forget changelog
  • change status of fixed bugs to "Fixed" when committed
  • change status of fixed bugs to "Closed" when released (as stable or devel version)
    • if the bug is svn only (something that was existing as bug in svn but in no release) you can directly close the bug after fixing it
  • register to commit mailing list: https://mail.gna.org/listinfo/wesnoth-commits
    • or get the list moderator to approve commit messages from you otherwise
  • don't forget http://www.wesnoth.org/wiki/EasyCoding if you're ever out of ideas :)
  • threading model: SDL threads via src/thread.hpp
    • synchronization model
      • avoid multi-threading code as much as possible
      • if you don't see threading code you probably don't need to learn about it

See also