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

Jump to bottom

Aborted in WebContainer #2

Open
d3lm opened this issue Feb 23, 2024 · 5 comments
Open

Aborted in WebContainer #2

d3lm opened this issue Feb 23, 2024 · 5 comments

Comments

@d3lm
Copy link
Contributor

d3lm commented Feb 23, 2024

I am trying to run this in WebContainer and now that #1 was merged and released I am seeing the following error:

RuntimeError: Aborted(). Build with -sASSERTIONS for more info.
    at abort (file:///home/projects/node-oqtitn/node_modules/@electric-sql/pglite/dist/chunk-RM7SEAOI.js:412:15)
    at _abort (file:///home/projects/node-oqtitn/node_modules/@electric-sql/pglite/dist/chunk-RM7SEAOI.js:5626:7)
    at wasm://wasm/02d347d2:wasm-function[123]:0x77e6
    at wasm://wasm/02d347d2:wasm-function[6809]:0x6ac11a
    at wasm://wasm/02d347d2:wasm-function[4366]:0x437a87
    at wasm://wasm/02d347d2:wasm-function[14076]:0x9a2381
    at wasm://wasm/02d347d2:wasm-function[15480]:0x9de758

But only if there's a db folder. If you remove the folder then it works fine.

Any idea? I think it'd be nice to have a debug build as well where -sASSERTIONS is enabled. This could maybe be published under a specific debug tag?
@samwillis
Copy link
Collaborator

Hey @d3lm,

I've published a debug build with -s ASSERTIONS=1 although I'm unsure is it's giving us any more detailed errors.
https://www.npmjs.com/package/@electric-sql/pglite/v/0.0.1-debug.2

There is also the ASSERTIONS=2 and SAFE_HEAP=1 options we can try.

@d3lm
Copy link
Contributor Author

d3lm commented Mar 1, 2024

Thanks!

That's strange, cause it's still not giving more info and prints the same error 🤔

@vBuravlev
Copy link

Confirm. I couldn't run PG lite in any of the options. This repository is 100% erroneous code.

@Yaromey
Copy link

Yaromey commented Mar 24, 2024

Confirm. I couldn't run PG lite in any of the options. This repository is 100% erroneous code.

Same here. On Windows machine. Do you also use Windows?

@samwillis
Copy link
Collaborator

Same here. On Windows machine. Do you also use Windows?

Hey @Yaromey, there's a bug on Windows #34, should be fixed this week!

Mike-FreeAI added a commit to Mike-FreeAI/pglite that referenced this issue Aug 13, 2024
@Mike-FreeAI
Add mobile support electric-sql#2
48a62f2 
Add support for mobile filesystem using Expo FileSystem API.

* **Add MobileFS class:**
  * Create `mobilefs.ts` file in `packages/pglite/src/fs/`.
  * Implement `MobileFS` class extending `FilesystemBase`.
  * Implement methods for reading, writing, and deleting files using Expo FileSystem API.

* **Update fs index:**
  * Import `MobileFS` class in `packages/pglite/src/fs/index.ts`.
  * Add case for `mobilefs` in `parseDataDir` function.
  * Add case for `mobilefs` in `loadFs` function.

* **Add tests for MobileFS:**
  * Create `mobile-fs.test.js` in `packages/pglite/tests/targets/`.
  * Write tests for reading, writing, and deleting files using MobileFS and Expo FileSystem API.
Mike-FreeAI added a commit to Mike-FreeAI/pglite that referenced this issue Aug 13, 2024
@Mike-FreeAI
Merge pull request #1 from Mike-FreeAI/add-mobile-support-1
b612f44 
Add mobile support electric-sql#2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@samwillis @d3lm @vBuravlev @Yaromey