write rights to log directory

Bug #838609 reported by Krzysztof Sikorski
4
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Vineyard
Fix Released
Medium
Christian Dannie Storgaard

Bug Description

Currently the programs are run with two log files (output+errors), written into tmp dir ("/tmp/python-wine" if I recall correctly). This creates a problem with multiple users:
1) log into one user, run some programs via vineyard
2) relog into a second user without turning off the machine (so the /tmp dir won't be cleared)
3) try to run a program via vineyard - it will fail on start with IOError (no write rights to the log dir)

The quick-and-dirty solution is of course to create the log dir in /tmp with 777 rights, a nicer solution would be IMHO to change the path somewhere into user's home dir, for example "~/.vineyard/logs/".

Or maybe it *is* configurable and I just did not notice that ;)

description: updated
Revision history for this message
Christian Dannie Storgaard (cybolic) wrote :

Good catch, hadn't thought about that.

I think the solution will be to rename the /tmp/python-wine folder to /tmp/python-wine-$UID (or similar, I'm not that happy with "python-wine"). Does that sound fine?

The reason I'm sticking to /tmp is that Vineyard doesn't clean up log files, so I don't want to risk filling the home dir with "spam".

Revision history for this message
Krzysztof Sikorski (krzysztof-sikorski) wrote :

Good idea, I feel ashamed that I have not thought about uids - and the spam argument is quite strong.
And I can't really help with renaming the prefix, maybe maybe "vineyard-launcher"? I was never good with names...

Revision history for this message
Christian Dannie Storgaard (cybolic) wrote :

There. Fixed in revision 352. The temp dir can now also be set with the environmental variable VINEYARDTMP.

Changed in vineyard:
status: New → Fix Committed
importance: Undecided → Medium
assignee: nobody → Christian Dannie Storgaard (cybolic)
Changed in vineyard:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.