Skip to content
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

pls only "suggest" angrychimp/php-dkim #32

Open
ThomasCr opened this issue Sep 18, 2018 · 3 comments
Open

pls only "suggest" angrychimp/php-dkim #32

ThomasCr opened this issue Sep 18, 2018 · 3 comments

Comments

@ThomasCr
Copy link

Hi,
today I wanted to install authres_status with composer.phar, but it stops with:

Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for roundcube/authres_status 0.5.0 -> satisfiable by roundcube/authres_status[0.5.0].
    - roundcube/authres_status 0.5.0 requires angrychimp/php-dkim @dev -> satisfiable by angrychimp/php-dkim[dev-master] but these conflict with your requirements or minimum-stability.

But when I am not wrong, this is only optional? So it s better to set it as suggest in your composer.json

Thanks

@pimlie
Copy link
Owner

pimlie commented Sep 18, 2018

I think that in general its better to have the dependency as a requirement. The install 'only' stops because you need to add "minimum-stability": "dev" to your composer.json. This is because on packagist there is no version available for php-dkim. I have talked to @angrychimp some time ago to see if he could release a version so we can skip the dev requirement. Unfortunately he still hasnt found the time yet to do that.

@ThomasCr
Copy link
Author

but when php-dkim is optional, why force to install it?
My mailserver adds the needed headers, so why should I need it?

@angrychimp
Copy link

Right now php-dkim validates DKIM signatures. I've been ignoring the project for quite some time but I do need to get back to it. I'll see if I can resolve this issue by preparing a new version that drops the requirement for phpseclib (making it a suggestion), and finally include a stable release version.

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

No branches or pull requests

3 participants