> It sounds to me like the easiest way to make the current behavior
> match the old behavior for recording-failed HDTV (from an interface
> point of view) would be to have it write a 0 byte file if the reception
> check fails. In 0.16, nothing would display, but the recording would be
> deletable and cancellable, which is at least functional.

Ideally, the recording should not even show up, so if there is a problem
recording, then it should be as if the recording never started.  There
should be nothing in the database to clean up.

> Does anyone out there who knows the HD reception/recording code care to
> take a stab at a quick hack? I may have some time to look this weekend,
> though I also need to debug why 0.17 broke recording things that get
> reception...

This is one solution, but long-term I think what I mention above is better.

The reason why the delete code was changed was that there have been quite
a few times where user's video directories were either unavailable, not
nfs-mounted, etc. and since their root drive didn't have enough free space
according to their auto-expire settings, Myth happily went through all
their programs and deleted them from the database since.  Since there
were no files to delete, deleting the first recording didn't free up enough
space, so the auto-expire went onto the next recording, and so on.  You
could argue that the auto-expire code shouldn't do this, but trying to
code auto-expire to handle all the things a user could do to make a
recording non-deletable at the OS level would be a major pain, so the
delete code was changed to not delete the database info for a recording
until it verified the file was actually there and that it was actually
deleted.  I'd rather have a few people touching files than another
bunch of users with their whole recordings table wiped out. :)

Just wanted to explain the reasoning behind the changes.  Data integity.
Don't delete the DB info if you can't delete the recorded file that goes
along with the DB info.

-- 

Chris

_______________________________________________
mythtv-users mailing list
[email protected]
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-users

Reply via email to