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

consider changing chmod to @chmod #50

Closed
mkocztorz opened this issue Dec 3, 2014 · 6 comments
Closed

consider changing chmod to @chmod #50

mkocztorz opened this issue Dec 3, 2014 · 6 comments

Comments

@mkocztorz
Copy link

I have just run into trouble when using SF2 + JMS Serializer Bundle on vagrant box (windows using SMB).

The thing is that os can't use chmod on shared filesystems and attempt made here:
https://github.com/schmittjoh/metadata/blob/master/src/Metadata/Cache/FileCache.php#L45
breaks the request.

I noticed that in many places Symfony suppresses the error using @chmod so despite general rule that using @ to hide errors is bad I'd consider using it in this case.

I made a workaround to place my cache dir outside the shared folder structure for now but this fix might be handy.

@gusdecool
Copy link
Contributor

This also happen on my machine with: vagrant box with NFS on OSX.
@mkocztorz the issue that you refer to: symfony/symfony#2125

@cryptiklemur
Copy link

Bump. Not sure why this just started happening.

gusdecool added a commit to gusdecool/metadata that referenced this issue Jun 23, 2015
@gusdecool
Copy link
Contributor

@schmittjoh in case you're agree with this change. i have created PR for it: #53

@dmirkitanov
Copy link

Any luck here? This is the original discussion on the symfony repo: symfony/symfony#8205 - they did switch to @chmod, so it might be a good idea to make such a change here too.

@Gemineye
Copy link

Can you please provide a new tag containing this fix?

@goetas
Copy link
Collaborator

goetas commented May 9, 2018

was solved in e50cc46

@goetas goetas closed this as completed May 9, 2018
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

No branches or pull requests

6 participants