Movatterモバイル変換


[0]ホーム

URL:


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

Sign up
Appearance settings

[Config] Do not use absolute path when computing the vendor freshness#32578

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
lyrixx wants to merge1 commit into3.4fromcache-composer

Conversation

@lyrixx
Copy link
Member

QA
Branch?3.4
Bug fix?yes
New feature?no
BC breaks?no
Deprecations?no
Tests pass?yes
Fixed tickets
LicenseMIT
Doc PR

When one uses Docker with a different mounting point between CLI & FPM,
the cache keeps regenerating because the ComposerResource class see a
different path for each SAPI. For example/home/app/app/vendor vs/var/www/app/vendor.
So if you hit FPM, then the CLI, then FPM, each time a new cache is
generated. So the application is quite slow in dev env. And for people
on MacOSX (with docker) is a big pain! And obvisouly, this never
stabilizes !

This occurs a lot when you have a worker, that crash and reboot in the
background, and you browse the web interface. Or when you have something
that hit your API every X secondes, and you are working on a worker.

matthieuwerner, damienalexandre, ternel, and bastnic reacted with thumbs up emoji
@nicolas-grekasnicolas-grekas added this to the3.4 milestoneJul 17, 2019
$v =\dirname(\dirname($r->getFileName()));
if (file_exists($v.'/composer/installed.json')) {
self::$runtimeVendors[$v] = @filemtime($v.'/composer/installed.json');
self::$runtimeVendors["__vendor_$i"] = @filemtime($v.'/composer/installed.json');

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

This break the getVendors method currently.

Copy link
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Better ?

When one uses Docker with a different mounting point between CLI & FPM,the cache keeps regenerating because the ComposerResource class see adifferent path for each SAPI. For example `/home/app/app/vendor` vs`/var/www/app/vendor`.So if you hit FPM, then the CLI, then FPM, each time a new cache isgenerated. So the application is quite slow in dev env. And for peopleon MacOSX (with docker) is a big pain! And obvisouly, this neverstabilizes !This occurs a lot when you have a worker, that crash and reboot in thebackground, and you browse the web interface. Or when you have somethingthat hit your API every X secondes, and you are working on a worker.
@lyrixx
Copy link
MemberAuthor

Sorry, I pushed on symfony/symony instead of my fork.
I reopened it here:#32579

@lyrixxlyrixx closed thisJul 17, 2019
@lyrixxlyrixx deleted the cache-composer branchJuly 17, 2019 15:36
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@nicolas-grekasnicolas-grekasnicolas-grekas left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.4

Development

Successfully merging this pull request may close these issues.

3 participants

@lyrixx@nicolas-grekas

[8]ページ先頭

©2009-2025 Movatter.jp