Commit message

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

Commit message

Jacques Le Roux
Administrator
Hi Commiters,

As suggested by Jacopo I (re)-read the commiters page. I w'd like to note something here, especially for new commiters (but not only
;o)

What Do I Need To Keep In Mind When Applying Patches From A Contributor?
You need to make sure that the commit message contains at least the name of the contributor and ideally a reference to the Bugzilla
or JIRA issue where the patch was submitted. The reasons: this preserves the legal trail and makes sure that contributors are
recognized. Obviously, the latter doesn't mean it's not a good idea to list the names of all contributors somewhere on the website.
To make it easier to "grep" for commits with patches from contributors, always use the same pattern in the commit message.
Traditionally, we use "Submitted by: <name>" or "Obtained from: <name>".
Here's an example of what such a commit message could look like:
Bugzilla #43835:
Added some cool new feature.
Submitted by: John Doe <john.doe.at.null.org>

taken from http://www.apache.org/dev/committers.html#applying-patches

Feek free to use your own pattern, it's easy with svn tools around.

Thanks

Jacques

Reply | Threaded
Open this post in threaded view
|

Re: Commit message

rajsaini
I have noted a practice in OpenEJB related to giving credit for the
patched to contributors. After applying the page they assign it to the
contributor which keeps a record in the Jira which is easier to search.

Thanks,

Raj

Jacques Le Roux wrote:

> Hi Commiters,
>
> As suggested by Jacopo I (re)-read the commiters page. I w'd like to note something here, especially for new commiters (but not only
> ;o)
>
> What Do I Need To Keep In Mind When Applying Patches From A Contributor?
> You need to make sure that the commit message contains at least the name of the contributor and ideally a reference to the Bugzilla
> or JIRA issue where the patch was submitted. The reasons: this preserves the legal trail and makes sure that contributors are
> recognized. Obviously, the latter doesn't mean it's not a good idea to list the names of all contributors somewhere on the website.
> To make it easier to "grep" for commits with patches from contributors, always use the same pattern in the commit message.
> Traditionally, we use "Submitted by: <name>" or "Obtained from: <name>".
> Here's an example of what such a commit message could look like:
> Bugzilla #43835:
> Added some cool new feature.
> Submitted by: John Doe <john.doe.at.null.org>
>
> taken from http://www.apache.org/dev/committers.html#applying-patches
>
> Feek free to use your own pattern, it's easy with svn tools around.
>
> Thanks
>
> Jacques
>
>
>  

Reply | Threaded
Open this post in threaded view
|

Re: Commit message

Jacopo Cappellato
the "Contributor Report" is also a useful tool:

https://issues.apache.org/jira/secure/ConfigureReport!default.jspa?selectedProjectId=12310500&reportKey=com.sourcelabs.jira.plugin.report.contributions:contributionreport

Jacopo


Raj Saini wrote:

> I have noted a practice in OpenEJB related to giving credit for the
> patched to contributors. After applying the page they assign it to the
> contributor which keeps a record in the Jira which is easier to search.
>
> Thanks,
>
> Raj
>
> Jacques Le Roux wrote:
>> Hi Commiters,
>>
>> As suggested by Jacopo I (re)-read the commiters page. I w'd like to
>> note something here, especially for new commiters (but not only
>> ;o)
>>
>> What Do I Need To Keep In Mind When Applying Patches From A Contributor?
>> You need to make sure that the commit message contains at least the
>> name of the contributor and ideally a reference to the Bugzilla
>> or JIRA issue where the patch was submitted. The reasons: this
>> preserves the legal trail and makes sure that contributors are
>> recognized. Obviously, the latter doesn't mean it's not a good idea to
>> list the names of all contributors somewhere on the website.
>> To make it easier to "grep" for commits with patches from
>> contributors, always use the same pattern in the commit message.
>> Traditionally, we use "Submitted by: <name>" or "Obtained from: <name>".
>> Here's an example of what such a commit message could look like:
>> Bugzilla #43835:
>> Added some cool new feature.
>> Submitted by: John Doe <john.doe.at.null.org>
>>
>> taken from http://www.apache.org/dev/committers.html#applying-patches
>>
>> Feek free to use your own pattern, it's easy with svn tools around.
>>
>> Thanks
>>
>> Jacques
>>
>>
>>  

Reply | Threaded
Open this post in threaded view
|

Re: Commit message

BJ Freeman
In reply to this post by rajsaini
only committers can be assigned, in the jira.
and when a committer commits, it is in the history.
for those of us that submit Jira, and not committers
it is up to the committer if they wish to reference's the source of the
commit.

Raj Saini sent the following on 12/13/2007 8:40 AM:

> I have noted a practice in OpenEJB related to giving credit for the
> patched to contributors. After applying the page they assign it to the
> contributor which keeps a record in the Jira which is easier to search.
>
> Thanks,
>
> Raj
>
> Jacques Le Roux wrote:
>> Hi Commiters,
>>
>> As suggested by Jacopo I (re)-read the commiters page. I w'd like to
>> note something here, especially for new commiters (but not only
>> ;o)
>>
>> What Do I Need To Keep In Mind When Applying Patches From A Contributor?
>> You need to make sure that the commit message contains at least the
>> name of the contributor and ideally a reference to the Bugzilla
>> or JIRA issue where the patch was submitted. The reasons: this
>> preserves the legal trail and makes sure that contributors are
>> recognized. Obviously, the latter doesn't mean it's not a good idea to
>> list the names of all contributors somewhere on the website.
>> To make it easier to "grep" for commits with patches from
>> contributors, always use the same pattern in the commit message.
>> Traditionally, we use "Submitted by: <name>" or "Obtained from: <name>".
>> Here's an example of what such a commit message could look like:
>> Bugzilla #43835:
>> Added some cool new feature.
>> Submitted by: John Doe <john.doe.at.null.org>
>>
>> taken from http://www.apache.org/dev/committers.html#applying-patches
>>
>> Feek free to use your own pattern, it's easy with svn tools around.
>>
>> Thanks
>>
>> Jacques
>>
>>
>>  
>
>
>
>

Reply | Threaded
Open this post in threaded view
|

Re: Commit message

Jacopo Cappellato
BJ Freeman wrote:
> only committers can be assigned, in the jira.
> and when a committer commits, it is in the history.
> for those of us that submit Jira, and not committers
> it is up to the committer if they wish to reference's the source of the
> commit.

No, this is not true: for a committer it is *mandatory* to reference the
author(s) of the patch or contribution.
So the author is in the svn log forever. And it is also recorded in Jira
forever (see the link I sent in my previous post in this thread).

Jacopo

>
> Raj Saini sent the following on 12/13/2007 8:40 AM:
>> I have noted a practice in OpenEJB related to giving credit for the
>> patched to contributors. After applying the page they assign it to the
>> contributor which keeps a record in the Jira which is easier to search.
>>
>> Thanks,
>>
>> Raj
>>
>> Jacques Le Roux wrote:
>>> Hi Commiters,
>>>
>>> As suggested by Jacopo I (re)-read the commiters page. I w'd like to
>>> note something here, especially for new commiters (but not only
>>> ;o)
>>>
>>> What Do I Need To Keep In Mind When Applying Patches From A Contributor?
>>> You need to make sure that the commit message contains at least the
>>> name of the contributor and ideally a reference to the Bugzilla
>>> or JIRA issue where the patch was submitted. The reasons: this
>>> preserves the legal trail and makes sure that contributors are
>>> recognized. Obviously, the latter doesn't mean it's not a good idea to
>>> list the names of all contributors somewhere on the website.
>>> To make it easier to "grep" for commits with patches from
>>> contributors, always use the same pattern in the commit message.
>>> Traditionally, we use "Submitted by: <name>" or "Obtained from: <name>".
>>> Here's an example of what such a commit message could look like:
>>> Bugzilla #43835:
>>> Added some cool new feature.
>>> Submitted by: John Doe <john.doe.at.null.org>
>>>
>>> taken from http://www.apache.org/dev/committers.html#applying-patches
>>>
>>> Feek free to use your own pattern, it's easy with svn tools around.
>>>
>>> Thanks
>>>
>>> Jacques
>>>
>>>
>>>  
>>
>>
>>

Reply | Threaded
Open this post in threaded view
|

Re: Commit message

BJ Freeman
Thanks for clarifying.
you email did not get to me before I sent mine.
:)

Jacopo Cappellato sent the following on 12/13/2007 8:57 AM:

> BJ Freeman wrote:
>> only committers can be assigned, in the jira.
>> and when a committer commits, it is in the history.
>> for those of us that submit Jira, and not committers
>> it is up to the committer if they wish to reference's the source of the
>> commit.
>
> No, this is not true: for a committer it is *mandatory* to reference the
> author(s) of the patch or contribution.
> So the author is in the svn log forever. And it is also recorded in Jira
> forever (see the link I sent in my previous post in this thread).
>
> Jacopo
>
>>
>> Raj Saini sent the following on 12/13/2007 8:40 AM:
>>> I have noted a practice in OpenEJB related to giving credit for the
>>> patched to contributors. After applying the page they assign it to the
>>> contributor which keeps a record in the Jira which is easier to search.
>>>
>>> Thanks,
>>>
>>> Raj
>>>
>>> Jacques Le Roux wrote:
>>>> Hi Commiters,
>>>>
>>>> As suggested by Jacopo I (re)-read the commiters page. I w'd like to
>>>> note something here, especially for new commiters (but not only
>>>> ;o)
>>>>
>>>> What Do I Need To Keep In Mind When Applying Patches From A
>>>> Contributor?
>>>> You need to make sure that the commit message contains at least the
>>>> name of the contributor and ideally a reference to the Bugzilla
>>>> or JIRA issue where the patch was submitted. The reasons: this
>>>> preserves the legal trail and makes sure that contributors are
>>>> recognized. Obviously, the latter doesn't mean it's not a good idea to
>>>> list the names of all contributors somewhere on the website.
>>>> To make it easier to "grep" for commits with patches from
>>>> contributors, always use the same pattern in the commit message.
>>>> Traditionally, we use "Submitted by: <name>" or "Obtained from:
>>>> <name>".
>>>> Here's an example of what such a commit message could look like:
>>>> Bugzilla #43835:
>>>> Added some cool new feature.
>>>> Submitted by: John Doe <john.doe.at.null.org>
>>>>
>>>> taken from http://www.apache.org/dev/committers.html#applying-patches
>>>>
>>>> Feek free to use your own pattern, it's easy with svn tools around.
>>>>
>>>> Thanks
>>>>
>>>> Jacques
>>>>
>>>>
>>>>  
>>>
>>>
>>>
>
>
>
>

Reply | Threaded
Open this post in threaded view
|

Re: Commit message

Jacques Le Roux
Administrator
In reply to this post by Jacopo Cappellato
Great, thanks Jacopo

Jacques

De : "Jacopo Cappellato" <[hidden email]>
> the "Contributor Report" is also a useful tool:
>
>
https://issues.apache.org/jira/secure/ConfigureReport!default.jspa?selectedProjectId=12310500&reportKey=com.sourcelabs.jira.plugin.report.contributions:contributionreport

>
> Jacopo
>
>
> Raj Saini wrote:
> > I have noted a practice in OpenEJB related to giving credit for the
> > patched to contributors. After applying the page they assign it to the
> > contributor which keeps a record in the Jira which is easier to search.
> >
> > Thanks,
> >
> > Raj
> >
> > Jacques Le Roux wrote:
> >> Hi Commiters,
> >>
> >> As suggested by Jacopo I (re)-read the commiters page. I w'd like to
> >> note something here, especially for new commiters (but not only
> >> ;o)
> >>
> >> What Do I Need To Keep In Mind When Applying Patches From A Contributor?
> >> You need to make sure that the commit message contains at least the
> >> name of the contributor and ideally a reference to the Bugzilla
> >> or JIRA issue where the patch was submitted. The reasons: this
> >> preserves the legal trail and makes sure that contributors are
> >> recognized. Obviously, the latter doesn't mean it's not a good idea to
> >> list the names of all contributors somewhere on the website.
> >> To make it easier to "grep" for commits with patches from
> >> contributors, always use the same pattern in the commit message.
> >> Traditionally, we use "Submitted by: <name>" or "Obtained from: <name>".
> >> Here's an example of what such a commit message could look like:
> >> Bugzilla #43835:
> >> Added some cool new feature.
> >> Submitted by: John Doe <john.doe.at.null.org>
> >>
> >> taken from http://www.apache.org/dev/committers.html#applying-patches
> >>
> >> Feek free to use your own pattern, it's easy with svn tools around.
> >>
> >> Thanks
> >>
> >> Jacques
> >>
> >>
> >>
>