Opened 12 years ago

Closed 12 years ago

#3938 closed patch (fixed)

Add ability to have grabber sub-dirs in MythWeather scripts directory

Reported by: Joe Ripley <vitaminjoe@…> Owned by: stuartm
Priority: minor Milestone: 0.21
Component: mythweather Version: head
Severity: low Keywords: grabber
Cc: stuartm Ticket locked: yes

Description

This patch allows the MythWeather? plugin to store its grabber scripts in subdirectories instead of all grabbers in one directory.

The new code recurses through the mythweather/scripts directory and finds any executable files it can. Any number of directory levels are possible.

Attachments (1)

mythweather.patch (4.0 KB) - added by Joe Ripley <vitaminjoe@…> 12 years ago.
Add ability to organize grabber scripts and related files into subdirectories of MythWeather? /usr/{local/}share/scripts directory

Download all attachments as: .zip

Change History (6)

Changed 12 years ago by Joe Ripley <vitaminjoe@…>

Attachment: mythweather.patch added

Add ability to organize grabber scripts and related files into subdirectories of MythWeather? /usr/{local/}share/scripts directory

comment:1 Changed 12 years ago by Joe Ripley <vitaminjoe@…>

I wasn't sure how to display the progress of the script search due to the nature of the search algorithm (i.e. recursive function). Ideas are welcome to improve the UI experience of waiting for MythWeather? to search for scripts. :)

comment:2 Changed 12 years ago by stuartm

Milestone: unknown0.21
Owner: changed from ldunning to stuartm

comment:3 Changed 12 years ago by stuartm

Resolution: fixed
Status: newclosed

(In [14994]) Allow mythweather grabbers to be kept in sub directories of the script directory. This should allow us to keep things a little tidier.

Patch by Joe Ripley.

Closes #3938

comment:4 Changed 12 years ago by anonymous

Resolution: fixed
Status: closedreopened

this one just broke my daily svn build: the MythProgressDialog? was renamed from busyd to busy, but in line 115/116 the two old calls survived:

busyd->Close(); busyd->deleteLater();

comment:5 Changed 12 years ago by stuartm

Resolution: fixed
Status: reopenedclosed
Ticket locked: set

I fix that problem 8 hours ago.

Note: See TracTickets for help on using tickets.