Difference between revisions of "Developer Central"
(→Getting started: remove sf.net reference) |
(Add section on PR approval (copy past from an email from sev with some small edits)) |
||
Line 17: | Line 17: | ||
* Our [http://doxygen.scummvm.org/ Doxygen source code documentation] may help you to get the big picture about ScummVM. | * Our [http://doxygen.scummvm.org/ Doxygen source code documentation] may help you to get the big picture about ScummVM. | ||
* Also, you might want to check our list for current [[Platform Limitations]]. | * Also, you might want to check our list for current [[Platform Limitations]]. | ||
== Pull Request approval process == | |||
The current procedure is as follow: | |||
# For a sizeable, significant changes it is advised to make a [https://help.github.com/articles/about-pull-requests/ Pull Request on GitHub]. | |||
# Such Pull Request will have to stay for at least 2 weeks open for the comments. | |||
# Everyone is invited to comment and review and voice their opinion (views of non-team members are valuable, but have no decisive power). | |||
# If there are no unaddressed objections after 2 weeks, the PR could be merged. Exception could be made if there are suggestions over refactoring or tidying up the code, granted that they will be addressed in-tree. | |||
# Immediately after the merge the PR maker ensures that the [http://buildbot.scummvm.org buildbot] stays happy and is not worsened (historically we had few ports broken for months). | |||
The definition of what makes a sizeable changes not cast in stone. This is up to the discretion of the PR creator, but normally we would expect that anything which breaks existing OSystem API and the Common code, or significantly extends these, especially if it requires more work from the Porters, should go via a Pull Request process. The goal is to ensure that everything stays maintainable. | |||
Any developer is free to open a PR for less significant changes, and that process could be used for facilitating the discussion and collecting feedback, but when there are no big changes involved, no 2 weeks timeout is enforced. | |||
Individual engine sub-teams decide by themselves on the process. For example currently the SCI development is performed via PRs as well for any sizeable changes. It is up to that sub-team to drop this rule any time or make something even stricter. Just make sure you continue enjoying hacking the ScummVM code. | |||
== For Engine authors == | == For Engine authors == |
Revision as of 17:51, 24 October 2016
The purpose of this page is to give a link to all kinds of resources that contain information valuable to current and future developers of ScummVM.
Getting started
- The first thing you need to be able to do is get the ScummVM code from our Git repository.
- Then you should compile ScummVM.
That was easy, right? Here are some more tips:
- Some hints on debugging ScummVM.
- Before you write/submit code, you must read our Code Formatting Conventions.
Patches which do not follow them will be rejected or at least delayed until they are cleaned up to comply to them. - Also, you should read and respect the general Coding Conventions.
- If you have write (push) access to our repository, you are expected to have read and to comply with our Commit Guidelines.
Also, please read the tips for using Git here: Git tips.
In particular, note the use of feature branches, when working on refactoring or other large connected changes, rather than committing directly to scummvm/master. - If you do not have write (push) access to our repository, you should submit your contributions by one of the following methods:
- Using a Github Pull Request.
This requires you to have/register a github account, fork our repository, commit your changes to a branch and then issue a Pull Request.
This is the current preferred method as it is easier for the team to review, discuss and amend prior to merging.
You are expected to have read and to comply with our Commit Guidelines. - Using the patch tracker.
This requires you to have/register a github account, and generate a patch file, but this may be easier for developers unfamilar with Git or for small single commit changes, where a Pull Request might be considered overkill. - If neither of the above methods are suitable, individual developers may be willing to accept patches or amended source files by other methods i.e. e-mail, but please ask in the project IRC channel or by e-mail before doing this.
- Using a Github Pull Request.
- Our Doxygen source code documentation may help you to get the big picture about ScummVM.
- Also, you might want to check our list for current Platform Limitations.
Pull Request approval process
The current procedure is as follow:
- For a sizeable, significant changes it is advised to make a Pull Request on GitHub.
- Such Pull Request will have to stay for at least 2 weeks open for the comments.
- Everyone is invited to comment and review and voice their opinion (views of non-team members are valuable, but have no decisive power).
- If there are no unaddressed objections after 2 weeks, the PR could be merged. Exception could be made if there are suggestions over refactoring or tidying up the code, granted that they will be addressed in-tree.
- Immediately after the merge the PR maker ensures that the buildbot stays happy and is not worsened (historically we had few ports broken for months).
The definition of what makes a sizeable changes not cast in stone. This is up to the discretion of the PR creator, but normally we would expect that anything which breaks existing OSystem API and the Common code, or significantly extends these, especially if it requires more work from the Porters, should go via a Pull Request process. The goal is to ensure that everything stays maintainable.
Any developer is free to open a PR for less significant changes, and that process could be used for facilitating the discussion and collecting feedback, but when there are no big changes involved, no 2 weeks timeout is enforced.
Individual engine sub-teams decide by themselves on the process. For example currently the SCI development is performed via PRs as well for any sizeable changes. It is up to that sub-team to drop this rule any time or make something even stricter. Just make sure you continue enjoying hacking the ScummVM code.
For Engine authors
- If you want to provide a new Engine for ScummVM, check out our Engines HOWTO.
- If you work on a game engine for ScummVM, consider implementing Advanced Engine Features.
- If your engine supports multiple games / game variants, properly detecting and differentiating them all can be a nuisance. Use the Advanced Detector to overcome this hurdle.
- Accessing files in a portable fashion is non-trivial. Read all about how to open files.
- Debugging endian issues can be tricky. This page gives some solutions.
- If you want to have your engine merged into the ScummVM mainline, check out our Engine Inclusion HOWTO.
- For those starting out with reverse engineering, check out our Reverse Engineering HOWTO.
- Apart from git bisection to locate regressions in gameplay, the (experimental) Event Recorder provides a framework for recording and replaying gameplay to detect regressions.
- For some general suggestions on developing engines, see the Engine Development Tips & Tricks
For Backend authors
- If you want to port ScummVM to a new platform, check out our Backends HOWTO.
- The Dynamic Modules HOWTO describes how to implement loadable modules on a smaller backend, even without operating system support.
For Web site maintainers
If you want to update the main web site:
- Update the files you want in the scummvm-web project
- Logon to sf.net shell. For more information on using the Interactive Shell Service of sf.net, check this tutorial.
- Change to the directory /home/project-web/scummvm/htdocs/
- Run ./update.sh.
- This is also covered in more detail by the relevant section of the Admin HOWTO here.
Projects, plans, things to do
We are usually happy to receive any kind of help with the things listed below.
- List of open tasks, with relatively detailed descriptions, and contact points. Some rather small ones, some bigger ones, take a look. GSoC Ideas also has some ideas, intended for students applying for Google's Summer of Code.
- Main TODO list (contains links to many further specialized TODO pages, e.g. for specific engines).
- Our bug tracker lists bugs that need to be resolved
- Our feature request tracker also lists features requested by all kinds of people.
The following are partially outdated, but kept for now for the sake of reference.
- Small Devices Backend -- this was under development as part of the GSoC, but additional help is welcome.
- Keymapping Improvements -- still in the design stage. Help is welcome.
- Music drivers redesign -- in phase of discussion
New GUI
Our "new" GUI is themeable and translatable. If you are interested in writing a custom theme, the following may be of help for you:
- GUI Themes
- GUI Theme Specs
- GUI Translation Support in code
- Translation HOWTO for translation authors
- GUI TODO
Networking
Release Management
- Our release guideline for ScummVM Release HOWTO
- Our release testing status for the latest stable release Release Testing
- Anyone who wishes to bundle ScummVM with games for legal commercial and non-commercial release should read the notes here.
Code statistics
Our commit logs are fed to several freely available statistics tools:
- ohloh project analysis nice graphs and devs activity
- CIA.vc statistics, current development
- GitHub page, current development on github
Team Member Benefits
Every active ScummVM Team member is eligible for enhanced access to our Project Services and some further benefits:
- being added to our main credits (do it by yourself in scummvm/devtools/credits.pl)
- @scummvm.org forwarding address
- operator status in our IRC Channel #scummvm @ freenode
- address cloak on freenode (in form of scummvm/undead/nick)
- moderator status on ScummVM Forums and appropriate badge "ScummVM Developer" or "ScummVM Porter" or special one
- ScummVM Wiki account
- having development blog displayed on ScummVM Planet
- using project donations for purchasing necessary things for the development, such as game copies, software licenses, etc.
If you need any of the above, talk to sev.
If you do admin tasks e.g. responding to bugs, moderating the forums, you should read the Admin HOWTO here for tips and guidelines on doing this.
Misc
- Documentation on how we performed our CVS to Subversion conversion.
- Old page listing pros and cons between CVS and SVN before the conversion was made.
- A guide on submitting Screenshots for use on our website.