Results 98 comments of josch

Maybe you'd like to add the testcases I submitted as a pull request to your repo: https://github.com/russdill/e2fsprogs/pull/1 These testcases are nearly 1:1 those that I wrote for #118 except that...

Thank you for your input @tytso! I added a commit to #118 that uses dlopen and I also pushed a commit that does the same for this pull request here:...

Use dlopen instead of linking against libarchive to keep runtime dependencies minimal as requested in https://github.com/tytso/e2fsprogs/pull/107#issuecomment-1193042420

Due to its current unmaintained state, pdfrw is now also flagged for removal from Debian https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=958362

@Alkaris That's what "remove the bashism from the script" means, yes. But that's not enough. There are other things like `${PARAMETER//[0-9]}` that would have to be replaced by something else.

> It's not a dependency of the library itself :) Yes it is. Read the `package.json`. Yes, this is probably a bug and the dependency should probably move to `devDependencies`...

The "easy" way is just to throw a big tarball at it. :smile: For example here is a big system image: https://mister-muffin.de/reform/target-userland-full.tar

Without it, handling a tarball in pax format containing utf8 characters in its filenames will result in this error message: archive_read_next_header(): Pathname can't be converted from UTF-8 to current locale.

Interesting. I'm a heavy user of the `-` output option and never saw this error. What OS are you running?

Oh I wasn't trying to shoot down your contribution. It looks sane to me (without having tested it myself).