forked from netwide-assembler/nasm
-
Notifications
You must be signed in to change notification settings - Fork 0
/
SubmittingPatches
116 lines (80 loc) · 3.66 KB
/
SubmittingPatches
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
How to submit patches into the NASM
===================================
Actually the rules are pretty simple
Obtaining the source code
-------------------------
The NASM sources are tracked by Git SCM at http://repo.or.cz/w/nasm.git
repository. You either could download packed sources or use git tool itself
git clone git://repo.or.cz/nasm.git
Changin the source code
-----------------------
When you change the NASM source code keep in mind -- we prefer tabs and
indentations to be 4 characters width, space filled.
Other "rules" could be learned from NASM sources -- just make your code
to look similar.
Producing patch
---------------
There are at least two ways to make it right.
1) git format-patch
You might need to read documentation on Git SCM how to prepare patch
for mail submission. Take a look on http://book.git-scm.com/ and/or
http://git-scm.com/documentation for details. It should not be hard
at all.
2) Use "diff -up"
Use "diff -up" or "diff -uprN" to create patches.
Signing your work
-----------------
To improve tracking of who did what we've introduced a "sign-off" procedure
on patches that are being emailed around.
The sign-off is a simple line at the end of the explanation for the
patch, which certifies that you wrote it or otherwise have the right to
pass it on as a open-source patch. The rules are pretty simple: if you
can certify the below:
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
then you just add a line saying
Signed-off-by: Random J Developer <random@developer.example.org>
using your real name (please, no pseudonyms or anonymous contributions if
it possible)
An example of patch message
---------------------------
From: Random J Developer <random@developer.example.org>
Subject: [PATCH] Short patch description
Long patch description (could be skipped if patch
is trivial enough)
Signed-off-by: Random J Developer <random@developer.example.org>
---
Patch body here
Mailing patches
---------------
The patches should be sent to NASM development mailing list
nasm-devel@lists.sourceforge.net
Please make sure the email client you're using doesn't screw
your patch (line wrapping and so on).
Wait for response
-----------------
Be patient. Most NASM developers are pretty busy people so if
there is no immediate response on your patch -- don't
be surprised, sometimes a patch may fly around a week(s) before
gets reviewed. But definitely the patches will not go to /dev/null.
---
With best regards,
NASM-team