Skip to content

-no-cache argument for "composer install" and "composer update"#7880

Merged
Seldaek merged 2 commits intocomposer:masterfrom
Dzhuneyt:6650-disable-cache-argument
Jan 28, 2019
Merged

-no-cache argument for "composer install" and "composer update"#7880
Seldaek merged 2 commits intocomposer:masterfrom
Dzhuneyt:6650-disable-cache-argument

Conversation

@Dzhuneyt
Copy link
Copy Markdown
Contributor

@Dzhuneyt Dzhuneyt commented Jan 7, 2019

Fixes #6650 and saltstack/salt#45093

However, I was only able to test it under Unix. If someone can check the solution under Windows - that'd be great. Tests run green.

@Seldaek Seldaek merged commit 8c30b12 into composer:master Jan 28, 2019
Seldaek added a commit that referenced this pull request Jan 28, 2019
@Seldaek
Copy link
Copy Markdown
Member

Seldaek commented Jan 28, 2019

Thanks, see 02ceb74 tho for some review/changes.

aboks pushed a commit to aboks/composer that referenced this pull request Jan 31, 2019
@Dzhuneyt Dzhuneyt deleted the 6650-disable-cache-argument branch February 1, 2019 13:08
@mnapoli
Copy link
Copy Markdown

mnapoli commented Feb 4, 2019

This is awesome!

However it creates this exception when there is a dependency using a git URL:

GitDriver requires a usable cache directory, and it looks like you set it to be disabled

Is there any way that the cache is still used when it's absolutely necessary? That would avoid the exception while still storing as little as possible on disk.

@Seldaek
Copy link
Copy Markdown
Member

Seldaek commented Feb 8, 2019

If you only care about disk space I think it'd be easiest to run it normally and then wipe the cache dir after your build is done.

@mnapoli
Copy link
Copy Markdown

mnapoli commented Feb 8, 2019

🤔 that's a good idea. I run Composer (and php-cs-fixer for https://prettyci.com/) on AWS Lambda, where I only have limited space in the /tmp folder. That would work, unless there's more of 500mb of composer cache, but I don't think that happens often! Thanks!

@Seldaek
Copy link
Copy Markdown
Member

Seldaek commented Feb 8, 2019

500mb sounds like a lot for a single install or update cycle.

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.

3 participants