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
sieve-connect: Init at 0.89 #43976
sieve-connect: Init at 0.89 #43976
Conversation
Failing check is unrelated |
@GrahamcOfBorg eval |
name = "sieve-connect-${version}"; | ||
version = "0.89"; | ||
|
||
src = fetchurl { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use fetchFromGitHub
instead of fetchurl
because it relies on the hash of the contents of the repository instead of the hash of a tarball, and it would allow us to fix all GitHub src fetchers, if GitHub changes something about how it works.
as specifed in RFC 5804. Historically, this was MANAGESIEVE as | ||
implemented by timsieved in Cyrus IMAP. | ||
''; | ||
homepage = "https://github.com/philpennock/sieve-connect"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
quotes are not needed around URLs
@GrahamcOfBorg build sieve-connect |
Success on x86_64-linux (full log) Attempted: sieve-connect Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: sieve-connect Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: sieve-connect Partial log (click to expand)
|
@ryantm Rebased and fixed |
@GrahamcOfBorg build sieve-connect |
Success on x86_64-darwin (full log) Attempted: sieve-connect Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: sieve-connect Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: sieve-connect Partial log (click to expand)
|
Thanks! |
Motivation for this change
I use this tool to administrate the mail rules on my mail server.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)