Opened 9 years ago
Closed 4 years ago
#12312 closed Patch - Bug Fix (Won't Fix)
mythfilldatabase (SD/DD) does not honour $TMPDIR
| Reported by: | Owned by: | sphery | |
|---|---|---|---|
| Priority: | minor | Milestone: | 31.1 |
| Component: | MythTV - Mythfilldatabase | Version: | 0.27.4 |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
The Schedules Direct grabber does not honour (i.e. use) $TMPDIR if it's set.
I will attach a patch that I *think* should do the trick. I only very very occasionally find myself C++ programming so it might be quite right. Constructive criticism welcomed.
Attachments (1)
Change History (6)
Changed 9 years ago by
| Attachment: | TMPDIR.patch added |
|---|
comment:1 Changed 9 years ago by
No comment at all on this patch? Am I not following the correct process to submit a patch?
comment:2 Changed 9 years ago by
btw, while I am not a dev, from a very cursory look, this patch does not appear to properly support Windows, which uses different environmental variables, and seems to override a mutable which may be (and is?) changed by other programs. And, does not address the explicit attempt to not rm the /tmp directory. Looks like more existing code evaluation work is needed to me before it is ready.
comment:3 Changed 9 years ago by
| Owner: | changed from stuartm to sphery |
|---|---|
| Status: | new → assigned |
| Summary: | mythcommflag (SD/DD) does not honour $TMPDIR → mythfilldatabase (SD/DD) does not honour $TMPDIR |
comment:4 Changed 4 years ago by
I think this ticket can be closed (as "Won't Fix"?) The proposed patch was for the legacy datadirect processor, which was removed in the latest MythTV version.
comment:5 Changed 4 years ago by
| Milestone: | unknown → 31.1 |
|---|---|
| Resolution: | → Won't Fix |
| Status: | assigned → closed |
Use $TMPDIR in the SD grabber