Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: download browsers as TAR #34033

Open
wants to merge 34 commits into
base: main
Choose a base branch
from
Open

feat: download browsers as TAR #34033

Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
a9b40bd
add second branch
Skn0tt Dec 16, 2024
0b139f1
move chromium, chromium-tot, and webkit to brotli
Skn0tt Dec 16, 2024
98d6138
fix import
Skn0tt Dec 16, 2024
657e435
feat: download browsers as .tar.br
Skn0tt Dec 16, 2024
c5ab0c5
respect revisionOverrides
Skn0tt Dec 18, 2024
3b72c3b
update thirdparty
Skn0tt Dec 18, 2024
411cf29
update log line matcher
Skn0tt Dec 18, 2024
d880a22
update expected error codes
Skn0tt Dec 18, 2024
f39ae50
ffmpeg also has brotli builds now
Skn0tt Dec 18, 2024
f071337
remove conflicting tar-fs types
Skn0tt Dec 18, 2024
45b84b7
add workaround for manually-created ffmpeg archive
Skn0tt Dec 18, 2024
a497b6b
remove console statement
Skn0tt Dec 18, 2024
b048d2a
Merge branch 'main' into tar-download-3rd-party-lib
Skn0tt Dec 18, 2024
affff4d
Merge branch 'main' into tar-download-3rd-party-lib
Skn0tt Dec 19, 2024
879afb1
adapt to new ffmpeg roll
Skn0tt Dec 19, 2024
16ff2df
vendor modified tar-fs
Skn0tt Jan 6, 2025
eaf6a52
update thirdpartynotices
Skn0tt Jan 6, 2025
220e28b
reset package-lock
Skn0tt Jan 6, 2025
0ccbef1
reset package.json
Skn0tt Jan 6, 2025
30c5ec6
Merge branch 'main' into tar-download-3rd-party-lib
Skn0tt Jan 13, 2025
e2b1500
firefox and chrome-headless-shell have tar now
Skn0tt Jan 13, 2025
25494cb
reuse existing code better
Skn0tt Jan 13, 2025
2224fd2
continue using close
Skn0tt Jan 13, 2025
41a552f
move to third_party
Skn0tt Jan 13, 2025
dce95a5
add license/readme
Skn0tt Jan 13, 2025
2749367
remove getStreamError
Skn0tt Jan 13, 2025
7744cc5
add diff
Skn0tt Jan 13, 2025
c717419
fix _predestroy
Skn0tt Jan 13, 2025
a2d214f
add notices
Skn0tt Jan 13, 2025
5086a56
fix linter
Skn0tt Jan 13, 2025
e24a0d7
add deps
Skn0tt Jan 13, 2025
746bb68
Merge branch 'main' into tar-download-3rd-party-lib
Skn0tt Jan 21, 2025
c8b2ce0
revert test changes
Skn0tt Jan 21, 2025
6fb5355
fix test
Skn0tt Jan 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1,586 changes: 1,585 additions & 1 deletion packages/playwright-core/ThirdPartyNotices.txt
View file
Open in desktop

Large diffs are not rendered by default.

209 changes: 209 additions & 0 deletions packages/playwright-core/bundles/zip/package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions packages/playwright-core/bundles/zip/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
},
"dependencies": {
"extract-zip": "2.0.1",
"tar-fs": "^3.0.6",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I understand the library is popular, but its deps list seem to be excessive for what it does a little. Did we consider alternatives?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh wow, "tar" is even more...

Copy link
Member Author

@Skn0tt Skn0tt Dec 19, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, we considered tar-fs, tar and writing our own. Writing our own turned out more complex than imagined, because webkit has very long path names and the format becomes tricky when that's involved. Of the three, tar-fs seemed the most focused.

"yauzl": "2.10.0",
"yazl": "2.5.1"
},
Expand Down
2 changes: 2 additions & 0 deletions packages/playwright-core/bundles/zip/src/zipBundleImpl.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,3 +18,5 @@ export * as yazl from 'yazl';
export * as yauzl from 'yauzl';
import extractZip from 'extract-zip';
export const extract = extractZip;
// @ts-expect-error @types/tar-fs is broken and we're fine without it
export * as tarFs from 'tar-fs';
Loading
Loading