Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stuffs... #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Stuffs... #1

wants to merge 4 commits into from

Conversation

jhnphm
Copy link

@jhnphm jhnphm commented Dec 28, 2012

I cheated and had to lookup how to use sed backreferences :(

My grep and then a sed isn't too elegant though, see if you can replace that w/ a single sed script. I also don't unescape entities.

I see why you're using Lynx- it renders the page rather than getting raw HTML. Hm, I didn't know that.

Since grabbing alt-text is desirable, and this is easier to do w/ raw HTML, replaced lynx w/ wget.

Also, may be of interests http://www.commandlinefu.com/commands/browse

@nelsonam
Copy link
Owner

you're welcome to use that in your branch I really would feel better keeping lynx for now though

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants