Bug #57413
closedSymlink check on /typo3 directory is not working correctly if target ends with a slash
100%
Description
I just did a test installation of TYPO3 6.2 and noticed that the Install Tool doesn't recognize my symlink for /typo3.
I created this symlink using tab-completion so the link target ends with a slash. But the Install Tool seems not to be able to read the target in that case.
Install Tool message:
/typo3 is a link, but link target is not as specified Link target should be typo3_src/typo3 but is typo3_src/typo3/
Symlink commands causing that message:
$ ln -s typo3_src/typo3/ $ ln -s typo3_src/typo3/ . $ ln -s typo3_src/typo3/ typo3
Symlink command NOT causing that message:
$ ln -s typo3_src/typo3 $ ln -s typo3_src/typo3 . $ ln -s typo3_src/typo3 typo3
A simple cli test is working for both cases:
$ ln -s typo3_src/typo3 $ php -r "var_dump(readlink('typo3'));" string(15) "typo3_src/typo3" $ ln -s typo3_src/typo3/ $ php -r "var_dump(readlink('typo3'));" string(16) "typo3_src/typo3/"
Updated by Gernot Ploiner about 10 years ago
I can confirm this issue.
To reproduce, use this command when installing TYPO3:
ln -s typo3_src/typo3/ typo3
But it works perfect with this command:
ln -s typo3_src/typo3 typo3
It occurs in all Versions of TYPO3 6.2. Tested today with the newest 6.2.5
I think this is a small issue and will resolve this problem for a lot of people.
thank you
gernott
Updated by Gerrit Code Review almost 10 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36339
Updated by Christian Kuhn almost 10 years ago
- Sprint Focus set to On Location Sprint
Updated by Gerrit Code Review almost 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36339
Updated by Andreas Fernandez almost 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 12ec9d2384e448cdc2044bf70a9314790fe7c11a.
Updated by Gerrit Code Review over 9 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37749
Updated by Andreas Fernandez over 9 years ago
- Status changed from Under Review to Resolved
Applied in changeset 65c2fd00490439621634939c012bac3c5210e31c.
Updated by Anja Leichsenring almost 9 years ago
- Sprint Focus deleted (
On Location Sprint)