Skip to content

maintainers: drop gm6k#439407

Merged
wolfgangwalther merged 1 commit into
NixOS:masterfrom
SigmaSquadron:push-vpqsnkyzprus
Sep 2, 2025
Merged

maintainers: drop gm6k#439407
wolfgangwalther merged 1 commit into
NixOS:masterfrom
SigmaSquadron:push-vpqsnkyzprus

Conversation

@SigmaSquadron

@SigmaSquadron SigmaSquadron commented Sep 2, 2025

Copy link
Copy Markdown
Contributor

Although this maintainer has responded to requests for maintainance and is active in Nixpkgs, the new data collection requirements introduced in #437085 have been a privacy concern for this maintainer, who has stated
(#437082 (comment)) that they do not wish for their data to be added to the maintainer list.

For this reason, there is no other recourse than to remove this maintainer's information from newer revisions of Nixpkgs, as their maintainer entry is now non-compliant with Nixpkgs' latest policies. Once more, this removal is NOT due to the usual inactivity reasons, but for specific privacy concerns expressed by the maintainer being removed.

The maintainer in question has already expressed his wishes in #437082.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@SigmaSquadron SigmaSquadron added the 1.severity: legal This PR or issue raises or fixes a legal issue, e.g. licensing compliance label Sep 2, 2025
@SigmaSquadron SigmaSquadron mentioned this pull request Sep 2, 2025
1 task
@qzylinra

qzylinra commented Sep 2, 2025

Copy link
Copy Markdown
Contributor

It might be better for the maintainer to proactively open a PR to remove the maintainer entry.

@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 6.topic: python Python is a high-level, general-purpose programming language. 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` labels Sep 2, 2025
@SigmaSquadron

SigmaSquadron commented Sep 2, 2025

Copy link
Copy Markdown
Contributor Author

They had seven days to do so. They opted to express their wishes via a comment instead, and that must be respected despite it not being in the form of a pull request.

@nixpkgs-ci nixpkgs-ci Bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Sep 2, 2025
Although this maintainer has responded to requests for maintainance and
is active in Nixpkgs, the new data collection requirements introduced in
NixOS#437085 have been a privacy
concern for this maintainer, who has stated (NixOS#437082 (comment))
that they do not wish for their data to be added to the maintainer list.

For this reason, there is no other recourse than to remove this
maintainer's information from newer revisions of Nixpkgs, as their
maintainer entry is now non-compliant with Nixpkgs' latest policies.
Once more, this removal is NOT due to the usual inactivity reasons, but
for specific privacy concerns expressed by the maintainer being removed.

Signed-off-by: Fernando Rodrigues <alpha@sigmasquadron.net>
@nixpkgs-ci nixpkgs-ci Bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Sep 2, 2025

@wolfgangwalther wolfgangwalther left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In #437082 (comment) they were given the choice. He made it clear in the following comments that he's not going to add github data to his maintainer entry.

@wolfgangwalther wolfgangwalther merged commit ca48433 into NixOS:master Sep 2, 2025
28 of 31 checks passed
@nixpkgs-ci

nixpkgs-ci Bot commented Sep 2, 2025

Copy link
Copy Markdown
Contributor

Backport failed for release-25.05, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin release-25.05
git worktree add -d .worktree/backport-439407-to-release-25.05 origin/release-25.05
cd .worktree/backport-439407-to-release-25.05
git switch --create backport-439407-to-release-25.05
git cherry-pick -x 81d1a3a2ae011634dc663816c72f4acf8c63c22f

@SigmaSquadron SigmaSquadron mentioned this pull request Sep 2, 2025
13 tasks
@SigmaSquadron SigmaSquadron deleted the push-vpqsnkyzprus branch September 2, 2025 08:07
@wolfgangwalther wolfgangwalther added the 8.has: port to stable This PR already has a backport to the stable release. label Sep 2, 2025
@zeuner

zeuner commented Sep 21, 2025

Copy link
Copy Markdown
Contributor

It's exciting to see that PRs get merged way faster than in the documented 7 days, within a few hours, thereby effectively preventing affected maintainers from reviewing.

I understand it's important for you to quickly remove people who get in the way. Still, I would propose not pushing inaccurate information into the commit history, even more if it the inaccurate information relates to a natural person.

In this case, it was repeatedly stated that the concern about having the GitHub handle added to the maintainer entry is related to ensuring reliable maintainer communication. GitHub based notifications have been unreliable every now and then, and it has even been shown that GitHub based communication can be deliberately hindered by other users. To ensure that the maintainer can be reached reliably, providing only contact details where reliability can be ensured seems to be the only effective way, since adding less reliably channels (like a GitHub handle) can lead to situations where people actually use them, resulting in a bad user experience because they think they cannot reach the maintainer.

The data privacy regulations present in various jurisdictions might be the most promising way for maintainers to defend their right to provide the most reliably communication channel. Still, it's inaccurate to claim unexplained "privacy concerns", and doing so can result in misunderstandings.

@pbsds

pbsds commented Sep 22, 2025

Copy link
Copy Markdown
Member

#437082 was open a week before, you had ample opportunity to fix your maintainer entry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.severity: legal This PR or issue raises or fixes a legal issue, e.g. licensing compliance 6.topic: python Python is a high-level, general-purpose programming language. 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 8.has: port to stable This PR already has a backport to the stable release. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants