Your README is incorrect #18

Merged
merged 1 commit into from May 13, 2011

Conversation

Projects
None yet
3 participants
@tubbo

tubbo commented May 12, 2011

Hi there!

First of all, I really love this tool. It's really been useful for testing out PCRE syntax, since I'm always so fuzzy with regex. However, I noticed that your README was a little out-of-date, as sudo easy_install sqlite3 doesn't seem to work ever. I have phpsh running on Python 2.6.1 on OS X 10.6, and I'm running pysqlite. So I updated your README to inform users that pysqlite is part of Python 2.6's stdlib, so you shouldn't need to easy_install it, and I removed the bit about sqlite3 since it doesn't seem to be relevant (anymore?).

Once again, thank you so much for creating this tool, it's been a major help.

aran added a commit that referenced this pull request May 13, 2011

Merge pull request #18 from tubbo/master
Update readme to eliminate unnecessary sqlite3.

@aran aran merged commit e839d58 into facebookarchive:master May 13, 2011

@aran

This comment has been minimized.

Show comment Hide comment
@aran

aran May 13, 2011

Contributor

I'm basically trusting that you're right; thanks for the fix :)

Contributor

aran commented May 13, 2011

I'm basically trusting that you're right; thanks for the fix :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment