[Filesystem] Add feature to create hardlinks for files #15458
Conversation
There was a problem hiding this comment.
you can make it a regular if condition.
andrerom
force-pushed
the
hard_links
branch
2 times, most recently
from
b43aa9c to
029faeb
Compare
|
Open question: Would it be ok to aim at backporting this for 2.3 and up? I have seen similar small useful features being backported before which is why I ask. For FOSHttpCache that would be helpful to not have to require 2.8. |
|
@andrerom no it is not. New features are never backported, unless it is necessary to fix a security issue (or a PHP compatibility issue) |
|
ok, clear. |
There was a problem hiding this comment.
please use sprintf rather than interpolation, as mandated by our coding standards: https://symfony.com/doc/current/contributing/code/standards.html#structure
This actually depends on the use case. Saying that hardlinks is better is misleading. For instance, if you use hardlinks on a PHP file, |
andrerom
force-pushed
the
hard_links
branch
2 times, most recently
from
8a35fee to
0af9468
Compare
Right, corrected that statement, I was actually never considering the use case of linking executable php files with actual code as opposed to cache files or binary files. I would have expected it to report |
andrerom
force-pushed
the
hard_links
branch
6 times, most recently
from
fae8b93 to
89e46ad
Compare
|
Not sure why this PR staled. @andrerom Can you rebase on current 2.8? I will then be able to merge this into master. |
|
@fabpot done, failure with fabbot.io seems to be unrelated to changes here. |
There was a problem hiding this comment.
should be private, and imo there's no need for the docblock here (same below for $symlinkOnWindows)
There was a problem hiding this comment.
Fixed in 584cafe
I did not remove the phpdoc, the other properties have doc so I rather fixed these to say null|bool to give correct hints for IDE's.
|
Thank you @andrerom. |
7 participants
Todo:
Why
Symlinks are good for directories, but for files hardlinks are sometime a better match when needing to represent same file in several locations.
One use case for this feature is for multi tagging of Http Cache when running against FileSystem. Multi tagging is used in FoSHttpCache and supported when running Varnish, but not with Symfony HTTPCache, yet.. With hardlinks combined with meta information containing tags within the file, lookup and reverse lookup is thus easily possible.
What
Introduces method
hardlink()with similar behavior assymlink, difference being:symlinkremoves existing target if not the same, but usesfileinodeto compare target & source