Bug #13979

Problem with deleting package

Added by Dawid over 10 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
Could have
Category:
Package
Start date:
2011-03-22
Due date:
% Done:

100%

Estimated time:
PHP Version:
Has patch:
Complexity:

Description

Looks like there is some kind of problem with recursive deleting files.

After typing ./flow3 package delete Packagekey i receive

Uncaught Exception in FLOW3 #1: Warning: rmdir(/var/www/flow/public/Packages/Application/Offices/Resources/Private/Templates/Office): Directory not empty in /var/www/flow/public/Packages/Framework/FLOW3/Classes/Utility/Files.php line 140
thrown in file Packages/Framework/FLOW3/Classes/Error/ErrorHandler.php
in line 82

Offices is my test package key.
It looks like it is tring to use rm with out -R parameter


Related issues

Related to TYPO3.Flow - Bug #25472: Files::emptyDirectoryRecursively() cannot remove directories starting with a dotResolvedKarsten Dambekalns2011-03-30

Actions
#1

Updated by Karsten Dambekalns over 10 years ago

  • Category set to Package
  • Status changed from New to Accepted
  • Assignee set to Karsten Dambekalns
  • Priority changed from Must have to Could have

No, it tries to do the right thing, but fails to remove some items due to permission problems. Thus some files are left and the rmdir() fails.

Solution: catch those errors and emit a more helpful message.

#2

Updated by Karsten Dambekalns over 10 years ago

  • Target version set to 1.0 beta 1

Another possible cause: there is a .svn or .git directory in your package directory. That cannot be deleted right now, see #25472.

#3

Updated by Mr. Hudson over 10 years ago

Patch set 1 of change I445530204cbe7413879b3130610ebc8ba3ab685f has been pushed to the review server.
It is available at http://review.typo3.org/1326

#4

Updated by Mr. Hudson over 10 years ago

Patch set 2 of change I445530204cbe7413879b3130610ebc8ba3ab685f has been pushed to the review server.
It is available at http://review.typo3.org/1326

#5

Updated by Karsten Dambekalns over 10 years ago

  • Status changed from Accepted to Under Review
#6

Updated by Dawid over 10 years ago

Hi guys, thanks for checking.

First of all..no .svn or .git folders where there. For sure

Priviliges also shouldn`t be a problem because all Packages for test had chmod 777 recursivly for all folders. So I think that those two basic things ain`t relevant

#7

Updated by Mr. Hudson over 10 years ago

Patch set 3 of change I445530204cbe7413879b3130610ebc8ba3ab685f has been pushed to the review server.
It is available at http://review.typo3.org/1326

#8

Updated by Mr. Hudson over 10 years ago

Patch set 4 of change I445530204cbe7413879b3130610ebc8ba3ab685f has been pushed to the review server.
It is available at http://review.typo3.org/1326

#9

Updated by Karsten Dambekalns over 10 years ago

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

Also available in: Atom PDF