Skip to content
This repository has been archived by the owner on Aug 14, 2021. It is now read-only.

Fix build #8

Closed
wants to merge 1 commit into from
Closed

Conversation

pantharshit00
Copy link

@pantharshit00 pantharshit00 commented Apr 12, 2020

Hi, I was watching the stream. In the end, Ben was unable to fix this build.

Turn out target: serverless uses ncc and __dirname is set to / so prisma can't the resolve rust binaries. But we are not targetting serverless here. So removing that fixes the build. Also, you might not need next.config.js all together as css support is built in now but I didn't delete it.

Also, fixes the port number in dev :)

@vercel
Copy link

vercel bot commented Apr 12, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

@radihuq
Copy link
Collaborator

radihuq commented Apr 12, 2020

Hey thanks a lot for the comment Harsh. Just wanted to also mention we got prisma working on my machine -- @alexluong found this issue prisma/prisma#1973 and it was relevant to me. I changed the folder name from "WHAT IM UP TO" to "WHAT_IM_UP_TO" and prisma generate started working for me.

@pantharshit00
Copy link
Author

@radihuq It is not about this. Try running yarn build right now without this PR, it would fail.

This PR fixes that

@pantharshit00
Copy link
Author

Ben manually fixed it locally

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants