-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
error when packageConfig.tmpdir set to false #1679
Comments
Thank you for your feedback. Personally, I am more inclined towards the third solution because the first two options are merely temporary measures that do not address the underlying issue. |
PTAL @electron/forgers |
Discussed in an @electron/wg-ecosystem meeting, going with option 3 and moving this to |
👋 Thanks for opening your first issue here! If you have a question about using Electron Packager, read the support docs. If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. Development and issue triage is community-driven, so please be patient and we will get back to you as soon as we can. To help make it easier for us to investigate your issue, please follow the contributing guidelines. |
Pre-flight checklist
Electron Forge version
7.2.0
Electron version
27.1.2
Operating system
maxOS 13.2.1
Last known working Electron Forge version
No response
Expected behavior
no error when packageConfig.tmpdir set to false
Actual behavior
When packageConfig.tmpdir set to false, will get the error below:
Error: Cannot copy '/path/to/project' to a subdirectory of itself, '/path/to/project/out/AppName***/Electron.app/Contents/Resources/app'.
This error was thrown by fs-extra, whem electron-packager try to copy project to the out dir.
Thats caused by out dir is not allowed to set, so out dir must be a subdirectory
I want to take a PR for this, but there are 3 ways to fix it, I dont know we prefer which one
false
, overwrite it toundefined
, fix(core): error if tmpdir is false, overwrite to undefined forge#3476false
, throw an error by forge, fix(core): error if tmpdir is false, throw error forge#3477If we choose 1 or 2, I will take the PR soon
Steps to reproduce
set packageConfig.tmpdir to false
run forge package
Additional information
No response
The text was updated successfully, but these errors were encountered: