File: ContributorInstructions.wiki

package info (click to toggle)
mongrel2 1.12.0-2
  • links: PTS, VCS
  • area: main
  • in suites: buster
  • size: 5,976 kB
  • sloc: ansic: 39,083; python: 2,833; sql: 1,555; sh: 467; makefile: 360; asm: 189; yacc: 145; php: 73; awk: 28; sed: 5
file content (148 lines) | stat: -rw-r--r-- 6,365 bytes parent folder | download | duplicates (5)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
<h1>Hacking On Mongrel2</h1>

This is organized more like a FAQ than a guide, but it gives all the
information you probably need to hack on the Mongrel2 source code. It talks
about getting the code, doing your first patch, coding guidelines, etc. These
aren't meant to be strict rules but more guidelines to follow.

<h2>I want to hack on Mongrel2. How do I join?</h2>

As our famous tagline says, "This isn't a hippie commune like Github." We love
patches and contributions from people, but Mongrel2 is a very specific kind of
C code project we have to keep the contributor list small and awesome. That
means we can't just let anyone in, we have to make you earn it. That doesn't
mean you're not awesome if you don't get in, it just means we make it a little
harder to join than your average github project does.
To get in on Mongrel2 you have to prove you can write code by writing some code
doing this:

  #  Follow the instructions in this document.
  #  Commit your changes to your repository when you think they're good.
  #  Then go find a_ticket_to_do and write code to fix it.
  #  Join the mongrel2@librelist.com mailing list or join #mongrel2 on
     irc.freenode.org.
  #  Finally, one of us will check out your changes and see if they're good to
     pull. If they are, and you want to get into the project, then just fix a
     few more bugs and we'll let you in.

Each of these steps is documented in this document, so just read it real quick
and get a good understanding of the project before you continue.

<h2>How do I get the source code?</h2>

The source code is currently maintained using git and the code itself is hosted
on github. The project is at https://github.com/zedshaw/mongrel2.

For information on how to use github see http://help.github.com/

The two branches available are:
 # master
 # develop

<h2>How do I find things to do?</h2>

We have a tickets system here and also at github that you can access. You
can get a_ticket_to_do and write code to fix it. Obviously, you'll have to
probably anonymous_login first if you want to do very much.

Why can't I just access everything without logging in?

We want to avoid spam and bots trolling our system and thrashing it or leaving
junk around, so we have a simple anonymous_login captcha you can use. It's a
minor inconvenience that helps us out a lot. It also weeds out people who
aren't smart enough or motivated enough to actually help.

<h2>How do I make a change?</h2>

Git works like most version control systems in that you make your changes
and "commit" them.
Since you are probably not a committer yet, you just have to do this:


We are also using git-flow to streamline our development efforts. Our model
of using git-flow is described in:

http://jeffkreeftmeijer.com/2010/why-arent-you-using-git-flow/

And also as shown in:

https://github.com/nvie/gitflow

Here are some quick instructions:

  # Install git-flow as in the README (https://github.com/nvie/gitflow).
  # In the mongrel2 run: git flow init
  # Just hit enter for all the defaults. Don't get fancy.
  # Start your hacking using: git flow feature start BlahBlahBlah
  # Commit like normal, don't push or tag anything.
  # End this feature with: git flow feature finish BlahBlahBlah
  # Finally: git push origin develop 

After you do this you just have to send a pull request via github and you 
are done.

<h2>If I become a contributor how do I get mentioned?</h2>

Everyone who submits a change using this method will have their username
mentioned in the timeline even if you're not registered as a core contributor.
Core contributors get mentioned on the home page.
After you become a contributor you then are thrown bags of money and caviar.

<h2>How do you prioritize what to work on?</h2>

We usually have a discussion on the mongrel2@librelist.com_mailing_list to
figure out what to do next. Then we fill in the tickets with the stuff to do.
Then we do that stuff.

<h2>Who comes up with the vision and direction?</h2>

Usually it's Zed, who wrote the majority of the content for this site.

<h2>What will get my contribution rejected?</h2>

Generally if your change adds too much code, is poorly written, doesn't work on
multiple platforms, or doesn't have testing when it needs it. Don't worry
though, we'll tell you how to clean it up and make it nice so that you learn
good style. As a starting point, here's what we consider our style guide:

<h2>What is your style guide?</h2>


  #  Keep your code clean and "flatter" with good use of white space for
     readability.
  #  Refactor common blocks of code or complex branches into functions,
     probably "static inline" is good.
  #  Aim for lines around 80 characters if possible.
  #  Check for errors from EVERY function using the check() macro from dbg.h.
  #  Check for memory allocation working with the check_mem() macro from dbg.h.
  #  Every if-elseif-else should have an else clause, and if that else
     shouldn't happen use the sentinel() macro to give an error.
  #  Same thing for case-switch, always have a default with sentinel() if that
     default should not happen.
  #  When in doubt, read and re-read the man page for function calls to make
     sure you got the error returns and parameters right.
  #  Don't use the C string functions, use bstring from bstring.h to do string
     operations.
  # Try to write a test for your code, which is hard sometimes in C, but
     attempt it at least.

In general the theme for Mongrel2 source is, "Don't code like an asshole." If
you write a piece of code and you didn't consider how another person will use
it, or just didn't care, then it'll probably get rejected.

<h2>How do I learn more about Mongrel2?</h2>

Check out GettingStarted or The_Manual for more.

<h2>How do I see what's been changed on a file or view a diff?</h2>

After you log in you can use the timeline_checkins_only to see the list of
changes. Then you pick one and there's various diffing options.
To see how a file changed, browse for the file in in_Files and find the one you
want to look at. You can then do advanced change logging and diffing of that
file.

The general way fossil works is that you get a lot of features from the command
line, but complex operations like analyzing diffs is better done in the Web
GUI. You can get to your own web GUI any time by doing fossil ui or fossil
serve.