Chris Adams

Results 315 comments of Chris Adams

@finoradin Thanks – I'll work on a pull request for this. Strong +1 on checksumming the source files — the way I typically do things like that would be streaming...

@runderwood +1 – that should cover the most common use-cases

@ntallman How do you use this — when bagging, updating existing bags, etc.? Since this is something of an edge case I'm wondering whether it should be a separate utility.

This sounds like it's related to https://github.com/LibraryOfCongress/bagit-python/pull/92 — the code originally was written assuming bag-in-place semantics but it would definitely be useful to have the ability to create a bag...

I'd take a look at #92 but if it's a significant amount of work to rebase that against master a new branch might be cleaner.

This is only true when running on Python 2. We could have a patch which switches to `collections.OrderedDict` on Python 2.7 but there's an argument which says we should just...

Indeed – I was also looking at a more comprehensive fix so we could also start supporting non-POSIX interfaces such as S3 in https://github.com/acdha/bagit-python/tree/flexible-fileio but I haven't worked on that...

That's certainly quite odd — does it reproduce if you simply run `python -c 'import os.path; os.path.expanduser("data/blah/~$_-expect_fail.doc")'`? Looking at the stdlib implementation, my first reaction is surprise that this isn't...

Ah, I missed that and it explains the problem: `$_` is a shell variable (last argument of the previous command, if memory serves), and that would explain why you're seeing...