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

Random build failures due to fs-extra defect #83

Closed
brodybits opened this issue Mar 19, 2019 · 5 comments
Closed

Random build failures due to fs-extra defect #83

brodybits opened this issue Mar 19, 2019 · 5 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@brodybits
Copy link

Random build failures due to fs-extra defect in: jprichardson/node-fs-extra#657

reported by @oliversalzburg in apache/cordova-common#64

also discussed in the following places:

The discussion is already in apache/cordova-common#64. I suspect we should keep it there for now. I think this issue needs to be reported and tracked at the top level since it affects multiple Cordova packages.

@brodybits brodybits added the bug Something isn't working label Mar 19, 2019
@oliversalzburg
Copy link

I really hope we can find a solution in fs-extra, but the maintainers are having a hard time deciding on what the right path to go is and there seems to be no hurry to work out a solution.

I previously had a similar experience when I tried to resolve the "link copy" issue, which is still open.

If someone has a suggestion on how to mitigate this issue in Cordova, I'd be glad to hear it. I currently see no other option than to implement a separate copy command that doesn't have an inode equality check (or a proper one). Which would be disappointing as that's the opposite of what you want when using such a library.

@brodybits
Copy link
Author

I can think of a couple ideas:

  • try using built-in fs.copySync from Node.js (I cannot tell if it would break anything else or not without some testing and analysis)
  • publish a new "copy" or "copy-sync" package with the JavaScript that does what we want

@dimitriscsd
Copy link

How is this coming along? Any decisions been taken? Fs extra is about to release v8 with apparently a workaround for this issue.

@brodybits
Copy link
Author

Please take a look at the recent comments in apache/cordova-common#64, where you should see the answer so far.

If we do incorporate a new fs-extra release, I suspect we would do a pipelined release of a number of components including cordova-common, some if not all Cordova platform components, cordova-lib, and cordova-cli.

@brodybits brodybits added the duplicate This issue or pull request already exists label Jun 7, 2019
@brodybits
Copy link
Author

I am now closing this issue as a duplicate of #121 which was just transferred from cordova-common.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

3 participants