Bug #51809

Commit "[BUGFIX] Published resources don't support symlinks" produces an fatal error on Windows

Added by Thomas Allmer almost 2 years ago. Updated about 1 month ago.

Status:Under Review Start date:2013-09-06
Priority:Must have Due date:
Assigned To:Adrian Föder % Done:

100%
Category:Core
Target version:TYPO3 Flow Base Distribution - 2.0.1
PHP Version:5.4 Complexity:medium
Has patch:No Affected Flow version:Git master

Description

hey
the following change
https://review.typo3.org/#/c/23555/
https://git.typo3.org/Packages/TYPO3.Flow.git/commit/64480cd47066458bfd9a15a9a10f1760e5a0a3b4

produces a fatal error on my System:
Windows 8
PHP 5.4.3

Detailed Error Message:
https://gist.github.com/daKmoR/6461532

Associated revisions

Revision 6d206c1d
Added by Adrian Föder over 1 year ago

[TASK] Resources are published with relative symlinks

Fixes an issue with the publication of persistent resources which
used absolute paths rather than relative paths for generating symbolic
links.

TYPO3.Surf heavily relies on symbolic links for pointing to the
currently active release. The key problem was that the symbolic
link which is put into the Web/_Resources folder used an absolute
path (realpath) to the resource file in Data/Persistent/ rather than
a possible symbolic link. This worked fine as long as the actual
directory (previous release) exists, but breaks as soon as old
releases have been removed.

Resolves: #51676
Resolves: #51809
Releases: master
Change-Id: I2e84cd8f9abd41578a87791813d03e764528feb9

History

#1 Updated by Aske Ertmann almost 2 years ago

  • Category set to Resource
  • Status changed from New to Accepted
  • Assigned To set to Robert Lemke
  • Target version set to 2.0.1
  • Complexity set to medium

#2 Updated by Aske Ertmann almost 2 years ago

Assigned this to you Robert since you merged the change, if you don't have time to fix it just unassign it again..

#3 Updated by Adrian Föder almost 2 years ago

  • Category changed from Resource to Core
  • Assigned To changed from Robert Lemke to Adrian Föder

#4 Updated by Gerrit Code Review almost 2 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23783

#5 Updated by Gerrit Code Review almost 2 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23783

#6 Updated by Gerrit Code Review almost 2 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23783

#7 Updated by Gerrit Code Review almost 2 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23783

#8 Updated by Gerrit Code Review over 1 year ago

Patch set 5 for branch master of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at https://review.typo3.org/23783

#9 Updated by Karsten Dambekalns over 1 year ago

The bugfix that broke this has been reverted in https://review.typo3.org/23785

#10 Updated by Adrian Föder over 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#11 Updated by Gerrit Code Review about 1 year ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch master of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at https://review.typo3.org/30519

#12 Updated by Gerrit Code Review 12 months ago

Patch set 2 for branch master of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/30519

#13 Updated by Gerrit Code Review 12 months ago

Patch set 3 for branch master of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/30519

#14 Updated by Gerrit Code Review 11 months ago

Patch set 4 for branch master of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/30519

#15 Updated by Gerrit Code Review 8 months ago

Patch set 1 for branch 2.3 of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/34878

#16 Updated by Gerrit Code Review 8 months ago

Patch set 2 for branch 2.3 of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/34878

#17 Updated by Gerrit Code Review about 1 month ago

Patch set 5 for branch master of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/30519

Also available in: Atom PDF