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

Prisma might have renamed their query engines? #4

Open
RoccoFarrell opened this issue Dec 5, 2023 · 2 comments
Open

Prisma might have renamed their query engines? #4

RoccoFarrell opened this issue Dec 5, 2023 · 2 comments

Comments

@RoccoFarrell
Copy link

Line 52 of the generate.js script has

const engineFile = (0, fs_1.readdirSync)(clientPath).find((f) => f.startsWith('libquery_engine-'));

where clientPath is .\node_modules\@prismany\client\base_schema

However the copied engine is named query_engine-windows.dll.node

Changing line 52 to const engineFile = (0, fs_1.readdirSync)(clientPath).find((f) => f.startsWith('query_engine-')); allows the script to fi nish

screenshot

@JoeRoddy
Copy link
Owner

JoeRoddy commented Dec 16, 2023

hmmmm, thanks for opening this @RoccoFarrell

I'm guessing you're right about the reason, I would say for now, if you really need this library, you would need to downgrade your Prisma version, or fork it with the change you suggested. I know that's not a satisfying answer. Just don't have time to test your change on all Os's.

I'm wrapping up a big leg of a project right now, but once I'm done that I can look into this. Probably a few weeks.

@JoeRoddy
Copy link
Owner

It's also possible it's a windows specific issue, I've never tested this library on Windows, so it's possible it was always broken there.

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

2 participants