search

mirage / ocaml-tar

Pure OCaml library to read and write tar files
ISC License
54 stars 34 forks source link

Missing padding in LongLink 'L' case #82

Closed dra27 closed 2 years ago

dra27 commented 3 years ago

Unfortunately, in #25, the re-writing of https://github.com/mirage/ocaml-tar/pull/25/files#diff-437e75b698404667f7caa6812dc0d41bc2aad6098bfb5afe7957ab1f95d26db5L449-L452 to https://github.com/mirage/ocaml-tar/pull/25/files#diff-437e75b698404667f7caa6812dc0d41bc2aad6098bfb5afe7957ab1f95d26db5R659 to factor the common code omitted the padding for 'L' case. It's present in the 'K' case.

MisterDA commented 2 years ago

ping @dinosaure

dra27 commented 2 years ago

ocaml-ci failure disappears with #84

dinosaure commented 2 years ago

Thanks!