search

pimlie / authres_status

Roundcube plugin that checks the Authentication-Results headers (rfc5451) of your emails and displays the verification status.
Other
34 stars 20 forks source link

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

Open ThomasCr opened 5 years ago

ThomasCr commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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

angrychimp commented 5 years ago

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.