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.

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

"Package size exceeded the configured limit of 150 MB" for dev269/dev271 builds #1838

Closed
holdenmatt opened this issue Sep 3, 2024 · 3 comments

Comments

@holdenmatt
Copy link

What happens?

(Apologies if you're already aware of this)

I was testing the latest dev build, and it looks like #1561 has come back since dev269.

To Reproduce

These links don't work:
https://cdn.jsdelivr.net/npm/@duckdb/duckdb-wasm@1.28.1-dev269.0/dist/duckdb-browser-eh.worker.js
https://cdn.jsdelivr.net/npm/@duckdb/duckdb-wasm@1.28.1-dev271.0/dist/duckdb-browser-eh.worker.js

The last working version:
https://cdn.jsdelivr.net/npm/@duckdb/duckdb-wasm@1.28.1-dev258.0/dist/duckdb-browser-eh.worker.js

Browser/Environment:

n/a

Device:

n/a

DuckDB-Wasm Version:

1.28.1-dev269.0

DuckDB-Wasm Deployment:

n/a

Full Name:

Matt Holden

Affiliation:

Independent developer (Capset Labs)

@carlopi
Copy link
Collaborator

carlopi commented Sep 3, 2024

Thanks, I am looking at a fix!

@tobilg
Copy link

tobilg commented Sep 26, 2024

The lastest few dev version are now at 135mb again...

@carlopi
Copy link
Collaborator

carlopi commented Sep 26, 2024

This has since been fixed, by reviewing compilation flags to actually exclude more unnecessary code (#1860) and cherry-picking some work from the SMALLER_BINARY duckdb feature (#1864).

After that check on total size has been reinstated (#1865), so we should keep track of this going forward.

@carlopi carlopi closed this as completed Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants