Opened 18 years ago
Closed 18 years ago
Last modified 18 years ago
#4593 closed defect (fixed)
/.mythtv/ should *not* get created (blank homedir)
Reported by: | xris | Owned by: | Nigel |
---|---|---|---|
Priority: | minor | Milestone: | 0.21 |
Component: | mythtv | Version: | head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
Myth should probably error out if no homedir is defined, rather than creating a .mythtv directory a /
Attachments (1)
Change History (13)
comment:1 by , 18 years ago
Owner: | changed from | to
---|---|
Status: | new → accepted |
comment:2 by , 18 years ago
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
comment:3 by , 18 years ago
Resolution: | fixed |
---|---|
Status: | closed → new |
Even though QDir::homeDirPath() will use other environment variables on Windows when HOME is not set, libdvdnav needs HOME, and segfaults without it.
comment:4 by , 18 years ago
An additional comment - I run my mythbackend out of inittab, and since HOME is not set there, this change would have broken my ability to do this without hacking around at the environment in a wrapper script. I have since commented out the new "return false" so that this works again, but I would suggest that preventing startup might be harsh.
comment:6 by , 18 years ago
How many windows users (home users, not developers) do you know who are aware that an application can be launched with command line parameters?
comment:7 by , 18 years ago
How many "windows users" are running mythbackend straight out of inittab?
My suggestion wasn't to replace the env var check, but to add extra functionality for the edge case.
comment:8 by , 18 years ago
Though, really, with a proper command line, the user can pass HOME (or any other environment variable) even when running directly out of inittab. I.e. as in Option 2 at http://www.mythtv.org/wiki/index.php/Frontend_Auto_Login (but without the need for a VT in the case of mythbackend, where the user can use either /bin/su - or /bin/sh -l to get a proper environment) or http://www.mythtv.org/wiki/index.php/Ubuntu_Dapper_Installation#Automatic_frontend_start.
comment:9 by , 18 years ago
Please back out the change that prevents startup if the HOME var is not found. Printing a warning is fine, but not preventing startup. This entire ticket is a non issue.
comment:10 by , 18 years ago
Anon - please state your reasons on the -dev list. We aren't going to respond to anonymous requests, especially when you don't give any reasons.
comment:11 by , 18 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Can we open a new ticket for the issue of setting HOME etc on other platforms? They aren't strictly related to this ticket.
comment:12 by , 18 years ago
Milestone: | unknown → 0.21 |
---|
Yes, if HOME isn't set, QDir::homeDirPath() seems to return '/'. The real question is, what should it be in this situation? Current directory? /tmp?