Skip to content
This repository has been archived by the owner on Apr 3, 2020. It is now read-only.

fix package name validation #162

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

boynet
Copy link

@boynet boynet commented Apr 15, 2015

you can see accepted aplications in google play that contain number in the first part of package name, those apps cant be build with crosswalk and intel xdk because of this validation bug\inconsistent with google play. for example this package: http://bit.ly/1aVO6Re

you can see accepted aplications in google play that contain number in the first part of package name, those apps cant be build with crosswalk and intel xdk because of this validation bug\unconsist with google play. for example this package: http://bit.ly/1aVO6Re
@boynet
Copy link
Author

boynet commented Apr 16, 2015

we should use theire working regex I think:
https://github.com/apache/cordova-android/blob/master/bin/lib/create.js#L153

@boynet
Copy link
Author

boynet commented Jun 29, 2015

anyone? cant understand why not fixing this simple bug?

@rakuco
Copy link

rakuco commented Jun 29, 2015

ping @fujunwei @huningxin

@fujunwei
Copy link
Contributor

Thanks so much. I think it's perfect solution if you cherry pick this commit to the branch apache/cordova-android@e4c9beb.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants