Skip to content
Snippets Groups Projects

Issue #3182188: Updates composer usage to point at ./vendor/bin/composer

Open Issue #3182188: Updates composer usage to point at ./vendor/bin/composer
Open Ryan Aslett requested to merge issue/drupal-3182188:3182188-core-composer-tests into 9.2.x
1 unresolved thread

This should update internal usages of core in tests to use ./vendor/bin/composer #3182188

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
24 24 'behat/mink-browserkit-driver' => ['tests'],
25 25 'behat/mink-goutte-driver' => ['tests'],
26 26 'behat/mink-selenium2-driver' => ['tests'],
27 'composer/composer' => ['bin'],
  • I don't think we should take this out. We don't want to have composer/composer exist in a hardened legacy project.

    I'd be +1 on simply removing the vendor hardening plugin from the root composer.json file in drupal/drupal. This would likely impact the vendor hardening tests.

    If we are not ready to do that, then perhaps we could add a vendor configuration override to the root composer.json file to disable removal of the Composer binary for drupal/drupal. We would have to enhance vendor hardening to allow subtractions from its config in order to do this, though; currently, the only option is to merge hardening configuration.

  • Yeah, that was just me experimenting to see if I got the path wrong or if I was stymied by the hardening cleanup.. turns out it was the former.

  • changed this line in version 3 of the diff

  • Please register or sign in to reply
  • Ryan Aslett added 1 commit

    added 1 commit

    Compare with previous version

  • Ryan Aslett added 1 commit

    added 1 commit

    • 0ad56a6d - Removes the ./ but I know this wont work

    Compare with previous version

  • Brad Jones added 4090 commits

    added 4090 commits

    Compare with previous version

  • Please register or sign in to reply
    Loading