feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

Utils: Implement FilePath::removeRecursively

Browse Source 
Sprinkle a few QTC_ASSERT to make sure we are in a known
context.

Cmake's FileApiReader uses this before moving directories
as QFile::rename does not do anything if the target exists.

Change-Id: I555f99e81a9fe7d93ae66145eeebfa9d5880bc51
Reviewed-by: Christian Stenger <christian.stenger@qt.io>
This commit is contained in:
hjk
2021-06-28 15:26:44 +02:00
parent 5470385085
commit 5ead2afeeb
8 changed files with 47 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/plugins/docker/dockerdevice.cpp
View File

@@ -878,6 +878,24 @@ bool DockerDevice::removeFile(const FilePath &filePath) const
return exitCode == 0;
}
bool DockerDevice::removeRecursively(const FilePath &filePath) const
{
QTC_ASSERT(handlesFile(filePath), return false);
QTC_ASSERT(filePath.path().startsWith('/'), return false);
tryCreateLocalFileAccess();
if (hasLocalFileAccess()) {
const FilePath localAccess = mapToLocalAccess(filePath);
const bool res = localAccess.removeRecursively();
LOG("Remove recursively? " << filePath.toUserOutput() << localAccess.toUserOutput() << res);
return res;
}
// Open this up only when really needed.
// const CommandLine cmd("rm", "-rf", {filePath.path()});
// const int exitCode = d->runSynchronously(cmd);
// return exitCode == 0;
return false;
}
bool DockerDevice::copyFile(const FilePath &filePath, const FilePath &target) const
{
QTC_ASSERT(handlesFile(filePath), return false);