[Enhancement] Make --test option to ebooks-convert take parameters specifying the number of feeds to download and number of items per feed to download

Bug #1246061 reported by TomasHnyk on 2013-10-29
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
calibre
Undecided
Unassigned

Bug Description

I am writing a new recipe for Calibre (I will submit it shortly). The use of --test switch is recommended here: http://manual.calibre-ebook.com/news.html#tips-for-developing-new-recipes

That is fine for initial testing, but sometimes I needed to download a bit more but preferably not all content. I had multiple feeds and olny the fourth contained images that I was trying to manipulate. As --test sets the number of feeds to be downloaded to 2, this feed never got downloaded and I had to download over 40 articles instead of 5, so testing the recipe took 2 minutes instead of 10 seconds.

It would be nice if --test took two optional arguments specifying the number of feeds to be downloaded and the number of items per each feed. If no argumet were given, it would work the same as before.

I am not convinced as to the use of this. If you are developing a recipe
anyway, why not just temporarily comment out all but the feed you are
interested in?

However, since adding this is harmless, it will be in the next release.

Fixed in branch master. The fix will be in the next release. calibre is usually released every Friday.

 status fixreleased

Changed in calibre:
status: New → Fix Released
TomasHnyk (sup) wrote :

Thanks a lot! (I am using parse_index, so commenting feeds out would involve manually checking for feeds name, so this is much more practical)

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers