Modify ↓
Ticket #10810 (closed Patch - Bug Fix: fixed)
Opened 13 months ago
Last modified 13 months ago
Commercial flagging on remote host
| Reported by: | fedora@… | Owned by: | mdean |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.25.2 |
| Component: | MythTV - Mythcommflag | Version: | 0.25-fixes |
| Severity: | low | Keywords: | |
| Cc: | Ticket locked: | no |
Description
I noticed that when mythcommflag is run on a host that does not have file access to the recording, it exists reporting that it is unable to find the recording. The following patch seems to have fixed the issue for me:
diff --git a/mythtv/programs/mythcommflag/main.cpp b/mythtv/programs/mythcommflag/main.cpp
index 74d161f..c9fb56a 100644
--- a/mythtv/programs/mythcommflag/main.cpp
+++ b/mythtv/programs/mythcommflag/main.cpp
@@ -627,12 +627,7 @@ static qint64 GetFileSize(ProgramInfo *program_info)
if (filename.startsWith("myth://"))
{
RemoteFile remotefile(filename, false, false, 0);
- struct stat filestat;
-
- if (remotefile.Exists(filename, &filestat))
- {
- size = filestat.st_size;
- }
+ size = remotefile.GetFileSize();
}
else
{
Attachments
Change History
comment:1 Changed 13 months ago by fedora@…
comment:2 Changed 13 months ago by mdean
- Owner changed from cpinkham to mdean
- Status changed from new to accepted
comment:3 Changed 13 months ago by Michael T. Dean <mdean@…>
- Status changed from accepted to closed
- Resolution set to fixed
comment:4 Changed 13 months ago by Michael T. Dean <mdean@…>
Note: See
TracTickets for help on using
tickets.

One minor note, without modifying the code mythcommflag does work when run remotely for recordings stored in a master backend storage group. I think this is because the static "Exists" method queries the master backend for the file size.