Opened 9 years ago

Closed 8 years ago

#9648 closed Patch - Bug Fix (fixed)

win32 pathspec local file support

Reported by: Mark Spieth Owned by: beirdo
Priority: minor Milestone: 0.25
Component: Ports - Windows Version: Unspecified
Severity: medium Keywords:
Cc: Ticket locked: no

Description

currently for Win32 builds, local files cant be specified with drive letters for both recording paths and video paths.

patch attached.

Attachments (1)

26-win32-path (2.7 KB) - added by Mark Spieth 9 years ago.

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by Mark Spieth

Attachment: 26-win32-path added

comment:1 Changed 9 years ago by robertm

Resolution: Won't Fix
Status: newclosed

Hi Mark,

Local file paths are deprecated and will likely be removed entirely by .25 (or at the latest, .26). I'm not comfortable applying this patch for that reason as it gives the mistaken impression that we are expanding support for this option, rather than removing it entirely in the near future. Thank you very much for the patch regardless.

comment:2 Changed 9 years ago by robertm

Resolution: Won't Fix
Status: closednew

Reopening as there's some code here that affects more than just MythVideo? (My bad), but note that the video scanner segment minimally should not be applied by whomever decides to look at the rest of this.

comment:3 Changed 8 years ago by beirdo

Owner: changed from Nigel to beirdo
Status: newaccepted

comment:4 Changed 8 years ago by Github

Milestone: unknown0.25
Resolution: fixed
Status: acceptedclosed

Absolute path testing fixups for win32 support

Currently for Win32 builds, local files can't be specified with drive letters for both recording paths and command execution paths.

Removed the video scanner portions at the request of Robert McNamara?.

Closes #9648

Signed-off-by: Mark Spieth <mspieth@…> Signed-off-by: Gavin Hurlbut <ghurlbut@…>

Branch: master Changeset: 5900d187af4743918993ebdfcb5b9d42971d38ca

Note: See TracTickets for help on using tickets.