Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Update Remote Build to Support Merges Directory #19

Closed
mwbrooks opened this Issue Mar 20, 2013 · 10 comments

Comments

Projects
None yet
8 participants
Owner

mwbrooks commented Mar 20, 2013

Description

Projects are now created using a Cordova project structure (#17).

One feature of the Cordova project structure is the merges/ directory, which adds support for platform-specific files to replace files in www/.

Cordova has a prepare command that can be leveraged to preprocess the assets before compiling. We should be able to prepare the assets, before building remotely.

Is this one planned beeing fixed?
We have splash images added inside the merges/android and merges/ios directory, but they don't get copied to the www folder when building remotely, only locally.

I would actually say that this is a bug. Since you will have to add the images inside the www folder, and you don't want all the images for the different devices added to the www folder when they are not needed. Application will be huuuuge.. and It seems strange that there are 2 different builds that should do the same (only remotely or locally), which behaves in 2 totally different ways.

Owner

mwbrooks commented Aug 1, 2013

@frankbolviken

Certainly, we want to implementation this feature.

I would actually say that this is a bug.

I think that's fair. I created this issue... 5 months ago... when it was just another feature. Now that everything else has been built, it is a gleaming bug.

Is there a work-around to this bug? You mentioned the prepare command in cordova, is it possible to manually call that and then do a remote build or would the remote build process overwrite the prepare?

louy commented Sep 4, 2013

This should be fixed!

Ventis commented Dec 12, 2013

The lack of this feature is a compelling argument not to use phonegap remote build

@lorinbeer lorinbeer added this to the Ice Box milestone Apr 16, 2014

Contributor

lorinbeer commented May 19, 2014

Given the feedback here, I think this issue is important enough to prioritize. Moving to this weeks milestone.

@lorinbeer lorinbeer self-assigned this May 19, 2014

Any word on when this feature will be implemented?

@amirudin amirudin referenced this issue in phonegap/build Aug 19, 2014

Open

[Build] Add 'merge' directory support #343

@mwbrooks mwbrooks removed this from the Ice Box milestone Jun 17, 2015

@lorinbeer lorinbeer was unassigned by mwbrooks Jun 17, 2015

@mwbrooks mwbrooks removed this from the Ice Box milestone Jun 17, 2015

@mwbrooks mwbrooks added this to the Backlog milestone Sep 15, 2015

Owner

filmaj commented Apr 18, 2017

Cordova no longer supports the merges/ directory - making this issue not applicable. Closing!

@filmaj filmaj closed this Apr 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment