-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Declaration of Drush\Command\DrushInputAdapter::hasParameterOption() must be compatible with Symfony\Component\Console\Input\InputInterface::hasParameterOption($values, $onlyParams = false) #2415
Comments
Doesn't happen in 9.x-dev, so maybe a non-issue. |
Why is this closed? Happens on 8.x after a fresh install via composer. |
Compare the parameters for each of these versions of Symfony Console
http://api.symfony.com/2.8/Symfony/Component/Console/Input/InputInterface.html#method_hasParameterOption
(1 param)
http://api.symfony.com/3.1/Symfony/Component/Console/Input/InputInterface.html#method_hasParameterOption
(2 params)
Drush 8 depends on Symfony Console version 2.8 and in the case of the Phar,
it ships with that inside. This is the same version that Drupal 8.2 depends
on.
I'm not sure where you are getting that Console version 3 from. Thats
really messing things up.
|
Going back to 8.1.4 fixed the problem |
This winds up being problematic for any use-case where you use Symfony console version 3, like if you're developing/using modules in Drupal 7 (which has no declared Symfony Console 2.8 requirement) that use it. Any reason the change in dependencies was added to a patch version insead of at least 8.2.0? @stmh From my testing, Drush 8.1.5 is also free from this problem. |
I had the issue today with drush 8.2-dev. |
I closed it because I thought my Drush was just too old to work with composer/console whatever. Seems like #2474 is the issue. |
I upgraded to Drush 9 beta, but not improving. Showing the same error. |
@SUN123RAY Maybe Drush 9 is redispatching to an old version of Drush 8. If your target site is composer-managed, check your composer.lock and see what version of Drush is installed. Update to 8.1.15 or 9. |
@greg-1-anderson Thanks for your comment.
I have tried to find Drush in my composer.lock, but I did not get. my
composer.lock file is attached here.
…On Wed, Nov 8, 2017 at 9:02 PM, Greg Anderson ***@***.***> wrote:
@SUN123RAY <https://github.com/sun123ray> Maybe Drush 9 is redispatching
to an old version of Drush 8. If your target site is composer-managed,
check your composer.lock and see what version of Drush is installed. Update
to 8.1.15 or 9.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2415 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ATOQFByylmHg6013p8sqPdrBIYDlgTxIks5s0gi3gaJpZM4KhjdR>
.
|
sorry @greg-1-anderson I think I have checked in wrong composer.lock. OK here is the correct one attached. |
From the 8.1.7 phar.
The text was updated successfully, but these errors were encountered: