On 14/07/18 14:31, Ralph Corderoy wrote:

get_iplayer --refresh-exclude-groups-radio national,regional,local
--refresh-include "BBC Radio 4" -f --type radio

It added 1840 radio programmes to the cache.  As far as I could see
from a visual inspection scrolling through the newly created
radio.cache file all the programmes were from Radio 4, Radio 4 Extra
and the World Service when joined with Radio 4.

`--refresh-include' takes a comma-separated list of case-insensitive
regexps, from looking at the `channels_filtered' subroutine,
so «--refresh-include '^BBC Radio 4$'» should cut out the `Extra'.
(The regexp is quoted for Unix.)

There should be a `INFO: Will refresh channel...' verbose log message
for each channel that makes it through.

Hi Ralph

Taking your last point first, the verbose messages are not very helpful because they are accompanied by a lot of messages about the schedule pages. Curiously for --refresh-include "radio 4" I get

INFO: Will refresh channel BBC Radio 4
INFO: Will refresh channel BBC Radio 4
INFO: Will refresh channel BBC Radio 4 Extra

Are you saying the onus is on the user to insert ^ and $ in a regex to suppress greedy matching? In the case of a search argument the documentation requires a regex. There is no similar requirement for --refresh-include. My view is that if the programmer chooses to use a regex there it is up to him or her to construct the regex.

My first attempt was --refresh-include radio4. Nothing was found. I concluded I had to match the channel name exactly as "BBC Radio 4". I was therefore surprised to have "BBC Radio 4 Extra" matched as well. As I have said above, --refresh-include "radio 4" does work while --refresh-include "radio4" does not. The regex needs to be further modified to allow zero or more spaces between radio and 4.

These are fairly minor issues. A more important question is do you agree that in order for --refresh-include to work it has to be accompanied by --refresh-exclude-groups national,regional,local?

Best wishes
Richard



_______________________________________________
get_iplayer mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/get_iplayer

Reply via email to