Opened 14 years ago
Closed 14 years ago
#7136 closed patch (fixed)
PreviewGenerator when no backend binary
Reported by: | Nigel | Owned by: | danielk |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | MythTV - General | Version: | head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
If an install has no backend binary, going into Watch Recordings can output lots of errors like:
2009-09-24 20:54:54.246 Preview Error: Encountered problems running '/Volumes/MythBuild/.osx-packager/src/myth-svn/mythtv/programs/Resources/bin/mythbackend --generate-preview 0x0 --chanid 1004 --starttime 20080206182700 ' sh: line 1: /Volumes/MythBuild/.osx-packager/src/myth-svn/mythtv/programs/Resources/bin/mythbackend: No such file or directory 2009-09-24 20:54:54.276 Preview Error: Encountered problems running '/Volumes/MythBuild/.osx-packager/src/myth-svn/mythtv/programs/Resources/bin/mythbackend --generate-preview 0x0 --chanid 1039 --starttime 20071029102347 ' sh: line 1: /Volumes/MythBuild/.osx-packager/src/myth-svn/mythtv/programs/Resources/bin/mythbackend: No such file or directory
(Mac frontend-only packages only have mythfrontend and mtd). This patch checks for the binary before trying to invoke it:
Index: previewgenerator.cpp =================================================================== --- previewgenerator.cpp (revision 22017) +++ previewgenerator.cpp (working copy) @@ -223,8 +223,12 @@ else { // This is where we fork and run mythbackend to actually make preview - QString command = GetInstallPrefix() + - "/bin/mythbackend --generate-preview "; + QString command = GetInstallPrefix() + "/bin/mythbackend"; + + if (!QFileInfo(command).exists()) + return false; + + command += " --generate-preview "; command += QString("%1x%2") .arg(outSize.width()).arg(outSize.height()); if (captureTime >= 0)
Attachments (2)
Change History (5)
comment:1 Changed 14 years ago by
Owner: | changed from Isaac Richards to danielk |
---|---|
Status: | new → assigned |
Changed 14 years ago by
Attachment: | mythtv-7136-remove_GeneratePreviewRemotely_setting.patch added |
---|
comment:2 Changed 14 years ago by
The attached patch, mythtv-7136-remove_GeneratePreviewRemotely_setting.patch , removes the "Generate preview image remotely" setting and does all preview generation from the backend process. It does not remove the preview generator's ability to be used for local versus remote generation--it just makes the playbackbox use the remote generator.
The recent improvements to UI responsiveness before previews are created and during preview generation make the amount of time it takes to generate a preview less important. Furthermore, since the local preview generation was only used for the occasional changed recording (after setting a bookmark or cutlist or watched flag or ...) and for recordings in progress, the setting was mostly unused. And, finally, since the days of the hugely-powerful frontend system with underpowered backend systems seem to be over and many users are actually taking the opposite approach (and creating powerful backends with low-powered frontends), this configuration should work well for everyone--even users who don't install mythbackend on dedicated frontend systems (or who are working on platforms to which mythbackend has not been ported).
Even with my ca. 2002 Athlon XP 2000+ system generating previews for HDTV broadcasts (which it can not decode in real time), the preview generation seems sufficiently fast. Therefore, rather than test for a local mythbackend binary before calling it, or even changing the defaults to enable the "Generate preview image remotely" setting, it makes more sense to just remove the setting completely.
Nigel: Until the setting is removed, you can simply enable the "Generate preview image remotely" setting in Utilities/Setup?|Setup|TV Settings|Playback.
Changed 14 years ago by
Attachment: | mythtv-7136-remove_GeneratePreviewRemotely_setting.2.patch added |
---|
Updated patch for playbackboxhelper changes
Removes the "Generate preview image remotely" and does all preview generation from the backend process.