[mythtv] [mythtv-commits] Ticket #3302: Do not update mfdb min/max hour for MythFillGrabberSuggestsTime
Michael T. Dean
mtdean at thirdcontact.com
Fri Apr 13 15:26:21 UTC 2007
On 04/13/2007 10:27 AM, aaron wrote:
> On 13/04/07, Chris Pinkham <cpinkham at bc2va.org> wrote:
>
>> I was going to comment on this when I saw the thread and ticket opened, but
>> now don't have to as much. :) I agree that we should be resetting
>> MythFillMinHour every day to make sure that our suggested time will be
>> honored. It would be nice if we could get the min/max settings greyed out
>> in the settings page if MythFillGrabberSuggestsTime is turned on, but that's
>> not required to fix this bug.
I agree. I'll put that on my (growing much faster than my available
Myth-hacking time) list. :)
> I was just thinking along similar lines, that it might be nice if it
> could "ignore" the mix/max time if Suggests was set...
That would be very easy to do, too (basically delete one line as it's
already in a section of code specific to those using suggested run
time). Thoughts, Chris? This would obviate the need to even mess with
changing the min hour setting.
> but then
> thought that might be too convoluted to implement cleanly. Disabling
> the options (and automagically setting them to sane values, I assume)
> is better than my idea. :) And I agree: not needed right away.
>
Yeah. The automatically setting them part probably should be done, too,
if we don't remove the check against min/max hour. It wouldn't be hard
to do that while graying out the options in the settings screen. If we
leave the min/max check in there, I'll make sure it automatically sets
them, too.
> I'll hopefully get a chance to put the patch in place this weekend
The mythtv-DataDirect-no_update_min_max_hour-20070413.patch should apply
cleanly to 0.20-fixes, too.
> ...Note to self: disable the cron job I had set up to work around the
> issue... ;-)
Please do. :)
Mike
More information about the mythtv-dev
mailing list