Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mailmap: Abhishek Lekshmanan affiliation #8615

Merged
1 commit merged into from Apr 15, 2016
Merged

Conversation

theanalyst
Copy link
Member

Adding an entry to normalize the personal email id and work email id (As reviews/and commits assoc'ed with gmail still got normalized to previous email)

@@ -9,6 +9,7 @@
Aaron Bassett <abassett@gmail.com>
Abhishek Lekshmanan <abhishek.lekshmanan@ril.com> <abhishek.lekshmanan@gmail.com>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this one should then be removed, right ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure, without removing it also, if do the credits.sh script from prev release I'm getting the correct output?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In any case only one of them will be taken into account. There is no way to resolve this so that past contributions stay with the previous mapping. There would need to be a time range for which a mailmap entry is valid, but that's not supported. In other words, unless you change your personal email, contributions from the past will be remapped incorrectly when using a recent .mailmap. There always is the option of using the former .mailmap to get the correct affiliation, which is probably the right thing to do.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ok, what is to be done here then, I'm ok with closing this. not that much of a deal, anyway most of the commits dont use this email id, only the reviewed-by since that is added by the dev. doing the merge. (for eg. in 10.1.2 contributor credits it appeared in reviewed-by mostly)

Copy link

@ghost ghost Apr 15, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would make sense to remove the

Abhishek Lekshmanan <abhishek.lekshmanan@ril.com> <abhishek.lekshmanan@gmail.com>

line.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

K will do that

Adding an entry to normalize the personal email id and work email id
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
Copy link
Member Author

@dachary updated as you suggested

@ghost ghost merged commit 07c945b into ceph:master Apr 15, 2016
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant