CONTRIBUTING.md 6.71 KB
Newer Older
1
2
3
4
# How to become a contributor and submit your own code

## Contributor License Agreements

misterg's avatar
misterg committed
5
6
We'd love to accept your patches! Before we can take them, we have to jump a
couple of legal hurdles.
7
8
9
10

Please fill out either the individual or corporate Contributor License Agreement
(CLA).

misterg's avatar
misterg committed
11
*   If you are an individual writing original source code and you're sure you
12
13
    own the intellectual property, then you'll need to sign an
    [individual CLA](https://developers.google.com/open-source/cla/individual).
misterg's avatar
misterg committed
14
*   If you work for a company that wants to allow you to contribute your work,
15
16
17
18
19
20
21
    then you'll need to sign a
    [corporate CLA](https://developers.google.com/open-source/cla/corporate).

Follow either of the two links above to access the appropriate CLA and
instructions for how to sign and return it. Once we receive it, we'll be able to
accept your pull requests.

Gennadiy Civil's avatar
Gennadiy Civil committed
22
23
## Are you a Googler?

misterg's avatar
misterg committed
24
25
26
If you are a Googler, plese make an attempt to submit an internal change rather
than a GitHub Pull Request. If you are not able to submit an internal change a
PR is acceptable as an alternative.
Gennadiy Civil's avatar
Gennadiy Civil committed
27

28
29
## Contributing A Patch

misterg's avatar
misterg committed
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
1.  Submit an issue describing your proposed change to the
    [issue tracker](https://github.com/google/googletest).
1.  Please don't mix more than one logical change per submittal, because it
    makes the history hard to follow. If you want to make a change that doesn't
    have a corresponding issue in the issue tracker, please create one.
1.  Also, coordinate with team members that are listed on the issue in question.
    This ensures that work isn't being duplicated and communicating your plan
    early also generally leads to better patches.
1.  If your proposed change is accepted, and you haven't already done so, sign a
    Contributor License Agreement (see details above).
1.  Fork the desired repo, develop and test your code changes.
1.  Ensure that your code adheres to the existing style in the sample to which
    you are contributing.
1.  Ensure that your code has an appropriate set of unit tests which all pass.
1.  Submit a pull request.

## The Google Test and Google Mock Communities
47
48

The Google Test community exists primarily through the
misterg's avatar
misterg committed
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
[discussion group](http://groups.google.com/group/googletestframework) and the
GitHub repository. Likewise, the Google Mock community exists primarily through
their own [discussion group](http://groups.google.com/group/googlemock). You are
definitely encouraged to contribute to the discussion and you can also help us
to keep the effectiveness of the group high by following and promoting the
guidelines listed here.

### Please Be Friendly

Showing courtesy and respect to others is a vital part of the Google culture,
and we strongly encourage everyone participating in Google Test development to
join us in accepting nothing less. Of course, being courteous is not the same as
failing to constructively disagree with each other, but it does mean that we
should be respectful of each other when enumerating the 42 technical reasons
that a particular proposal may not be the best choice. There's never a reason to
be antagonistic or dismissive toward anyone who is sincerely trying to
65
66
contribute to a discussion.

misterg's avatar
misterg committed
67
68
69
Sure, C++ testing is serious business and all that, but it's also a lot of fun.
Let's keep it that way. Let's strive to be one of the friendliest communities in
all of open source.
70

misterg's avatar
misterg committed
71
72
As always, discuss Google Test in the official GoogleTest discussion group. You
don't have to actually submit code in order to sign up. Your participation
73
74
itself is a valuable contribution.

75
76
## Style

misterg's avatar
misterg committed
77
78
79
80
81
82
83
To keep the source consistent, readable, diffable and easy to merge, we use a
fairly rigid coding style, as defined by the
[google-styleguide](https://github.com/google/styleguide) project. All patches
will be expected to conform to the style outlined
[here](https://google.github.io/styleguide/cppguide.html). Use
[.clang-format](https://github.com/google/googletest/blob/master/.clang-format)
to check your formatting
84

misterg's avatar
misterg committed
85
## Requirements for Contributors
86

misterg's avatar
misterg committed
87
88
If you plan to contribute a patch, you need to build Google Test, Google Mock,
and their own tests from a git checkout, which has further requirements:
89

misterg's avatar
misterg committed
90
91
92
93
94
*   [Python](https://www.python.org/) v2.3 or newer (for running some of the
    tests and re-generating certain source files from templates)
*   [CMake](https://cmake.org/) v2.6.4 or newer
*   [GNU Build System](https://en.wikipedia.org/wiki/GNU_Build_System) including
    automake (>= 1.9), autoconf (>= 2.59), and libtool / libtoolize.
95

misterg's avatar
misterg committed
96
## Developing Google Test
97
98
99

This section discusses how to make your own changes to Google Test.

misterg's avatar
misterg committed
100
### Testing Google Test Itself
101
102

To make sure your changes work as intended and don't break existing
misterg's avatar
misterg committed
103
104
functionality, you'll want to compile and run Google Test's own tests. For that
you can use CMake:
105
106
107
108
109

    mkdir mybuild
    cd mybuild
    cmake -Dgtest_build_tests=ON ${GTEST_DIR}

misterg's avatar
misterg committed
110
111
112
113
Make sure you have Python installed, as some of Google Test's tests are written
in Python. If the cmake command complains about not being able to find Python
(`Could NOT find PythonInterp (missing: PYTHON_EXECUTABLE)`), try telling it
explicitly where your Python executable can be found:
114
115
116

    cmake -DPYTHON_EXECUTABLE=path/to/python -Dgtest_build_tests=ON ${GTEST_DIR}

misterg's avatar
misterg committed
117
118
Next, you can build Google Test and all of its own tests. On \*nix, this is
usually done by 'make'. To run the tests, do
119
120
121
122
123

    make test

All tests should pass.

misterg's avatar
misterg committed
124
### Regenerating Source Files
125

misterg's avatar
misterg committed
126
127
128
Some of Google Test's source files are generated from templates (not in the C++
sense) using a script. For example, the file
include/gtest/internal/gtest-type-util.h.pump is used to generate
129
130
gtest-type-util.h in the same directory.

misterg's avatar
misterg committed
131
132
133
You don't need to worry about regenerating the source files unless you need to
modify them. You would then modify the corresponding `.pump` files and run the
'[pump.py](googletest/scripts/pump.py)' generator script. See the
134
[Pump Manual](googletest/docs/PumpManual.md).
135

misterg's avatar
misterg committed
136
## Developing Google Mock
137
138
139

This section discusses how to make your own changes to Google Mock.

misterg's avatar
misterg committed
140
#### Testing Google Mock Itself
141
142

To make sure your changes work as intended and don't break existing
misterg's avatar
misterg committed
143
144
145
146
functionality, you'll want to compile and run Google Test's own tests. For that
you'll need Autotools. First, make sure you have followed the instructions above
to configure Google Mock. Then, create a build output directory and enter it.
Next,
147
148
149

    ${GMOCK_DIR}/configure  # try --help for more info

misterg's avatar
misterg committed
150
151
Once you have successfully configured Google Mock, the build steps are standard
for GNU-style OSS packages.
152
153
154
155
156

    make        # Standard makefile following GNU conventions
    make check  # Builds and runs all tests - all should pass.

Note that when building your project against Google Mock, you are building
misterg's avatar
misterg committed
157
against Google Test as well. There is no need to configure Google Test
158
separately.