[LEDE-DEV] Proposal to sign all commits
Daniel Dickinson
lede at daniel.thecshore.com
Fri May 6 17:45:53 PDT 2016
On the signature agree
On 16-05-06 08:26 PM, David Lang wrote:
> On Fri, 6 May 2016, Daniel Dickinson wrote:
>
>
> This gets back to the question about what we are trying to achieve by
> signing the commits.
>
> If the purpose is to track who put what into the central tree, then
> having the merge masters sign the merge commits is useful, and not an
> imposition on people submitting patches. We also recognize that this is
> all the signature represents.
>
> If the purpose is to track back to the originator of the patch, that's a
> different, and much more difficult situation.
Agreed.
>
>> I think even with a large group of merge master, if pull requests are
>> indeed planned as a means of making community participation easier, then
>> requiring commits is counter-productive.
>
> I think you mean signed commits
Erm, yeah it'd be somewhat difficult to not require commits :-P
>
>> In addition, it's not like pull requests should go directly into master
>> anyway, but rather should be merged into a staging tree and only pulled
>> into master once the person(s) responsible for giving the okay to the
>> pull request have deemed it acceptable.
>>
>> This goes back to something I've said in other message, where I have
>> suggested that it would be useful to have one or more
>> experimental/testing branches where pull requests and patches could be
>> more widely tested than an individual can do (in fact I'd argue that
>> ideally *everyone* goes through some level of testing from someone other
>> than themselves.
>
> Ideally, but we need to recognize that we don't live in an ideal world :-)
True. Getting as close as is reasonably feasible is a good goal though.
(Note all the weasel words...)
Regards,
Daniel
More information about the Lede-dev
mailing list