Update fs-extra to v8

[dpogue]

Platforms affected

All, but hopefully mostly Windows

Motivation and Context

Hopefully resolves #64 apache/cordova#121 on cordova-common only

Description

Updated fs-extra to the latest version (8.0.0).

Testing

npm test passed locally on Linux, will ensure it's green on CI.

Checklist

• I've run the tests to see all new and existing tests pass
• I added automated test coverage as appropriate for this change
• If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)

[codecov-io]

Codecov Report

Merging #70 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #70   +/-   ##
=======================================
  Coverage   89.04%   89.04%           
=======================================
  Files          20       20           
  Lines        1826     1826           
  Branches      374      374           
=======================================
  Hits         1626     1626           
  Misses        200      200

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9cd117e...c0cb95e. Read the comment docs.

[raphinesse]

AFAICT, just updating will not fix anything. I think the new version just includes an opt-in workaround to disable some checks during copy*. The downside being that this could delete your copy source. And that we'd have to edit each and every location where we copy something.

Or is that information outdated?

[FlorinSerban]

@raphinesse

AFAICT, just updating will not fix anything. I think the new version just includes an opt-in workaround to disable some checks during copy*. The downside being that this could delete your copy source. And that we'd have to edit each and every location where we copy something.

Or is that information outdated?

I think the right fix was implemented. Please check this comment.

[raphinesse]

Looks like you're right, @FlorinSerban. Thanks for the pointer!

[catalinmitrofan]

When are you guys planning a release with this fix?

[oliversalzburg]

@catalinmitrofan It hasn't even been merged yet. Calm down 😁

[catalinmitrofan]

@oliversalzburg, I just want to know, when the changes will be merged => when is a new release planned?

[oliversalzburg]

I believe there is no schedule. If this is a blocking issue for you, I recommend implementing a workaround for the time being.

[dimitriscsd]

Considering the severity of the issue (Everyone with a windows build server is unable to build their cordova project), I believe there should be a schedule asap.

The workaround is a nice band aid but a proper fix is always better.

[oliversalzburg]

@dimitriscsd We have waited a long time for fs-extra to fix their code. There is really no need for this kind of pressure so soon after a change has been proposed in Cordova. I'm sure everyone agrees that this needs to be fixed ASAP.

[dimitriscsd]

That's perfect. I didn't mean to apply pressure, I just pointed out that it's a serious problem users of Cordova are facing and it should be places somewhere in the pipeline rather than be left in a backlog somewhere.

It's great to hear that you guys understand the severity of the issue and I'm grateful for your work and efforts.

[brodybits]

Let's get this one merged and published. Thanks guys for the patience.

[dpogue]

Note that we also need to update fs-extra in any other modules that use it, which includes cordova-lib, cordova-create, and cordova-fetch, and possibly others

[catalinmitrofan]

Thanks again to everyone for their fast work on this issue.

[catalinmitrofan]

So, is there a planned release with this fix?

[brodybits]

So, is there a planned release with this fix?

I suggest you post this kind of question to the forum at: dev@cordova.apache.org

I would like to clarify a couple things:

To fully "release" this fix, we would actually need to apply this fix and make a release for multiple components as @dpogue said in #70 (comment).

I personally do not have so much time to make a new release of the components right now. Keep in mind that this is a 100% volunteer project with very limited corporate sponsorship, if any. (I think a few big companies had donated maintainer time in the past, seems to be gone now.)

Thanks for your continued patience.

[catalinmitrofan]

@brodybits , thank you for the explanation and your answer. I will wait then for the next release

[l3ender]

Thanks for all those who have worked on this. Is there anything I could do to help out to get it merged and released?

[brodybits]

I would like to merge this, maybe with my fs-extra@^8.0.1 suggestion if there are no objections from other maintainers. Then one of us maintainers will have to publish a new cordova-common release in the next 1-2 weeks.

@l3ender and others please followup with us via email to dev@cordova.apache.org in case we drop this one. I think all maintainers are extremely busy in general.

[dpogue]

As I understand it, the reason this has not been merged is because nobody has been able to actually confirm whether it fixes the problem or not (although it's very hard to test that when other libraries are also pulling in their own copies of fs-extra).

As long as we're confident that the major version bump of fs-extra doesn't break anything here, I'm good with seeing this merged.

[brodybits]

I would personally favor publishing this change as a minor release; fs-extra@8 seems to not break for Node.js back to version 6. Any other comments or objections?

[catalinmitrofan]

@dpogue , I have moved the cordova-common repository to our own, and I patched it there. Since I patched it, I didn't get any issues when trying to build the app for windows. I have been using it like this for the last 20 days, without any issues. When the change will be published, then I will revert back to using the repository from here, until then, I am using my "patched" repository.

[brodybits]

I have just merged this proposal, with an updated title since it will include latest patch and minor release on fs-extra@8. I will now propose a minor release to be published within the next week.

[brodybits]

This update is now published in cordova-common@3.2.0. I documented the recommended update procedure in apache/cordova#121 (comment).

[gwynjudd]

I'm still getting this issue with the updated version of cordova-common and fs-extra. Things I've checked:

1. version of cordova-common is 3.2.0
2. version of fs-extra is 8.1.0

node version is 8.9.1

Any other things I should be checking?

[brodybits]

It is more likely to work on Node.js 10 which supports bigint. There may be some more workarounds for older Node.js versions but I would consider it to be a waste of time.

[gwynjudd]

@brodybits thanks I'll give it a shot

EDIT: looks to have gotten it working with node v 10.16.3