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

Symbolic Links #89

Closed
hborawski opened this issue Sep 4, 2014 · 8 comments
Closed

Symbolic Links #89

hborawski opened this issue Sep 4, 2014 · 8 comments
Assignees
Labels

Comments

@hborawski
Copy link

In a testing situation, I will need the php builtin readlink to resolve a symbolic link in the virtual filesystem.

I will gladly implement this, and would just like to know of any existing code towards this feature so that it may be utilized (I suspect code does exists because the const for TYPE_LINK exists in vfsStreamContent already).

@mikey179
Copy link
Member

mikey179 commented Sep 4, 2014

There is no further code for this feature. I had the thought of implementing it, but didn't come up with a good idea how to do it, and as I didn't have the need for it no further attempts were made. So go ahead. :-)

@hborawski
Copy link
Author

Alright, I'm working on some solutions, I'll let you know what I come up with.

mikey179 pushed a commit that referenced this issue Nov 30, 2014
In order to make this work you need to compile your own PHP from
https://github.com/thekid/php-src branch rfc/linking_in_stream_wrappers
@mikey179 mikey179 self-assigned this Nov 30, 2014
@mikey179
Copy link
Member

Started working on this topic together with @thekid. See our RFC for PHP. The current implementation state can be retrieved from https://github.com/thekid/php-src, branch rfc/linking_in_stream_wrappers. Additionally, in vfsStream's linking branch implementation has started.

@malc0mn
Copy link

malc0mn commented May 28, 2018

Is this one ready for release? Anything that can be done to help?

@mikey179
Copy link
Member

We started but never got around to finish it. As for now I think this feature is dead - mostly because it also requires changes in PHP and I don't see anyone going ahead to make those changes. Also, I believe the chance of those changes being merged into PHP are quite low, as to my knowledge they would only be beneficial for vfsStream and nothing else. But if you want to, you can take over the RFC, provide an implementation and see if it gets accepted.

@malc0mn
Copy link

malc0mn commented May 29, 2018

Thanks for the status update. I must admit that I totally missed the part about the RFC, otherwise I wouldn't have even asked.
Quite bold to go there though as I do agree that linking in stream wrappers would only be useful for vfsStream 😄

@mikey179
Copy link
Member

mikey179 commented Jun 4, 2018

Closing this for now.

@mikey179 mikey179 closed this as completed Jun 4, 2018
@pascal-chenevas
Copy link

pascal-chenevas commented Apr 3, 2023

Hi!

I am in the same case as @hborawski. I have some tests scenarios that must involve symbolic links.
May I ask please, what is the state of this topic in the project?
Did you solve this with any workaround?
Thank you in advance!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants