Bug #51542

Install Tool: Warning: touch() Utime failed: Permission denied

Added by Ernesto Baschny almost 6 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Category:
Install Tool
Target version:
Start date:
2013-08-29
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

In #20686 this has been fixed in the "old install tool" but this problem appears again in the "new install tool" since the touch() is made again without a @ to suppress warnings.

A more robust enhancement would be to - as a fallback in case "touch" is not allowed - to remove the file and create it again. This just requires write permissions in the typo3conf directory - which should be there in any case - instead of inode-modify permissions on the ENABLE_INSTALL_TOOL file, which might not be there in case "root" or someone else has created the ENABLE file.


Related issues

Related to TYPO3 Core - Bug #20686: Warning: touch() [function.touch]: Utime failed Closed 2009-06-27
Related to TYPO3 Core - Task #49162: Rewrite install tool Closed 2011-10-30

Associated revisions

Revision 792f84ca (diff)
Added by Ernesto Baschny almost 6 years ago

[BUGFIX] Install Tool: Warning: touch() Utime failed: Permission denied

Despite getting rid of this warning, extending the lifetime of this file
gets more robust by a second method, which is recreating it (which should
be possible even if the file is not owned by the PHP process).

Resolves: #51542
Releases: 6.2
Change-Id: Icb7805ee1f6a08654c3f11eb317a2ad3cadbad76
Reviewed-on: https://review.typo3.org/23477
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

History

#1 Updated by Gerrit Code Review almost 6 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/23477

#2 Updated by Gerrit Code Review almost 6 years ago

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

#3 Updated by Ernesto Baschny almost 6 years ago

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

#4 Updated by Riccardo De Contardi over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF