[GitHub] brodybits edited a comment on issue #599: cdvMinSdkVersion seems to not work in 7.1.4

[GitBox <gi...@apache.org>]

brodybits edited a comment on issue #599: cdvMinSdkVersion seems to not work in 7.1.4
URL: https://github.com/apache/cordova-android/issues/599#issuecomment-455416993
 
 
   > Would the Solution be to move back to 7.1.2 while this is fixed ?
   
   I cannot promise this will ever be fixed in a patch release. I personally do not have any extra time to make such a patch release and I think not other committers will have the extra time either.
   
   The only way I can think of to fix this in a patch release would be to revert 00134cf (cherry-pick of PR #495) in 7.1.x. This would be a breaking change for people who configure via Gradle as described in [[1]](https://cordova.apache.org/docs/en/latest/guide/platforms/android/#setting-gradle-properties), which would be a step backward.
   
   For future releases such as upcoming Cordova 9 (apache/cordova#10) I would say the minimum & target SDK should be configured as described in [[1]](https://cordova.apache.org/docs/en/latest/guide/platforms/android/#setting-gradle-properties), not by setting the preferences in config.xml.
   
   [1] <https://cordova.apache.org/docs/en/latest/guide/platforms/android/#setting-gradle-properties>

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org