Skip to content
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

Pass workdir parameter to composer config #247

Closed
wants to merge 1 commit into from

Conversation

mxr576
Copy link

@mxr576 mxr576 commented Aug 21, 2023

Fixes #246

Description

See #246

Motivation and context

See #246

How has this been tested?

How this can be tested? 🤔

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

PR checklist

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • I have added tests to cover my changes.

@mxr576 mxr576 requested a review from ramsey as a code owner August 21, 2023 08:49
@mxr576
Copy link
Author

mxr576 commented Aug 21, 2023

This was POC that I ran locally and figured that this can be the root cause.

$ cd /tmp
$ composer config cache-dir

In ConfigCommand.php line 206:
                                                                   
  File "./composer.json" cannot be found in the current directory  
                                                                   

config [-g|--global] [-e|--editor] [-a|--auth] [--unset] [-l|--list] [-f|--file FILE] [--absolute] [-j|--json] [-m|--merge] [--append] [--source] [--] [<setting-key> [<setting-value>...]]

$ composer -d /mnt/files/local_mount/build config cache-dir # this is a composer project root
/home/wodby/.composer/cache

@mxr576 mxr576 mentioned this pull request Aug 21, 2023
7 tasks
@mxr576
Copy link
Author

mxr576 commented Aug 21, 2023

Okay, so this is already addressed in #233

@mxr576 mxr576 closed this Aug 21, 2023
@mxr576 mxr576 reopened this Aug 29, 2023
@ramsey ramsey closed this Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

File "./composer.json" cannot be found in the current directory
2 participants