<div dir="ltr"><div><div>OK I'm convinced. Let's remove it altogether. Anyone who needs a serious answer will be able to find one.<br><br></div>Thank you!<br></div>Brian<br><br><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 4, 2018 at 12:55 PM, David Davis <span dir="ltr"><<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">This is @dalley’s response on the PR which I agree with:<div><br></div><div><div>I would rather get rid of the AUTHORS entirely and put the documentation for `git shortlog -sne` elsewhere (or nowhere).</div><div><br></div><div>People who want that info should just look at the contributors page for the repository on github and, if they need the list in text form, `git shortlog -sn` is very very high in the search rankings for that query.</div><span class="HOEnZb"><font color="#888888"><div><br></div></font></span></div></div><div class="gmail_extra"><span class="HOEnZb"><font color="#888888"><br clear="all"><div><div class="m_-8927211647592444691gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div></font></span><div><div class="h5">
<br><div class="gmail_quote">On Wed, Apr 4, 2018 at 12:30 PM, Brian Bouterse <span dir="ltr"><<a href="mailto:bbouters@redhat.com" target="_blank">bbouters@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>You did update the dev guide already. Thank you for pointing that out. I was mistaken.<br><br></div>My main thinking is about the copyright which indicates the code is owned by the pulp contributors. Having a simple way to provide a definitive answer to that question is useful enough to document. An alternative to keeping it in the AUTHORS file contents would be to add it somewhere in the docs.<br></div><div><div><div><br></div><div>What do others want to do?<br><br></div><div>Thanks all,<br></div><div>Brian<br></div><div><div class="m_-8927211647592444691h5"><div><br><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 4, 2018 at 12:09 PM, Irina Gulina <span dir="ltr"><<a href="mailto:igulina@redhat.com" target="_blank">igulina@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
Brian,<br>
<br>
* the dev-guide doc was already updated in that PR, or do I miss smth?<br>
* Does anybody else want the AUTHORS file be updated with smth like "To see the list of contributors run `git shortlog -sne` command?<br>
Personally, I think it's not needed.<br>
* ok, I will do it for pulp 2 and 3, as soon as we agree on the previous point.<br>
<br>
Thanks all,<br>
Irina<br>
<span class="m_-8927211647592444691m_570631127858774400m_5612579863209426238im m_-8927211647592444691m_570631127858774400m_5612579863209426238HOEnZb"><br>
<br>
----- Original Message -----<br>
From: "Brian Bouterse" <<a href="mailto:bbouters@redhat.com" target="_blank">bbouters@redhat.com</a>><br>
To: "Daniel Alley" <<a href="mailto:dalley@redhat.com" target="_blank">dalley@redhat.com</a>><br>
Cc: "Pulp-dev" <<a href="mailto:pulp-dev@redhat.com" target="_blank">pulp-dev@redhat.com</a>><br>
Sent: Wednesday, 4 April, 2018 5:26:30 PM<br>
Subject: Re: [Pulp-dev] remove AUTHORS<br>
<br>
+1 to removing AUTHORS. I left some requests on the issue:<br>
<br>
* update some of the necessary docs<br>
* replace the AUTHORS content docs for the git shortlog -sne command<br>
* to do it for both pulp2 and pulp3.<br>
<br>
<a href="https://github.com/pulp/pulp/pull/3393#issuecomment-378631901" rel="noreferrer" target="_blank">https://github.com/pulp/pulp/p<wbr>ull/3393#issuecomment-37863190<wbr>1</a><br>
<br>
On Wed, Apr 4, 2018 at 10:09 AM, Daniel Alley < <a href="mailto:dalley@redhat.com" target="_blank">dalley@redhat.com</a> > wrote:<br>
<br>
<br>
<br>
+0<br>
<br>
On Wed, Apr 4, 2018 at 9:14 AM, Dana Walker < <a href="mailto:dawalker@redhat.com" target="_blank">dawalker@redhat.com</a> > wrote:<br>
<br>
<br>
<br>
</span><span class="m_-8927211647592444691m_570631127858774400m_5612579863209426238im m_-8927211647592444691m_570631127858774400m_5612579863209426238HOEnZb">I like Austin's point about getting to specify contact info for a specific project as I have different emails for different types of contributions myself, but I don't want to have to update things in multiple places down the road when I can just keep track of my listed contact details in my github settings. The same goes for someone who wants privacy and doesn't want their email listed or someone who has changed their name. I created the AUTHORS file for pulp_file recently and had misgivings about whether each of these contributors would have chosen to have the info I put together as their representative contact info, especially since we squashed down to one name and email which involves making some assumptions.<br>
<br>
I also like Irina's points about redundancy with GitHub's tracking of contributors (though that's a very specific thing that excludes merge commits, and for instance, I have made contributions to pulp through PRs and my name is not listed there), and more importantly, inaccurate and out of date info when hardcoded.<br>
<br>
I'm +1 to this PR to remove AUTHORS files moving forward.<br>
<br>
--Dana<br>
<br>
<br>
<br>
<br>
Dana Walker<br>
<br>
Associate Software Engineer<br>
<br>
Red Hat<br>
<br>
<br>
</span><div class="m_-8927211647592444691m_570631127858774400m_5612579863209426238HOEnZb"><div class="m_-8927211647592444691m_570631127858774400m_5612579863209426238h5">On Wed, Apr 4, 2018 at 8:36 AM, Austin Macdonald < <a href="mailto:amacdona@redhat.com" target="_blank">amacdona@redhat.com</a> > wrote:<br>
<br>
<br>
<br>
One reason to keep the AUTHORS file is that it allows contributors to specify their contact info instead of just linking to their GitHub page, were they might not have an email listed.<br>
<br>
I assume this discussion is restricted to pulp/pulp. The plugins are free to handle however they prefer.<br>
<br>
Personally, I am fine either way. Remove +0, one less file.<br>
<br>
<br>
On Wed, Apr 4, 2018, 05:22 Irina Gulina < <a href="mailto:igulina@redhat.com" target="_blank">igulina@redhat.com</a> > wrote:<br>
<br>
<br>
Hello Pulp Team,<br>
<br>
Recently, I have opened a PR to remove the AUTHORS file [1], and was suggested to bring it up here.<br>
Please let us/me know what you think about it. Any feedback will be appreciated.<br>
<br>
Happy Wednesday,<br>
Irina<br>
<br>
<br>
[1] <a href="https://github.com/pulp/pulp/pull/3393" rel="noreferrer" target="_blank">https://github.com/pulp/pulp/p<wbr>ull/3393</a><br>
<br>
______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
<br>
<br>
______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
</div></div></blockquote></div><br></div></div></div></div></div></div></div></div>
<br>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
<br></blockquote></div><br></div></div></div>
</blockquote></div><br></div></div></div></div></div>