Opened 6 years ago

Closed 13 days ago

#12312 closed Patch - Bug Fix (Won't Fix)

mythfilldatabase (SD/DD) does not honour $TMPDIR

Reported by: brian@… 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)

TMPDIR.patch (1.3 KB) - added by brian@… 6 years ago.
Use $TMPDIR in the SD grabber

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by brian@…

Attachment: TMPDIR.patch added

Use $TMPDIR in the SD grabber

comment:1 Changed 6 years ago by brian@…

No comment at all on this patch? Am I not following the correct process to submit a patch?

comment:2 Changed 6 years ago by gary.buhrmaster@…

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 6 years ago by stuartm

Owner: changed from stuartm to sphery
Status: newassigned
Summary: mythcommflag (SD/DD) does not honour $TMPDIRmythfilldatabase (SD/DD) does not honour $TMPDIR

comment:4 Changed 13 days ago by Gary Buhrmaster

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 13 days ago by gigem

Milestone: unknown31.1
Resolution: Won't Fix
Status: assignedclosed
Note: See TracTickets for help on using tickets.