cordova-hcp deploy copy all the file, should also use chcpignore

[dongdh]

cordova-hcp deploy copy all the file, should also use chcpignore

[nikDemyankov]

Yes, you are correct: it seems deploy doesn't respect the ignore list. At the moment it is used only for the build operation. This should be fixed.

Either way, can you check, that generated chcp.manifest file doesn't contain chcp.manifest in it? It feels like the issue you are having in nordnet/cordova-hot-code-push#133 is because of this.

Just open via browser https://path/to/your/content/chcp.manifest and search for chcp.manifest in it. If it's there - remove it.

[dongdh]

ye, the bug is from manifest file in the manifest file, i added it to ignore list, then it worked.

[nikDemyankov]

Good, thanks. Also, please check, that chcp.json is not in manifest file. These files should be ignored by CLI client by default. Strange, that they are not in your case.

[davidovich]

@nikDemyankov I have glanced over the code, and if you specify a .chcpigore file, it overrides the default array. see https://github.com/nordnet/cordova-hot-code-push-cli/blob/master/src/context.js#L57

[nikDemyankov]

@davidovich good catch, thanks :)

[nikDemyankov]

Released in v1.1.0. Should now work. If not - please, reopen the issue.

[diegodotta]

I'm using the v1.1.1 and the deploy doesn't respect the .chcpignore or chcp.manifest.

[nikDemyankov]

@diegodotta Are you running on Windows machine?

[diegodotta]

Yes, let me try on mac.

[anthonytvamp]

I can confirm @diegodotta that I'm using v1.1.1 on Mac and deploy still doesn't respect .chcpignore or chcp.manifest.

[nordnet-deprecation-bot]

👋 Hi! Thank you for your interest in this repo.

😢 We are not using nordnet/cordova-hot-code-push-cli anymore, and we lack the manpower and the experience needed to maintain it. We are aware of the inconveniece that this may cause you. Feel free to use it as is, or create your own fork.

🔒 This will now be closed & locked.

ℹ️ Please see #79 for more information.