deno_std icon indicating copy to clipboard operation
deno_std copied to clipboard

feat: web streams based archive/tar

Open crowlKats opened this issue 2 years ago • 13 comments

still WIP

closes #1658

crowlKats avatar Mar 03 '22 06:03 crowlKats

@crowlKats there's already https://github.com/denoland/deno_std/pull/1855. Should one of the PRs be closed?

bartlomieju avatar Mar 08 '22 01:03 bartlomieju

@bartlomieju well, the approach of the other PR wont work, and this PR right now is stuck due to upstream spec change needed

crowlKats avatar Mar 08 '22 01:03 crowlKats

due to upstream spec change needed

I'm curious if there is a reference on what is needed / where the conversation is?

Also as an outsider it seems that the filePath shorthand parameter will not be available in the new API to allow removing all Deno.* dependencies, is that right?

Cross link:

  • #1658

danopia avatar Mar 23 '22 13:03 danopia

@danopia https://github.com/whatwg/streams/pull/1145, and yes, filePath wont be available

crowlKats avatar Mar 23 '22 13:03 crowlKats

Still blocked by spec update

bartlomieju avatar May 26 '22 11:05 bartlomieju

@crowlKats, are there any possible workarounds for this PR? Perhaps, some alternative API that's come up since the last update or a WASM alternative? Disclaimer: I'm unfamiliar with this implementation.

iuioiua avatar Dec 21 '22 20:12 iuioiua

@iuioiua I sadly believe no

crowlKats avatar Dec 22 '22 01:12 crowlKats

The https://github.com/whatwg/streams/pull/1145 PR has now been merged. Is this PR now unblocked, or does this require V8/Deno updates before STD can be updated?

csvn avatar Nov 15 '23 09:11 csvn

Nevermind, sorry. Saw there is work on this in https://github.com/denoland/deno/pull/20849.

csvn avatar Nov 15 '23 09:11 csvn

yep, once that PR lands in Deno, I'll work on this PR again

crowlKats avatar Nov 15 '23 22:11 crowlKats

Update: the deflateTarArchive test is failing because the TarEntry.#readableInner is locked when executing this line. Any insight would be welcome.

iuioiua avatar Dec 08 '23 03:12 iuioiua

Blocked on #3982

crowlKats avatar Dec 19 '23 10:12 crowlKats

@crowlKats, are you able to complete documentation so deno doc --lint archive/tar_stream.ts and deno doc --lint archive/untar_stream.ts pass?

iuioiua avatar Dec 20 '23 06:12 iuioiua