<div dir="ltr">Hi Alessandro,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 10, 2014 at 6:57 AM, Alessandro Pasotti <span dir="ltr"><<a href="mailto:apasotti@gmail.com" target="_blank">apasotti@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">2014-10-10 14:44 GMT+02:00 Jürgen E. <<a href="mailto:jef@norbit.de">jef@norbit.de</a>>:<br>
<span class="">> Hi Matthias,<br>
><br>
> On Fri, 10. Oct 2014 at 11:53:28 +0200, Matthias Kuhn wrote:<br>
>> To make an example, this is a good commit message.<br>
>><br>
>> <a href="https://github.com/qgis/QGIS/commit/a132bc9a9e318c2321c8fba13dc9503b4d11e2aa" target="_blank">https://github.com/qgis/QGIS/commit/a132bc9a9e318c2321c8fba13dc9503b4d11e2aa</a><br>
>><br>
>> It states<br>
>>  * which problem it solves<br>
>>  * and what it does exactly<br>
>><br>
>> And I could not care less about an additional line at the bottom stating<br>
>> "Funded by John Wayne"<br>
><br>
> Almost.  It should have "Fixes #8166" in it to automatically link to the issue<br>
> and close it - Bugfix #8166 doesn't create that link.<br>
><br>
<br>
</span>Good hints. If it's not already in CODING, we could add these<br>
guidelines in that file.<br>
<br>
Also, since most of you don't seem to be annoyed by the "funded by"<br>
sentences, I  think we can agree on Victor's proposal: if the new<br>
feature/bug fix etc. was funded by an organization, a company or an<br>
individual, a single line (80 chars max) sentence can be added as the<br>
last line of the commit, after a blank line.<br>
<br>
That line should be added only in the last "merge" (final) commit and<br>
not on all and every individual commit.<span class="im"><br></span></blockquote><div><br></div><div>I disagree. "Funded by" attribution lines should go wherever the author feels is appropriate, regardless of whether the commit is a merge type, pulling in a whole branch, or the commits are simply rebased on top of master.<br></div><div><br></div><div>It is a matter of public record:<br><br></div><div>* QGIS is a public project<br></div><div>* The funding is for a public project.<br></div><div>* The funding is a matter of public record (unless otherwise deemed by sponsor).<br></div><div>* The commit log is a public record (and the longest standing one for the project).<br><br></div><div>I fail to see how any of that can be misconstrued as "spam," if the line in a commit is merely an attribution, i.e. not part of the first line of the commit.<br><br></div><div>Even if *every* single commit message in the QGIS repo had a "Funded by" attribution line, I could not see how that is anything but a matter of fact. Actually, I think that would be an awesome record of the project's prowess of finding sponsorship.<br><br></div><div>Regards,<br><br>Larry Shaffer<br>Dakota Cartography<br>Black Hills, South Dakota<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="im">
<br>
--<br>
Alessandro Pasotti<br>
w3:   <a href="http://www.itopen.it" target="_blank">www.itopen.it</a><br>
</span><div class=""><div class="h5">_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a></div></div></blockquote></div><br></div></div></div>