This work for me

composer require vendor/package=version

But this is not

drupal composer-manager require vendor/package=version

https://github.com/andytruong/composer_manager/commit/05f11279a7b44c3ec2...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cpliakas’s picture

Status: Active » Needs review

Hi thehong.

Thanks for the contribution. The arguments are definitely supposed to be passed, so I'll try out your patch.

Once concern I have with what you posted is that the require vendor/package=version will likely not work as expected. It will require the package when you run the command, however the package definition will be overwritten whenever Composer Manager rebuilds the composer.json file. I just want to make sure that you are aware that if you are requiring a package through this mechanism that you will have to run it every time you update composer dependencies.

Thanks!
Chris

cpliakas’s picture

Status: Needs review » Reviewed & tested by the community

Regardless of my comment, the patch works as expected.

Thanks for the contribution!

  • Commit ac70100 on 7.x-2.x authored by thehong, committed by cpliakas:
    Issue #2248475 by thehong: Arguments are not sent to composer as...

  • Commit ca63d9c on 7.x-1.x authored by thehong, committed by cpliakas:
    Issue #2248475 by thehong: Arguments are not sent to composer as...

  • Commit 4b355d3 on 6.x-2.x authored by thehong, committed by cpliakas:
    Issue #2248475 by thehong: Arguments are not sent to composer as...

  • Commit 235b9eb on 6.x-1.x authored by thehong, committed by cpliakas:
    Issue #2248475 by thehong: Arguments are not sent to composer as...

  • Commit 3b865cc on 8.x-1.x authored by thehong, committed by cpliakas:
    Issue #2248475 by thehong: Arguments are not sent to composer as...
cpliakas’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.