Conversation
waybackpack/pack.py
Outdated
|
|
||
| filepath = os.path.join(filedir, path_tail) | ||
| filepath = os.path.join(filedir, | ||
| ','.join((path_tail, asset.timestamp))) |
There was a problem hiding this comment.
Most of these change to the path are because I found it more convenient, for my own purposes, to have different versions of the same resource name in the same file. I can make a version that keeps your current, separate directory, behavior.
There was a problem hiding this comment.
Ah, interesting. I'd prefer to keep the current, separate-directory behavior. But I'm curious: What makes the other way more convenient for your purposes?
There was a problem hiding this comment.
p.s., Thanks! This seems to be a really elegant solution.
|
K, this should be good to merge. |
|
Hi @fgregg and apologies for the radio silence. I actually do like this approach! I've been working on a new version that integrates the bulk of your PR (with a few tweaks), but have gotten distracted by other things. Thanks for the nudge! |
|
@jsvine Any updates on this?
Any updates on this? Would you like it implemented in a different way? |
|
Hi @Joey-Einerhand, unfortunately there are no major updates on this. I ran into some operating-system-appeasing issues, got stumped, and haven't fully revisited. |
|
Hey @jsvine, would love to help get this PR over the line if I can. I wonder if some of those OS issues are no longer factors with the passage of time / newer Python versions? I'm using a version of the repo checked out at the PR, but at this point it's missing some nice-to-have features like |
|
@tomcardoso Many thanks for the offer and the enthusiasm! I'm open to merging this PR / something like it, with the main caveat that I'd prefer it not be the default behavior. I.e., it would require a flag like My main reason for that is that the issues with case-insensitivity on some OSes, including MacOS, remain. But the documentation for Would you be up for resolving the merge conflicts and adding that warning? |
|
@jsvine makes sense to me. Do you have a preference on globbing/no globbing? Not sure how annoying that is to implement, but I think it would be valuable. (I'm also not even sure the Archive.org server would support that kind of behaviour easily – I'll need to read the docs.) Re: casing, what was the issue originally? That some OSes are case sensitive and others aren't, so the functionality would have to default to case-insensitive? I think I'd prefer case-insensitivity be the default behaviour anyway, so it's no big deal. I'll take a stab over the next week or two. Evenings and weekends, you know how it goes! |
|
Hiya, and apologies for the delay in answering these questions:
If globbing is possible, I agree that it'd be a handy feature.
The situation in this comment attempts to illustrate it: #8 (comment) But, in short, the issue is that some OSes don't distinguish between differently-capitalized paths. E.g., |
This PR enables wildcard url fetches from wayback machine.