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

Ingo Klöcker kloecker at kde.org
Fri Jun 14 08:57:30 BST 2024


On Donnerstag, 13. Juni 2024 23:08:43 CEST Elvis Angelaccio wrote:
> On 09/06/24 17:16, Thiago Macieira wrote:
> > The path is correct at the point it is issued, referring to the same
> > directory. The fact that it is in a different form was not expected to be
> > a
> > breaking change.
> 
> I had a look at the 6.7 release notes and I've found the commit 935562a77b
> which seems to be what changed this behavior.
> 
> Indeed, if I apply the same fix done in that commit to the test-case I
> attached before, i.e. if I replace
> 
> 	auto tempDir = new QTemporaryDir(".test-qtempdir-");
> 
> with
> 	auto tempDir = new QTemporaryDir(QDir::currentPath() + "/.test-
qtempdir-");
> 
> then the returned QTemporaryDir::path() goes back to being absolute.
> 
> I'd expect at least the QTemporaryDir documentation to get updated with a
> warning about this new behavior. 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.

That looks like a serious unforeseen regression and was surely not intended. 
Best provide a simple test case for QTemporaryDir for this.

Regards,
Ingo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-devel/attachments/20240614/18bdcf9f/attachment.sig>


More information about the kde-devel mailing list