Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation for this change
https://roundcube.net/news/2019/11/09/roundcube-1.4.0-released
curl
cmd in the test can fail as roundcube returns a http/401 ifunauthorized (and we're explicitly requesting the login form). By
checking if the
persistent_login
plugin is loaded, the assertion isstill valid.
Use
$argv[0]
to determine install path in the installer script. I'mnot exactly sure why, but it seems as
__DIR__
now resolves symlinkswhich breaks the installer if roundcube is in a
buildEnv
withthird-party plugins.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)