Skip to content

getmail: Use newer getmail6 over getmail package#2982

Merged
teto merged 1 commit intonix-community:masterfrom
romanofski:feat/getmail6
Jun 5, 2022
Merged

getmail: Use newer getmail6 over getmail package#2982
teto merged 1 commit intonix-community:masterfrom
romanofski:feat/getmail6

Conversation

@romanofski
Copy link
Copy Markdown
Contributor

The getmail package will soon be removed from nixpkgs. The
nixos-unstable channel already has it removed and using the service
will result in:

error: getmail has been removed from nixpkgs, migrate to getmail6

Upgrade to the getmail6 package which is already available and backwards
compatible.

@romanofski romanofski requested a review from rycee as a code owner May 27, 2022 09:55
@teto
Copy link
Copy Markdown
Collaborator

teto commented May 30, 2022

The `getmail` package will soon be removed from nixpkgs. The
`nixos-unstable` channel already has it removed and using the service
will result in:

    error: getmail has been removed from nixpkgs, migrate to getmail6

Upgrade to the getmail6 package which is already available and backwards
compatible.
Comment thread modules/services/getmail.nix
@teto teto merged commit a3638db into nix-community:master Jun 5, 2022
@rycee
Copy link
Copy Markdown
Member

rycee commented Jun 6, 2022

@romanofski When opening the PR did you get a template message with a checklist of things to do? If you did and intentionally removed it then please don't do that. It is there to help us review PRs and to avoid inconsistencies in the project. It is fine to disagree with the project contribution guidelines but please raise your concerns publically instead of surreptitiously removing the PR checklist.

@romanofski
Copy link
Copy Markdown
Contributor Author

@romanofski When opening the PR did you get a template message with a checklist of things to do? If you did and intentionally removed it then please don't do that. It is there to help us review PRs and to avoid inconsistencies in the project. It is fine to disagree with the project contribution guidelines but please raise your concerns publically instead of surreptitiously removing the PR checklist.

@rycee I'm very sorry. I did not mean to disagree with any of the guidelines. I presume I removed the checklist since I thought it was a one line change. Again I'm very sorry if it promoted bad behaviour or disagreement. I will not remove the template when filing a new PR.

jevy pushed a commit to jevy/home-manager that referenced this pull request Jul 1, 2022
The `getmail` package will soon be removed from nixpkgs. The
`nixos-unstable` channel already has it removed and using the service
will result in:

    error: getmail has been removed from nixpkgs, migrate to getmail6

Upgrade to the getmail6 package which is already available and backwards
compatible.
@teto teto mentioned this pull request Aug 22, 2022
7 tasks
teto pushed a commit to teto/home-manager that referenced this pull request Aug 22, 2022
The `getmail` package will soon be removed from nixpkgs. The
`nixos-unstable` channel already has it removed and using the service
will result in:

    error: getmail has been removed from nixpkgs, migrate to getmail6

Upgrade to the getmail6 package which is already available and backwards
compatible.
spacekookie pushed a commit to spacekookie/home-manager that referenced this pull request Feb 10, 2023
The `getmail` package will soon be removed from nixpkgs. The
`nixos-unstable` channel already has it removed and using the service
will result in:

    error: getmail has been removed from nixpkgs, migrate to getmail6

Upgrade to the getmail6 package which is already available and backwards
compatible.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants