|
Packit Service |
9402ce |
# Contributing to libva
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Libva is an open source project licensed under the [MIT License](https://opensource.org/licenses/MIT)
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
## Coding Style
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Libva does not have a defined coding style at this time, but that will be updated.
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
## Certificate of Origin
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
In order to get a clear contribution chain of trust we use the [signed-off-by language](https://01.org/community/signed-process)
|
|
Packit Service |
9402ce |
used by the Linux kernel project.
|
|
Packit Service |
9402ce |
## Patch format
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Beside the signed-off-by footer, we expect each patch to comply with the following format:
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
```
|
|
Packit Service |
9402ce |
<component>: Change summary
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
More detailed explanation of your changes: Why and how.
|
|
Packit Service |
9402ce |
Wrap it to 72 characters.
|
|
Packit Service |
9402ce |
See [here](http://chris.beams.io/posts/git-commit/)
|
|
Packit Service |
9402ce |
for some more good advices.
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Signed-off-by: <contributor@foo.com>
|
|
Packit Service |
9402ce |
```
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
For example:
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
```
|
|
Packit Service |
9402ce |
drm: remove va_drm_is_authenticated check
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
If we do not use a render node we must authenticate. Doing the extra
|
|
Packit Service |
9402ce |
GetClient calls/ioctls does not help much, so don't bother.
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Cc: David Herrmann <dh.herrmann@gmail.com>
|
|
Packit Service |
9402ce |
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
|
|
Packit Service |
9402ce |
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
|
|
Packit Service |
9402ce |
Reviewed-by: Sean V Kelley <seanvk@posteo.de>
|
|
Packit Service |
9402ce |
```
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
## Pull requests
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
We accept github pull requests.
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Once you've finished making your changes push them to your fork and send the PR via the github UI.
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
## Reporting a security issue
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Please mail to secure-opensource@intel.com directly for security issue
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
## Public issue tracking
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
If you have a problem, please let us know. IRC is a perfectly fine place
|
|
Packit Service |
9402ce |
to quickly informally bring something up, if you get a response. The
|
|
Packit Service |
9402ce |
[mailing list](https://lists.01.org/mailman/listinfo/intel-vaapi-media)
|
|
Packit Service |
9402ce |
is a more durable communication channel.
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
If it's a bug not already documented, by all means please [open an
|
|
Packit Service |
9402ce |
issue in github](https://github.com/intel/libva/issues/new) so we all get visibility
|
|
Packit Service |
9402ce |
to the problem and can work towards a resolution.
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
For feature requests we're also using github issues, with the label
|
|
Packit Service |
9402ce |
"enhancement".
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Our github bug/enhancement backlog and work queue are tracked in a
|
|
Packit Service |
9402ce |
[Libva waffle.io kanban](https://waffle.io/intel/libva).
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
## Closing issues
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
You can either close issues manually by adding the fixing commit SHA1 to the issue
|
|
Packit Service |
9402ce |
comments or by adding the `Fixes` keyword to your commit message:
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
```
|
|
Packit Service |
9402ce |
ssntp: test: Add Disconnection role checking tests
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
We check that we get the right role from the disconnection
|
|
Packit Service |
9402ce |
notifier.
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Fixes #121
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
|
|
Packit Service |
9402ce |
```
|
|
Packit Service |
9402ce |
|
|
Packit Service |
9402ce |
Github will then automatically close that issue when parsing the
|
|
Packit Service |
9402ce |
[commit message](https://help.github.com/articles/closing-issues-via-commit-messages/).
|