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

Error cleaning up npm scenario: Error: Cannot copy '../../../../../esprima/bin/esvalidate.js' to a subdirectory of itself #967

Closed
elwayman02 opened this issue Oct 11, 2023 · 2 comments
Closed

Error cleaning up npm scenario: Error: Cannot copy '../../../../../esprima/bin/esvalidate.js' to a subdirectory of itself #967

elwayman02 opened this issue Oct 11, 2023 · 2 comments

Comments

@elwayman02
Copy link

I'm seeing this error when bumping @embroider/test-setup from 2.1.1 to 3.0.2. It only occurs when running embroider-safe and embroider-optimized builds. Given the nature of the error seems related to #813 and is thrown from ember-try's copy functionality, this seemed the best place to file the issue for investigation.

Error cleaning up npm scenario: Error: Cannot copy '../../../../../esprima/bin/esvalidate.js' to a subdirectory of itself, '../../../../../esprima/bin/esvalidate.js'.
    at /home/runner/work/ember-resize-modifier/ember-resize-modifier/node_modules/ember-try/node_modules/fs-extra/lib/copy/copy.js:210:21
    at FSReqCallback.oncomplete (fs.js:180:23)

Here's a full PR reproduction: elwayman02/ember-resize-modifier#808
@elwayman02 elwayman02 mentioned this issue Oct 11, 2023
Open
@kategengler
Copy link
Member

This is fixed in v3.0 of ember-try by #912

@elwayman02
Copy link
Author

Ah, I think dependabot was having a stroke here...my library is already using ember-try but for some reason it wasn't rebasing. Sorry for the false alarm! Should be able to close this out momentarily.

dwickern added a commit to dwickern/ember-data.model-fragments that referenced this issue May 3, 2024
@dwickern
upgrade ember-try
dc95246 
Fixes the following error:
Error: Cannot copy '../../../browserslist/cli.js' to a subdirectory of itself, '../../../browserslist/cli.js'.
    at /home/runner/work/ember-data-model-fragments/ember-data-model-fragments/node_modules/ember-try/node_modules/fs-extra/lib/copy/copy.js:210:21
    at FSReqCallback.oncomplete (fs.js:180:23)

ember-cli/ember-try#967
@dwickern dwickern mentioned this issue May 5, 2024
Merged
knownasilya pushed a commit to adopted-ember-addons/ember-data-model-fragments that referenced this issue May 6, 2024
@dwickern
Fix updating fragment arrays multiple times in the same runloop (#487)
f04e04a 
* run CI in production environment

* upgrade ember-try

Fixes the following error:
Error: Cannot copy '../../../browserslist/cli.js' to a subdirectory of itself, '../../../browserslist/cli.js'.
    at /home/runner/work/ember-data-model-fragments/ember-data-model-fragments/node_modules/ember-try/node_modules/fs-extra/lib/copy/copy.js:210:21
    at FSReqCallback.oncomplete (fs.js:180:23)

ember-cli/ember-try#967

* upgrade to node.js 18

* fix store.find deprecation

* replace assert.throws with expectAssertion

* fix updating fragment array multiple times

* revert to node 14 and use `yarn --ignore-engines`
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
2 participants
@kategengler @elwayman02