search

manolomartinez / greg

A command-line podcast aggregator
GNU General Public License v3.0
299 stars 37 forks source link

Fix data storage format #95

Closed timwhite closed 3 years ago

timwhite commented 5 years ago

Current data storage format is prone to breakages when filenames have spaces in them. Instead of relying on delimiters and splitting, lets use JSON to store the data in a safe form. This pull requests maintains compatibility with the old file format, and handles any read errors so even if a broken line was written to a data file in the past, it won't prevent us continuing (at the cost that it may redownload that episode)

manolomartinez commented 5 years ago

Hey, this is a good idea, and I am willing to merge something along these lines. Right now, we would also need to modify greg.aux_functions.get_date(), which relies on the old format, and is used when editing the feed with --downloadfrom. I need to think if there are other similar corner cases.

manolomartinez commented 3 years ago

Finally (almost two years after the fact) I got around to finishing this. Thanks a lot, Tim Manolo