Skip to content

remove username from repo #20010

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

Closed
ghost opened this issue Jul 28, 2022 · 20 comments
Closed

remove username from repo #20010

ghost opened this issue Jul 28, 2022 · 20 comments
Labels
Infrastructure Things needed to maintain Perl development

Comments

@ghost
Copy link

ghost commented Jul 28, 2022

I want to have my name, e-mail address and username, removed from the repo.

affected files

.mailmap
AUTHORS
Porting/checkAUTHORS.pl

Would a PR removing the relevant lines suffice, or if any of the above files are auto generated, and would be recreated, are there special instructions to remove the relevant lines?

If special instructions are indicated please provide these.

@ghost ghost added the Needs Triage label Jul 28, 2022
@Leont
Copy link
Contributor

Leont commented Jul 28, 2022

Given two of those are generated from git history, it's currently not possibly. But I guess it could be made possible (by using a hash of some sort to filter you out).

@demerphq ideas?

@ghost
Copy link
Author

ghost commented Jul 28, 2022

I neglected to mention the file pod/perl5340delta.pod.

@ghost
Copy link
Author

ghost commented Jul 29, 2022

Presently there are no commits to the repo, consequently no git history, associated with this username.

https://github.com/Perl/perl5/commits?author=

Perhaps a Contributor would oblige and remove these lines, which would not result in a git history associated with this username.

@rwp0
Copy link
Contributor

rwp0 commented Jul 29, 2022

@apparluk Sorry, but why would you like to remove your name from the authors?

@Leont
Copy link
Contributor

Leont commented Jul 29, 2022

Perhaps a Contributor would oblige and remove these lines, which would not result in a git history associated with this username.

Like I said before, .mailmap and AUTHORS are generated files, so there's no point to manually removing them: they'd be added back on regeneration.

Porting/checkAUTHORS.pl is the source of the link between the two addresses, I'm not sure we can remove that without things breaking. I suspect we need to block your email first if we don't want to break tests.

The perl5340delta entry can be fixed easily enough though.

@ghost
Copy link
Author

ghost commented Jul 30, 2022

Update nmake for VS 2017
committed on Jun 14, 2017

I don't think there's any code I wrote in the repo since then.

@Leont
Copy link
Contributor

Leont commented Jul 30, 2022

b998bd1

@ghost
Copy link
Author

ghost commented Jul 30, 2022

If removal of the content it is too much trouble, I could just use a different GitHub account and e-mail address.

I presume in the foregoing that a merged commit - using a different GitHub account and e-mail address - would overwrite the current information in .mailmap, AUTHORS and Porting/checkAUTHORS.pl.

@Leont
Copy link
Contributor

Leont commented Jul 30, 2022

If removal of the content it is too much trouble, I could just use a different GitHub account and e-mail address.

I'm sure it can be done, but it requires code and someone will need to write that first.

I presume in the foregoing that a merged commit - using a different GitHub account and e-mail address - would overwrite the current information in .mailmap, AUTHORS and Porting/checkAUTHORS.pl.

I'm not sure what you mean with that. What would a merged commit have to do with it? And no information would be overwritten in .mailmap, it only gets information added (AUTHORS we can redirect using Porting/checkAUTHORS.pl).

@ghost
Copy link
Author

ghost commented Jul 31, 2022

not sure what you mean

my misunderstanding

Where a Committer has used more than one e-mail address multi-line Committers' entries appear in Porting/checkAUTHORS.pl, and, like you indicated before git history plays a role in generation of two repository files, so, using a different GitHub account and e-mail address wouldn't result in overwriting .mailmap and the old content would remain in Porting/checkAUTHORS.pl.

@demerphq
Copy link
Collaborator

demerphq commented Aug 1, 2022 via email

@ghost
Copy link
Author

ghost commented Aug 1, 2022

The GitHub account used at the time of the relevant commit was subsequently deleted. For a reason separate to this repository it was recreated, so {id} for the deleted and recreated GitHub accounts are not the same, whereas {user_name} for the former and current accounts are the same.

{id}+{user_name}@users.noreply.github.com

Both AUTHORS and .mailmap appear to be indexed by search engines.

I inform that as removing both the github email {id}+{user_name}@users.noreply.github.com entirely from the repositories' files and my name from perl5340delta can be arranged I would choose that option.

fundamental problem

Leaving my name and email in AUTHORS would be acceptable, in the context of the foregoing.

@demerphq
Copy link
Collaborator

demerphq commented Aug 5, 2022 via email

@demerphq
Copy link
Collaborator

demerphq commented Aug 8, 2022

Just wanted to post an update that I am working on this as time permits, my changes are nearly done, and a PR will be forthcoming this week.

@demerphq
Copy link
Collaborator

demerphq commented Aug 8, 2022

Hi all, I have pushed #20070 to remove this users details, and restructure and unify our AUTHORS related logic into one place (updateAUTHORS.pl) and add the ability to exclude people in a somewhat PII friendly way (hashed details). I expect to make some further patches to updateAUTHORS.pm once it gets merged or as part of the review process depending on my time availability.

demerphq added a commit that referenced this issue Aug 12, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 13, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 13, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 13, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 18, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 18, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 18, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 18, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 18, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 18, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 18, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 18, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 19, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 19, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 19, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 19, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 19, 2022
Contributor has requested they be excluded from our contributor
databases.
bram-perl pushed a commit to bram-perl/perl5 that referenced this issue Aug 19, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue Perl#20010.
bram-perl pushed a commit to bram-perl/perl5 that referenced this issue Aug 19, 2022
Contributor has requested they be excluded from our contributor
databases.
bram-perl pushed a commit to bram-perl/perl5 that referenced this issue Aug 19, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue Perl#20010.
bram-perl pushed a commit to bram-perl/perl5 that referenced this issue Aug 19, 2022
Contributor has requested they be excluded from our contributor
databases.
bram-perl pushed a commit to bram-perl/perl5 that referenced this issue Aug 19, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue Perl#20010.
bram-perl pushed a commit to bram-perl/perl5 that referenced this issue Aug 19, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 20, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 20, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 20, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 20, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 20, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 20, 2022
Contributor has requested they be excluded from our contributor
databases.
@karenetheridge
Copy link
Member

I would fulfill this user's request by adding a .mailmap entry for the user's name+email to "Anonymous" (or whatever other placeholder is deemed suitable). That will prevent the name from being published in perl releases (release announcements, other pod files etc).

Removing the name from the repository entirely is not possible because we do not change the history of git commits - once a commit is made it is permanent.

@jkeenan jkeenan added the Infrastructure Things needed to maintain Perl development label Aug 20, 2022
demerphq added a commit that referenced this issue Aug 21, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 21, 2022
Contributor has requested they be excluded from our contributor
databases.
demerphq added a commit that referenced this issue Aug 21, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue #20010.
demerphq added a commit that referenced this issue Aug 21, 2022
Contributor has requested they be excluded from our contributor
databases.
@demerphq
Copy link
Collaborator

This is resolved now. Thank you for your patience while we got it merged.

@demerphq
Copy link
Collaborator

demerphq commented Oct 11, 2022 via email

@demerphq
Copy link
Collaborator

demerphq commented Oct 11, 2022 via email

@demerphq
Copy link
Collaborator

demerphq commented Oct 11, 2022 via email

scottchiefbaker pushed a commit to scottchiefbaker/perl5 that referenced this issue Nov 3, 2022
This add a new file Porting/exclude_contrib.txt to hold a list
of base64 SHA-256 digests of the user name and email who should be
excluded.

This adds the options

    --exclude-me
    --exclude-contrib=NAME_AND_EMAIL
    --exclude-missing

to add exclusions in different ways. --exclude-me uses the git
credentials for the current user, --exclude-contrib expects a name
and email, and --exclude-missing excludes someone who is missing from
the change log.

When excluding someone their name will be removed from AUTHORS if it is
already listed, and the relevant entries to .mailmap will be removed,
and if necessary additional exclusion entries will be added for any of
their old identities which were mentioned in .mailmap.

This feature will be used in a later patch to resolve GH issue Perl#20010.
scottchiefbaker pushed a commit to scottchiefbaker/perl5 that referenced this issue Nov 3, 2022
Contributor has requested they be excluded from our contributor
databases.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infrastructure Things needed to maintain Perl development
Projects
None yet
Development

No branches or pull requests

5 participants