<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">I don't have the time to do it at the moment, but I'll work on it ASAP :)<br><br></div><div class="gmail_quote">In particular:<br></div><div class="gmail_quote">

<br>On Wed, Jan 15, 2014 at 3:50 AM, Yan Zhu <span dir="ltr"><<a href="mailto:yan@eff.org" target="_blank">yan@eff.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

I'd suggest checking out Python's subprocess module; you can use<br>
subprocess.call or subprocess.Popen to make simple shell commands<br>
(like 'git diff') from within a Python script. As an extra<br>
convenience, you might also want to automatically download the Alexa<br>
Top 1M csv file from the canonical source so that people don't have to<br>
look up the URL. The urllib module or just opening a wget subprocess<br>
would work.<br>
<br></blockquote><div><br></div><div>I'll check out both of them! I thought about doing so, but because it was my first attempt, I wanted to keep it as simple as possible :)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


For unzipping, python has a built-in zip module too.<br></blockquote><div><br></div><div>Will do!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


Also, you should probably close files after you open them once you're<br>
done. :)<br></blockquote><div><br></div><div>Whoops, didn't  think about that (lol) I'll fix it<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


<br>
PS: Do you use git? It makes code review like this much easier, as<br>
well as tracking changes. We use it pretty extensively in HTTPS<br>
Everywhere, so now would be a good time to learn!<br>
<br></blockquote><div><br></div><div>I do, actually, but because it was just a simple file and I was reeeally tired I didn't want to set everything up :P<br></div><div>I forked HTTPS Everywhere on GitHub, added my "merger.py" to the utils/ folder and renamed it to "alexa-ruleset-checker.py"<br>

<br></div><div>You can find everything here: <a href="https://github.com/flyingstar16/https-everywhere">https://github.com/flyingstar16/https-everywhere</a><br><br></div><div>It's _not_ working, meaning that it's the same script with a different name and location, but as I said I'll work on it as soon as I get the time :)<br>

</div><div> </div>Cheers,<br><br>Claudio<br></div></div></div>