-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
nixos/roundcube: add roundcube module #48049
Conversation
@globin I try to make test like this :
but I have a problem when I test it with
Maybe can you help me ? |
I'm working on finishing this up, will probably be ready tomorrow. |
Thanks :) |
@globin I'm currently setting up a mailing environment and managed to get roundcube running, so in case I can help with review/test/whatever feel free to ping me :) |
There's a work-in-progress: https://github.com/mayflower/nixpkgs/commits/roundcube-module I privately hacked something together to get this working on my personal setup, but this is certainly not suitable for NixOS upstream and I don't want to duplicate efforts now :) |
a9c7917
to
6a0788d
Compare
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.
Some comments, but looks quite good so far 👍
Can you also add some basic module documentation, like done in #51053 for nextcloud?
e7e052b
to
801cc81
Compare
801cc81
to
74f926c
Compare
@GrahamcOfBorg test roundcube |
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: tests.roundcube Partial log (click to expand)
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: tests.roundcube Partial log (click to expand)
|
74f926c
to
5b836a0
Compare
5b836a0
to
1262a5c
Compare
@GrahamcOfBorg test roundcube |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: tests.roundcube Partial log (click to expand)
|
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: tests.roundcube Partial log (click to expand)
|
I'll do a manual test of the database migration functionality today, but looking quite good so far! |
I downgraded roundcube to 1.2.2 with that module, created a container with it inside, then reverted the downgrade. Due to the current flakyness of declarative containers and reload not working properly, I had to restart the container. On bootup,
|
Motivation for this change
After failing me on my local repo, I suggest this module for roundcube, and I closed the #47655. I try to take in account @globin remarks.
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)