Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Add repositories path glob #7444

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 17, 2018
Merged

Add repositories path glob #7444

merged 1 commit into from
Jul 17, 2018

Conversation

ina6ra
Copy link
Contributor

@ina6ra ina6ra commented Jul 6, 2018 •
edited
Loading

In the Composer document, it is stated that glob can be used, but only some functions can be used.
All functions are unnecessary, but I would like to use GLOB_BRACE.

Since now GLOB_BRACE can not be used, it will be written as follows.

{
  "repositories": [{
    "type": "path",
    "url": "$HOME/Scripts/composer",
  } {,
    "type": "path",
    "url": "$HOME/Scripts/php-test",
  }, {
    "type": "path",
    "url": "$HOME/Scripts/php-unit",
  }]
}

If you can use GLOB_BRACE you can write more easily

{
  "repositories": [{
    "type": "path",
    "url": "$HOME/Scripts/{composer,php-*}"
  }]
}

@Seldaek Seldaek added this to the 1.7 milestone Jul 17, 2018
@Seldaek
Copy link
Member

Seldaek commented Jul 17, 2018

Makes sense, thanks.

@Seldaek Seldaek merged commit b3d6a17 into composer:master Jul 17, 2018
@joelwurtz
Copy link
Contributor

joelwurtz commented Jul 18, 2018 •
edited
Loading

This make composer install fail on alpine unfortunately :/

see zendframework/zend-stdlib#58 and wp-cli/wp-cli#4353

@staabm staabm mentioned this pull request Jul 18, 2018
Merged
@staabm
Copy link
Contributor

staabm commented Jul 18, 2018 •
edited
Loading

@joelwurtz could you test whether #7477 fixes your problem?

@joelwurtz
Copy link
Contributor

just do the same and yes it works

@Mekton Mekton mentioned this pull request Sep 2, 2018
Closed
@bastnic bastnic mentioned this pull request Dec 17, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Feature
Projects
None yet
Milestone
1.7
Development

Successfully merging this pull request may close these issues.
4 participants
@ina6ra @Seldaek @joelwurtz @staabm
Morty Proxy This is a proxified and sanitized view of the page, visit original site.