KDE Gear projects with failing CI (master) (4 June 2024)

Thiago Macieira thiago at kde.org
Sat Jun 15 00:58:00 BST 2024


On Thursday 13 June 2024 14:08:43 GMT-7 Elvis Angelaccio wrote:
>       auto tempDir = new QTemporaryDir(QDir::currentPath() +
> "/.test-qtempdir-");

Indeed, that works, as does trying QFileInfo or QDir on the returned path and 
then using absolutePath.

> I'd expect at least the QTemporaryDir documentation to get updated with a
> warning about this new behavior. 

True, but I think it's unnecessary. That doesn't fix existing code. For anyone 
writing new code, they'll realise the problem right away, so they'll make the 
path absolute or deal with the relative path.

> Currently it still claims that
> "Auto-remove mode will automatically delete the directory from disk upon
> destruction.", but this is no longer true if you happened to create the
> QTemporaryDir with a relative template and then change the current
> directory in the process before deleting the QTemporaryDir instance.

True.

But there's also a converse case: if you renamed the current directory, the 
old code had failed, but now it works.
-- 
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
  Principal Engineer - Intel DCAI Fleet Systems Engineering





More information about the kde-devel mailing list