glfw/.github/CONTRIBUTING.md

347 lines
13 KiB
Markdown
Raw Normal View History

2016-02-17 22:03:12 +01:00
# Contribution Guide
2016-06-21 16:35:36 +02:00
## Contents
- [Asking a question](#asking-a-question)
2016-06-21 16:35:36 +02:00
- [Reporting a bug](#reporting-a-bug)
- [Reporting a compile or link bug](#reporting-a-compile-or-link-bug)
- [Reporting a segfault or other crash bug](#reporting-a-segfault-or-other-crash-bug)
- [Reporting a context creation bug](#reporting-a-context-creation-bug)
- [Reporting a monitor or video mode bug](#reporting-a-monitor-or-video-mode-bug)
- [Reporting an input or event bug](#reporting-an-input-or-event-bug)
- [Reporting some other library bug](#reporting-some-other-library-bug)
- [Reporting a documentation bug](#reporting-a-documentation-bug)
- [Reporting a website bug](#reporting-a-website-bug)
- [Requesting a feature](#requesting-a-feature)
- [Contributing a bug fix](#contributing-a-bug-fix)
- [Contributing a feature](#contributing-a-feature)
2016-02-17 22:03:12 +01:00
## Asking a question
2016-08-11 19:04:21 +02:00
Questions about how to use GLFW should be asked either in the [support
section](http://discourse.glfw.org/c/support) of the forum, under the [Stack
Overflow tag](https://stackoverflow.com/questions/tagged/glfw) or [Game
Development tag](https://gamedev.stackexchange.com/questions/tagged/glfw) on
Stack Exchange or in the IRC channel `#glfw` on
[Freenode](http://freenode.net/).
Questions about the design or implementation of GLFW or about future plans
should be asked in the [dev section](http://discourse.glfw.org/c/dev) of the
2016-08-11 19:04:21 +02:00
forum or in the IRC channel. Please don't open a GitHub issue to discuss design
questions without first checking with a maintainer.
2016-02-17 22:03:12 +01:00
## Reporting a bug
2016-06-21 16:35:36 +02:00
If GLFW is behaving unexpectedly at run-time, start by setting an [error
callback](http://www.glfw.org/docs/latest/intro_guide.html#error_handling).
GLFW will often tell you the cause of an error via this callback. If it
doesn't, that might be a separate bug.
2016-02-17 22:03:12 +01:00
If GLFW is crashing or triggering asserts, make sure that all your object
handles and other pointers are valid.
2016-08-11 19:04:21 +02:00
For bugs where it makes sense, a [Short, Self Contained, Correct (Compilable),
Example](http://www.sscce.org/) is absolutely invaluable. Just put it inline in
the body text. Note that if the bug is reproducible with one of the test
programs that come with GLFW, just mention that instead.
__Don't worry about adding too much information__. Unimportant information can
be abbreviated or removed later, but missing information can stall bug fixing,
especially when your schedule doesn't align with that of the maintainer.
2016-06-21 16:35:36 +02:00
There are issue labels for both platforms and GPU manufacturers, so there is no
need to mention these in the subject line. If you do, it will be removed when
the issue is labeled.
2016-06-23 01:10:26 +02:00
If your bug is already reported, please add any new information you have, or if
it already has everything, give it a :+1:.
2016-06-21 16:35:36 +02:00
### Reporting a compile or link bug
__Note:__ GLFW needs many system APIs to do its job, which on some platforms
means linking to many system libraries. If you are using GLFW as a static
library, that means your application needs to link to these in addition to GLFW.
2016-06-21 20:40:09 +02:00
__Note:__ Check the [Compiling
GLFW](http://www.glfw.org/docs/latest/compile.html) guide and or [Building
applications](http://www.glfw.org/docs/latest/build.html) guide for before
opening an issue of this kind. Most issues are caused by a missing package or
linker flag.
2016-06-21 16:35:36 +02:00
2016-07-17 15:38:40 +02:00
Always include the __operating system name and version__ (e.g. `Windows
7 64-bit` or `Ubuntu 15.10`) and the __compiler name and version__ (e.g. `Visual
2016-06-21 16:35:36 +02:00
C++ 2015 Update 2`). If you are using an official release of GLFW,
2016-07-17 15:38:40 +02:00
include the __GLFW release version__ (e.g. `3.1.2`), otherwise include the
__GLFW commit ID__ (e.g. `3795d78b14ef06008889cc422a1fb8d642597751`) from Git.
2016-06-21 16:35:36 +02:00
Please also include the __complete build log__ from your compiler and linker,
even if it's long. It can always be shortened later, if necessary.
#### Quick template
```
OS and version:
Compiler version:
Release or commit:
Build log:
```
### Reporting a segfault or other crash bug
2016-07-17 15:38:40 +02:00
Always include the __operating system name and version__ (e.g. `Windows
2016-02-17 22:03:12 +01:00
7 64-bit` or `Ubuntu 15.10`). If you are using an official release of GLFW,
2016-07-17 15:38:40 +02:00
include the __GLFW release version__ (e.g. `3.1.2`), otherwise include the
__GLFW commit ID__ (e.g. `3795d78b14ef06008889cc422a1fb8d642597751`) from Git.
2016-02-17 22:03:12 +01:00
2016-06-21 16:35:36 +02:00
Please also include any __error messages__ provided to your application via the
[error
callback](http://www.glfw.org/docs/latest/intro_guide.html#error_handling) and
the __full call stack__ of the crash, or if the crash does not occur in debug
mode, mention that instead.
2016-02-17 22:03:12 +01:00
2016-06-21 16:35:36 +02:00
#### Quick template
2016-02-17 22:03:12 +01:00
2016-06-21 16:35:36 +02:00
```
OS and version:
Release or commit:
Error messages:
Call stack:
```
2016-02-17 22:03:12 +01:00
### Reporting a context creation bug
__Note:__ Windows ships with graphics drivers that do not support OpenGL. If
GLFW says that your machine lacks support for OpenGL, it very likely does.
Install drivers from the computer manufacturer or graphics card manufacturer
([Nvidia](http://www.geforce.com/drivers),
2016-06-21 16:35:36 +02:00
[AMD](http://support.amd.com/en-us/download),
[Intel](https://www-ssl.intel.com/content/www/us/en/support/detect.html)) to
2016-02-17 22:03:12 +01:00
fix this.
2016-06-21 16:35:36 +02:00
__Note:__ AMD only supports OpenGL ES on Windows via EGL. See the
[GLFW\_CONTEXT\_CREATION\_API](http://www.glfw.org/docs/latest/window_guide.html#window_hints_ctx)
hint for how to select EGL.
2016-02-17 22:03:12 +01:00
2016-06-21 16:35:36 +02:00
Please verify that context creation also fails with the `glfwinfo` tool before
reporting it as a bug. This tool is included in the GLFW source tree as
`tests/glfwinfo.c` and is built along with the library. It has switches for all
GLFW context and framebuffer hints. Run `glfwinfo -h` for a complete list.
2016-02-17 22:03:12 +01:00
2016-07-17 15:38:40 +02:00
Always include the __operating system name and version__ (e.g. `Windows
2016-06-21 16:35:36 +02:00
7 64-bit` or `Ubuntu 15.10`). If you are using an official release of GLFW,
2016-07-17 15:38:40 +02:00
include the __GLFW release version__ (e.g. `3.1.2`), otherwise include the
__GLFW commit ID__ (e.g. `3795d78b14ef06008889cc422a1fb8d642597751`) from Git.
2016-06-21 16:35:36 +02:00
Please also include the __GLFW version string__ (`3.2.0 X11 EGL clock_gettime
2016-12-10 23:41:19 +01:00
/dev/js`), as described
2016-06-21 16:35:36 +02:00
[here](http://www.glfw.org/docs/latest/intro.html#intro_version_string), the
2016-07-17 15:38:40 +02:00
__GPU model and driver version__ (e.g. `GeForce GTX660 with 352.79`), and the
2016-06-21 16:35:36 +02:00
__output of `glfwinfo`__ (with switches matching any hints you set in your
code) when reporting this kind of bug. If this tool doesn't run on the machine,
mention that instead.
#### Quick template
```
OS and version:
GPU and driver:
Release or commit:
Version string:
2016-06-21 16:46:17 +02:00
glfwinfo output:
2016-06-21 16:35:36 +02:00
```
2016-02-17 22:03:12 +01:00
### Reporting a monitor or video mode bug
__Note:__ On headless systems on some platforms, no monitors are reported. This
causes glfwGetPrimaryMonitor to return `NULL`, which not all applications are
prepared for.
2016-06-21 16:35:36 +02:00
__Note:__ Some third-party tools report more video modes than are approved of
by the OS. For safety and compatibility, GLFW only reports video modes the OS
2016-02-17 22:03:12 +01:00
wants programs to use. This is not a bug.
The `monitors` tool is included in the GLFW source tree as `tests/monitors.c`
2016-06-21 16:35:36 +02:00
and is built along with the library. It lists all information GLFW provides
about monitors it detects.
2016-02-17 22:03:12 +01:00
2016-07-17 15:38:40 +02:00
Always include the __operating system name and version__ (e.g. `Windows
2016-06-21 16:35:36 +02:00
7 64-bit` or `Ubuntu 15.10`). If you are using an official release of GLFW,
2016-07-17 15:38:40 +02:00
include the __GLFW release version__ (e.g. `3.1.2`), otherwise include the
__GLFW commit ID__ (e.g. `3795d78b14ef06008889cc422a1fb8d642597751`) from Git.
2016-06-21 16:35:36 +02:00
Please also include any __error messages__ provided to your application via the
[error
callback](http://www.glfw.org/docs/latest/intro_guide.html#error_handling) and
the __output of `monitors`__ when reporting this kind of bug. If this tool
doesn't run on the machine, mention this instead.
2016-02-17 22:03:12 +01:00
2016-06-21 16:35:36 +02:00
#### Quick template
2016-02-17 22:03:12 +01:00
2016-06-21 16:35:36 +02:00
```
OS and version:
Release or commit:
Error messages:
2016-06-21 16:46:17 +02:00
monitors output:
2016-06-21 16:35:36 +02:00
```
### Reporting an input or event bug
2016-06-21 20:40:09 +02:00
__Note:__ The exact ordering of related window events will sometimes differ.
__Note:__ Window moving and resizing (by the user) will block the main thread on some
platforms. This is not a bug. Set a [refresh
callback](http://www.glfw.org/docs/latest/window.html#window_refresh) if you
want to keep the window contents updated during a move or size operation.
2016-02-17 22:03:12 +01:00
The `events` tool is included in the GLFW source tree as `tests/events.c` and is
built along with the library. It prints all information provided to every
callback supported by GLFW as events occur. Each event is listed with the time
and a unique number to make discussions about event logs easier. The tool has
command-line options for creating multiple windows and full screen windows.
2016-07-17 15:38:40 +02:00
Always include the __operating system name and version__ (e.g. `Windows
2016-06-21 16:35:36 +02:00
7 64-bit` or `Ubuntu 15.10`). If you are using an official release of GLFW,
2016-07-17 15:38:40 +02:00
include the __GLFW release version__ (e.g. `3.1.2`), otherwise include the
__GLFW commit ID__ (e.g. `3795d78b14ef06008889cc422a1fb8d642597751`) from Git.
2016-06-21 16:35:36 +02:00
Please also include any __error messages__ provided to your application via the
[error
callback](http://www.glfw.org/docs/latest/intro_guide.html#error_handling) and
if relevant, the __output of `events`__ when reporting this kind of bug. If
this tool doesn't run on the machine, mention this instead.
#### Quick template
```
OS and version:
Release or commit:
Error messages:
2016-06-21 16:46:17 +02:00
events output:
2016-06-21 16:35:36 +02:00
```
### Reporting some other library bug
2016-07-17 15:38:40 +02:00
Always include the __operating system name and version__ (e.g. `Windows
2016-06-21 16:35:36 +02:00
7 64-bit` or `Ubuntu 15.10`). If you are using an official release of GLFW,
2016-07-17 15:38:40 +02:00
include the __GLFW release version__ (e.g. `3.1.2`), otherwise include the
__GLFW commit ID__ (e.g. `3795d78b14ef06008889cc422a1fb8d642597751`) from Git.
2016-06-21 16:35:36 +02:00
Please also include any __error messages__ provided to your application via the
[error
callback](http://www.glfw.org/docs/latest/intro_guide.html#error_handling), if
relevant.
#### Quick template
```
OS and version:
Release or commit:
Error messages:
```
2016-02-17 22:03:12 +01:00
### Reporting a documentation bug
2016-06-21 16:35:36 +02:00
If you found a bug in the documentation, including this file, then it's fine to
just link to that web page or mention that source file. You don't need to match
the source to the output or vice versa.
### Reporting a website bug
If the bug is in the documentation (anything under `/docs/`) then please see the
section above. Bugs in the rest of the site are reported to to the [website
source repository](https://github.com/glfw/website/issues).
## Requesting a feature
Please explain why you need the feature and how you intend to use it. If you
have a specific API design in mind, please add that as well. If you have or are
planning to write code for the feature, see the section below.
2016-02-17 22:03:12 +01:00
2016-08-11 19:04:21 +02:00
If there already is a request for the feature you need, add your specific use
case unless it is already mentioned. If it is, give it a :+1:.
2016-02-17 22:03:12 +01:00
## Contributing a bug fix
2016-08-11 19:04:21 +02:00
__Note:__ You must have all necessary [intellectual
property rights](https://en.wikipedia.org/wiki/Intellectual_property) to any
code you contribute. If you did not write the code yourself, you must explain
where it came from and under what license you received it. Even code using the
same license as GLFW may not be copied without attribution.
2016-06-21 16:46:17 +02:00
2016-07-14 18:02:44 +02:00
__There is no preferred patch size__. A one character fix is just as welcome as
2016-06-21 20:40:09 +02:00
a thousand line one, if that is the appropriate size for the fix.
In addition to the code, a complete bug fix includes:
2016-06-21 16:35:36 +02:00
2016-06-21 20:40:09 +02:00
- Change log entry in `README.md`, describing the incorrect behavior
- Credits entries for all authors of the bug fix
2016-07-14 18:02:44 +02:00
Bug fixes will not be rejected because they don't include all the above parts,
but please keep in mind that maintainer time is finite and that there are many
other bugs and features to work on.
2016-06-21 20:40:09 +02:00
If the patch fixes a bug introduced after the last release, it should not get
a change log entry.
2016-02-17 22:03:12 +01:00
## Contributing a feature
2016-07-14 18:02:44 +02:00
__Note:__ You must have all necessary rights to any code you contribute. If you
did not write the code yourself, you must explain where it came from and under
what license. Even code using the same license as GLFW may not be copied
without attribution.
__There is no preferred patch size__. A one character change is just as welcome
as one adding a thousand line one, if that is the appropriate size for the
feature.
2016-06-21 16:35:36 +02:00
2016-06-21 20:40:09 +02:00
In addition to the code, a complete feature includes:
2016-06-21 16:35:36 +02:00
2016-06-21 20:40:09 +02:00
- Change log entry in `README.md`, listing all new symbols
- News page entry, briefly describing the feature
- Guide documentation, with minimal examples, in the relevant guide
- Reference documentation, with all applicable tags
- Cross-references and mentions in appropriate places
- Credits entries for all authors of the feature
2016-06-21 16:35:36 +02:00
2016-07-14 18:02:44 +02:00
If the feature requires platform-specific code, at minimum stubs must be added
for the new platform function to all supported and experimental platforms.
If it adds a new callback, support for it must be added to `tests/event.c`.
2016-06-21 16:35:36 +02:00
2016-07-14 18:02:44 +02:00
If it adds a new monitor property, support for it must be added to
`tests/monitor.c`.
2016-06-21 16:35:36 +02:00
2016-07-14 18:02:44 +02:00
If it adds a new OpenGL, OpenGL ES or Vulkan option or extension, support
for it must be added to `tests/glfwinfo.c` and the behavior of the library when
the extension is missing documented in `docs/compat.dox`.
2016-06-21 16:35:36 +02:00
2016-07-14 18:02:44 +02:00
Features will not be rejected because they don't include all the above parts,
but please keep in mind that maintainer time is finite and that there are many
other features and bugs to work on.
2016-06-21 16:35:36 +02:00
2016-07-14 18:02:44 +02:00
Please also keep in mind that any part of the public API that has been included
in a release cannot be changed until the next _major_ version. Features can be
2016-06-21 16:35:36 +02:00
added and existing parts can sometimes be overloaded (in the general sense of
doing more things, not in the C++ sense), but code written to the API of one
minor release should both compile and run on subsequent minor releases.
2016-02-17 22:03:12 +01:00