Use wget for downloading

[jrdnbradford]

Closes #85.

There are many ways this particular issue can be solved, here's one option.

Unfortunately this is currently a breaking change as many users may have been using this without having wget installed. Let me know if you think we should perhaps add some logic to allow continued use of non-wget methods.

[jonthegeek]

Until we get told to stop, I don't think we should do this. I wouldn't be able to use the package in my normal dev environment for one, and I'm guessing that also goes for the majority of our users. Is there any indication that this rule is anything more than an artifact from old instructions?

[jrdnbradford]

@jonthegeek I can see if I can get some clarity via email with them if you like. Then if the decision is to keep as is, we can just update the README.

[jonthegeek]

I'm not sure I even read their rules as saying you HAVE to use wget. It's giving it as an example of how to access the files, it seems to me.

[jrdnbradford]

Aha, I wonder if my reading is affected by the reference in the README

Project Gutenberg allows wget to harvest Project Gutenberg using this list of links. The gutenbergr package visits that page once to find the recommended mirror for the user’s location.

and usage of wget_url in the code. Might just be artifacts of a time long past.

[jonthegeek]

Aha! Yeah, I think we should instead make our reference a bit less specific.

[jrdnbradford]

Looking at Terms of Service and a mirror's robots.txt, I think you're right.

I will edit this PR!

[jrdnbradford]

Fixed up now. I removed the specific references to wget in code and docs. Apologies for getting antsy on this! Should have waited for discussion in #85. 🤓

[jonthegeek]

This worked out! Thanks for giving me a distraction that I could work through on my phone! (see https://bsky.app/profile/jonthegeek.com/post/3lau5bcqdss2t for why that's helpful today!)