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

Resolve build errors of vite-plugin-moncao-editor in Windows #4531

Closed
wants to merge 3 commits into from

Conversation

qiuquanwu
Copy link

fix build error in Windows

Change Summary

Provide summary of changes with issue number if any.

Change type

  • feat: (new feature for the user, not a new feature for build script)
  • [* ] fix: (bug fix for the user, not a fix to a build script)
  • docs: (changes to the documentation)
  • style: (formatting, missing semi colons, etc; no production code change)
  • refactor: (refactoring production code, eg. renaming a variable)
  • test: (adding missing tests, refactoring tests; no production code change)
  • chore: (updating grunt tasks etc; no production code change)

Test/ Verification

Resolve build errors in Windows

 ERROR  [vite-plugin-moncao-editor] ENOENT: no such file or directory, mkdir 'D:\workspace\nocodb\packages\nc-gui\D:\workspace\nocodb\packages\nc-gui\.nuxt\dist\client\monacoeditorwork'   


 ERROR  ENOENT: no such file or directory, mkdir 'D:\workspace\nocodb\packages\nc-gui\D:\workspace\nocodb\packages\nc-gui\.nuxt\dist\client\monacoeditorwork'                     19:08:24  

  at Object.mkdirSync (node:fs:1386:3)
  at Object.writeBundle (node_modules\vite-plugin-monaco-editor\dist\index.js:99:20)
  at /D:/workspace/nocodb/packages/nc-gui/node_modules/@nuxt/vite-builder/node_modules/vite/node_modules/rollup/dist/es/shared/rollup.js:22710:40
  at async Promise.all (index 0)
  at async PluginDriver.hookParallel (/D:/workspace/nocodb/packages/nc-gui/node_modules/@nuxt/vite-builder/node_modules/vite/node_modules/rollup/dist/es/shared/rollup.js:22638:9)
  at async /D:/workspace/nocodb/packages/nc-gui/node_modules/@nuxt/vite-builder/node_modules/vite/node_modules/rollup/dist/es/shared/rollup.js:23763:13
  at async catchUnfinishedHookActions (/D:/workspace/nocodb/packages/nc-gui/node_modules/@nuxt/vite-builder/node_modules/vite/node_modules/rollup/dist/es/shared/rollup.js:23088:20)        
  at async doBuild (/D:/workspace/nocodb/packages/nc-gui/node_modules/@nuxt/vite-builder/node_modules/vite/dist/node/chunks/dep-4da11a5e.js:45831:20)
  at async Module.build (/D:/workspace/nocodb/packages/nc-gui/node_modules/@nuxt/vite-builder/node_modules/vite/dist/node/chunks/dep-4da11a5e.js:45662:16)
  at async buildClient (/D:/workspace/nocodb/packages/nc-gui/node_modules/@nuxt/vite-builder/dist/shared/vite-builder.8e427c63.mjs:318:5)
  at async bundle (/D:/workspace/nocodb/packages/nc-gui/node_modules/@nuxt/vite-builder/dist/shared/vite-builder.8e427c63.mjs:853:3)
  at async build (/D:/workspace/nocodb/packages/nc-gui/node_modules/nuxt/dist/index.mjs:2084:5)
  at async Object.invoke (/D:/workspace/nocodb/packages/nc-gui/node_modules/nuxi/dist/chunks/build.mjs:51:5)
  at async _main (/D:/workspace/nocodb/packages/nc-gui/node_modules/nuxi/dist/cli.mjs:50:20)

Additional information / screenshots (optional)

Anything for maintainers to be made aware of

fix build error in Windows
@CLAassistant
Copy link

CLAassistant commented Nov 29, 2022

CLA assistant check
All committers have signed the CLA.

@qiuquanwu qiuquanwu changed the title Update nuxt.config.ts Resolve build errors of vite-plugin-moncao-editor in Windows Nov 29, 2022
@wingkwong
Copy link
Member

Can you share the steps of reproducing the error?

@qiuquanwu
Copy link
Author

Can you share the steps of reproducing the error?
Windows:
execute
cd nc-gui/
npm i
npm run build

@wingkwong
Copy link
Member

@pranavxc please help verify the fix on Windows machine.

@wingkwong
Copy link
Member

@qiuquanwu Please remove those unrelated changes.

@wingkwong wingkwong added 🏌️Status: Action Needed Some actions are needed to be done 🛑 Status: Do Not Merge Used in PR only. The PR cannot be merged due to some reasons. labels Dec 1, 2022
@pranavxc
Copy link
Member

pranavxc commented Dec 1, 2022

Verified in Windows 11 Home edition x64

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏌️Status: Action Needed Some actions are needed to be done 🛑 Status: Do Not Merge Used in PR only. The PR cannot be merged due to some reasons.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants