Economist (payed) update

[Movitz]

Hi,
Im sorry if this has allready come up, i have tried to search for it first. I only just downloaded calibre yesterday and it is fantastic.

When trying to download the payed version of economist though the meter froze after 34%. I made a small change to the receipe and succeded to download the paper. The changes are due to that the ""Classified ads" name is not under a H1 tag but an H2 one.

Now:
def economist_parse_index(self):
...
if 'Classified ads' in text:
break
if tag.name == 'h1':
if 'The world this week' in text or 'The world this year' in text:
index_started = True
if not index_started:
continue

before:

def economist_parse_index(self):
...
if tag.name == 'h1':
if 'The world this week' in text or 'The world this year' in text:
index_started = True
if 'Classified ads' in text:
break
if not index_started:
continue

How do i contribute these kind of changes in a more effecient way, i guess this board isnt the ultimate way of doing so? Is there a svn server i can download from and create patches?

[kovidgoyal]

Instructions for setting up a calibre development environment are here

http://calibre-ebook.com/user_manual/develop.html

[Movitz]

I just downloaded the source and found that you had put in the required changes to the receipe. Next time i will hopefully be able to post a ticket with a patch (or merge as it seems to be called in Bazaar) directly to make it easier.

Thanks again for some great software.

[DoctorOhh]

Originally Posted by Movitz

Im sorry if this has already come up, i have tried to search for it first. I only just downloaded calibre yesterday and it is fantastic.

For future info

Requests like this are best made in the Sticky thread Custom Recipes.