Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#4212 closed defect (wontfix)

mythweb doesn't support multiple directories for mythvideo

Reported by: Ash Bostock <abostock@…> Owned by:
Priority: minor Milestone: unknown
Component: Plugin - MythWeb Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

The mythvideo part of mythweb will try to auto create a symlink in the data directory for mythvideo files. If you have defined multiple directories for mythvideo then this fails.

Attachments (3)

mythweb_video.patch​ (1.6 KB) - added by octoberblu3 11 years ago.
mythweb_video2.patch​ (3.4 KB) - added by octoberblu3 11 years ago.
mythweb_video.22.patch​ (3.3 KB) - added by octoberblu3 11 years ago.
updated patch for trunk

Download all attachments as: .zip

Change History (14)

comment:1 Changed 12 years ago by xris

Owner: changed from Isaac Richards to Rob Smith

comment:2 Changed 12 years ago by Rob Smith

Status: new → accepted

comment:3 Changed 12 years ago by Rob Smith

Component: mythtv → mythweb

comment:4 Changed 12 years ago by Rob Smith

Milestone: unknown → 0.22

comment:5 Changed 12 years ago by Damian O'Sullivan <damian@…>

Hi kormoc,

Have you any ideas about this one? I also have a : seperated list of video dirs.

Thanks.

Changed 11 years ago by octoberblu3

Attachment: mythweb_video.patch​ added

comment:6 Changed 11 years ago by octoberblu3

Hi, I had this same problem. My work around was to manually create symlinks to the various directories and then use the patch I just uploaded to modify the Video.php file in the objects directory to handle the colon delimited list properly.

Just name the symlinks video, video2, video3 and so on in the same order as they appear in the settings.

I am using this on 0.21 fixes, current as of today (svn 19785).

Changed 11 years ago by octoberblu3

Attachment: mythweb_video2.patch​ added

comment:7 Changed 11 years ago by octoberblu3

I added the other patch that will create the multiple symlinks. It works on my system, so I hope this helps others as well.

comment:8 Changed 11 years ago by Rob Smith

Component: mythweb → Plugin - MythWeb
Owner: Rob Smith deleted
Status: accepted → new

comment:9 Changed 11 years ago by stuartm

Milestone: 0.22 → unknown

comment:10 Changed 11 years ago by Rob Smith

Resolution: → wontfix
Status: new → closed

With the migration of mythvideo to storage groups, we won't be needing this in the future.

Changed 11 years ago by octoberblu3

Attachment: mythweb_video.22.patch​ added

updated patch for trunk

comment:11 Changed 11 years ago by octoberblu3

Since I recently migrated to trunk, I updated my patch to work with 0.22. It seems the mythvideo storage groups won't make it until 0.23 (#4623), so others may need this patch in the meantime.

Note: See TracTickets for help on using tickets.