Bug 116789

Summary:	Get new Superkaramba theme from internet (with KHotNewStuff) doesn't work
Product:	superkaramba	Reporter:	Michal Krenek (Mikos) <mikos>
Component:	general	Assignee:	Ryan Nickell <p0z3r>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:

Description Michal Krenek (Mikos) 2005-11-20 21:53:45 UTC

Version:            (using KDE KDE 3.5.0)
Installed from:    Unlisted Binary Package

If I try Get New Stuff in SuperKaramba, it seems to work at first, but after download it displays this error message:

Could not rename partial file /tmp/kde-mikos/.
Please check permissions.

But /tmp/kde-mikos is directory, not file. My permissions on /tmp are drwxrwxrwt and /tmp/kde-mikos have drwx------ permissions (and I am owner of this directory).

Comment 1 Ryan Nickell 2005-11-21 05:13:34 UTC

The only time I see this is when a download link posted on kdelook.org is actually a redirect to a website instead of the link to the download file.
For example I think that Liquid Weather updated the link to point to his new site, but TechMon downloaded and unpacked fine for me.

Please verify this, as it is not an error with SK, however, we could provide a better error message to indicate what went wrong in the process.

Comment 2 Hogne Titlestad 2005-12-05 08:14:41 UTC

It seems like a bug, as when the file fails, SuperKaramba acts as if the file is installed, and so you can't to install it anew, nor uninstall the failed file.

Comment 3 Ryan Nickell 2005-12-14 05:16:51 UTC

SVN commit 488353 by nickell:

BUG:116789  Links are now shown in a browser window for users to
download manually.


 M  +2 -1      sknewstuff.cpp  


--- branches/KDE/3.5/kdeutils/superkaramba/src/sknewstuff.cpp #488352:488353
@@ -138,7 +138,8 @@
   QString file(source.fileName());
   if ( file.isEmpty() )
   {
-    kdDebug() << "the file was empty, this must be a direct URL" << endl;
+    kdDebug() << "The file was empty. " << source.url() << 
+      " must be a direct URL" << endl;
     KRun::runURL( source, "text/html");
     return file;
   }