By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 2105 - K3B can't find /tmp/kde-{username}
Summary: K3B can't find /tmp/kde-{username}
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: non-core programs (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Linux
: P5 minor
Assignee: Slávek Banko
URL:
Depends on:
Blocks:
 
Reported: 2014-09-05 14:03 CDT by Kristopher
Modified: 2014-12-08 21:37 CST (History)
5 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:


Attachments
tdebase: update r14-xdg-update: update references to temp folders (1.84 KB, patch)
2014-12-08 12:26 CST, Slávek Banko
Details | Diff
tdebase: update r14-xdg-update: update references to temp folders (1) (2.75 KB, patch)
2014-12-08 15:40 CST, Slávek Banko
Details | Diff
tdebase: update r14-xdg-update: update references to temp folders (2) (3.65 KB, patch)
2014-12-08 18:10 CST, Slávek Banko
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kristopher 2014-09-05 14:03:26 CDT
When applying settings changes in K3B in TDE R14, I get a dialog asking me to create /tmp/kde-{username} . Looking in /tmp reveals that there is a /tmp/tde-{username} but not a /tmp/kde-{username} . This suggests to me that there was a renaming of the directories, and K3B was not informed of the change.

Allowing K3B to create the directory /tmp/kde-{username} allows it to run normally.
Comment 1 Kristopher 2014-12-07 13:14:10 CST
I just realized I can set K3B to use /tmp/tde-{username} , however I will leave this bug open since the default should be changed to the correct path.
Comment 2 Timothy Pearson 2014-12-07 14:08:14 CST
(In reply to Kristopher from comment #1)
> I just realized I can set K3B to use /tmp/tde-{username} , however I will
> leave this bug open since the default should be changed to the correct path.

This might have been a result of the upgrade from 3.5.13.x to R14, as the k3b configuration file would likely have retained the old kde- temp directory name.  Can you see if this bug exists in your clean (non-upgraded) R14 VM?

Thanks!

Tim
Comment 3 Kristopher 2014-12-07 14:26:55 CST
(In reply to Timothy Pearson from comment #2)
> (In reply to Kristopher from comment #1)
> > I just realized I can set K3B to use /tmp/tde-{username} , however I will
> > leave this bug open since the default should be changed to the correct path.
> 
> This might have been a result of the upgrade from 3.5.13.x to R14, as the
> k3b configuration file would likely have retained the old kde- temp
> directory name.  Can you see if this bug exists in your clean (non-upgraded)
> R14 VM?
> 
> Thanks!
> 
> Tim

No need for testing this in a VM (I've copied my settings anyways for testing another bug, I forget which): I switched to a dummy user I have set up, and under that user K3B uses the correct /tmp/tde-{username} .

If this is as a result of upgrading from 3.5.x, I'd suggest adding a check to the R14 version TDE applications that use the old /tmp/kde-{username} to see if that needs updating. That will save a lot of confusion for non-advanced users when R14 becomes stable.
Comment 4 Timothy Pearson 2014-12-07 14:33:38 CST
(In reply to Kristopher from comment #3)
> (In reply to Timothy Pearson from comment #2)
> > (In reply to Kristopher from comment #1)
> > > I just realized I can set K3B to use /tmp/tde-{username} , however I will
> > > leave this bug open since the default should be changed to the correct path.
> > 
> > This might have been a result of the upgrade from 3.5.13.x to R14, as the
> > k3b configuration file would likely have retained the old kde- temp
> > directory name.  Can you see if this bug exists in your clean (non-upgraded)
> > R14 VM?
> > 
> > Thanks!
> > 
> > Tim
> 
> No need for testing this in a VM (I've copied my settings anyways for
> testing another bug, I forget which): I switched to a dummy user I have set
> up, and under that user K3B uses the correct /tmp/tde-{username} .
> 
> If this is as a result of upgrading from 3.5.x, I'd suggest adding a check
> to the R14 version TDE applications that use the old /tmp/kde-{username} to
> see if that needs updating. That will save a lot of confusion for
> non-advanced users when R14 becomes stable.

Thanks for the info.  It seems then that the R14 upgrade script isn't properly upgrading the k3b files so I'm going to add Darrell and Slavek to the CC list.  Can one of you please fix the R14 upgrade script to handle this case?

Thanks!
Comment 5 Slávek Banko 2014-12-08 12:26:14 CST
Created attachment 2391 [details]
tdebase: update r14-xdg-update: update references to temp folders

Update references for /tmp/kde-*, /tmp/ksocket-* and /var/tmp/kdecache-*.

Tim is already too late to push this patch into R14.0.0?
Comment 6 Timothy Pearson 2014-12-08 13:14:32 CST
(In reply to Slávek Banko from comment #5)
> Created attachment 2391 [details]
> tdebase: update r14-xdg-update: update references to temp folders
> 
> Update references for /tmp/kde-*, /tmp/ksocket-* and /var/tmp/kdecache-*.
> 
> Tim is already too late to push this patch into R14.0.0?

Go ahead and push now; I didn't have time to update the TDE version code or tag R14 so this one patch can go in.
Comment 7 Darrell 2014-12-08 13:20:08 CST
The patch is hard-coded to look in /tmp. In the situation where $TMP is defined in user's environment, the temporary files will not be stored in /tmp but in $TMP.

Rather than hard-code for /tmp, use the variables defined at lines 278-280?

CACHE_DIR="`readlink $PROFILE_DIR/cache-\`uname -n\``"
SOCKET_DIR="`readlink $PROFILE_DIR/socket-\`uname -n\``"
TMP_DIR="`readlink $PROFILE_DIR/tmp-\`uname -n\``"
Comment 8 Timothy Pearson 2014-12-08 14:10:40 CST
(In reply to Darrell from comment #7)
> The patch is hard-coded to look in /tmp. In the situation where $TMP is
> defined in user's environment, the temporary files will not be stored in
> /tmp but in $TMP.
> 
> Rather than hard-code for /tmp, use the variables defined at lines 278-280?
> 
> CACHE_DIR="`readlink $PROFILE_DIR/cache-\`uname -n\``"
> SOCKET_DIR="`readlink $PROFILE_DIR/socket-\`uname -n\``"
> TMP_DIR="`readlink $PROFILE_DIR/tmp-\`uname -n\``"

I'll leave the merge window for this specific bug open until 12/09/2014 06:00 GMT to allow time to "get it right".

Tim
Comment 9 Slávek Banko 2014-12-08 15:40:38 CST
Created attachment 2393 [details]
tdebase: update r14-xdg-update: update references to temp folders (1)

Well, I modified the patch.
Please verify before I push it.
Comment 10 Slávek Banko 2014-12-08 18:10:15 CST
Created attachment 2394 [details]
tdebase: update r14-xdg-update: update references to temp folders (2)

Forget the previous variant of the patch! First, the directories may be deleted in the previous step. In addition, variables may contain original folder names. Now is from the variables used only path to parent directory.
Comment 11 Slávek Banko 2014-12-08 21:36:24 CST
Comment on attachment 2394 [details]
tdebase: update r14-xdg-update: update references to temp folders (2)

Well, no comments, and time is running out. Therefore, I pushed the patch.