-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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 Messages: [Source and destination must not be the same.] in cordova-android@8.0.0 #692
Comments
Are you on Windows? This sounds like the same problem as https://github.com/apache/cordova-common/issues/64 |
Hello all ! I may add that I am encountering this issue too. For me the only workaround was to run the following commands once or twice…
When, with the same project, I was on Regarding the environment, I am on windows 10. Here is the complete environment info:
For me, the fs-extra defect referred in the apache/cordova-common#64 issue could well be the source of this bug ! |
I'm going to close this issue in favour of keeping the discussion in one spot at https://github.com/apache/cordova-common/issues/64. Unfortunately, our ability to solve this problem is largely limited by NodeJS and fs-extra. |
@apiaget Your "workaround" to reinstall the android platform is for sure a random and temporary behavior and not a solution. Because this error does not always occur. In my case I always build from a blank (git tracked file only) state. Without node_modules and no platforms folder. A hard-clean. In my case the build fails in most of tries. I don't even know the core of this problem. I hope for a upcoming solution and fast workaround. Currently I can't build the app. |
@dpogue This seems to be happen since Cordova-Android 8.0.0. What changed in relation to fs-extra? Maybe Cordova-Android should use older - working - version? In fact, the build nearly always fails. It's a very critical bug. ... What do you think about it? I have not quite understood the problem yet. I'm looking for a workaround. Until jprichardson/node-fs-extra#657 / jprichardson/node-fs-extra#626 is / are fixed. |
As part of the major version bump to several underlying libraries (cordova-common, cordova-lib, cordova-fetch, etc.) we migrated a bunch of code from shelljs to use fs-extra instead. This significantly improved the reliability of tests and resolved several weird bugs that people had been seeing for years. Unfortunately, this issue didn't show up in any of our test suites (which we do run on Windows). So it's not a change at the cordova-android level, it's in the cordova-common library which is used by all the platforms and the Cordova tooling. This issue appears to only affect people using Windows, which is probably why it's only getting reported on the Android repo and not iOS. |
Thanks. In fact, with this problem I can't build my app. It's a K.O. Be free to join the discussion: |
Yes. |
Should be fixed in v8.0.1 |
^8.1.0 worked for me though |
Bug Report
Problem
Error Messages: [Source and destination must not be the same.] in cordova-android@8.0.0
What is expected to happen?
cordova platform add android@8.0.0
What does actually happen?
Information
Command or Code
Environment, Platform, Device
Version information
Checklist
The text was updated successfully, but these errors were encountered: