Yes, the output cordova.js file should contain the sha hash and probably
also, the entire command-line args used to create it.  So we can track back
issues.

@purplecabbage
risingj.com


On Fri, Aug 8, 2014 at 4:27 PM, Shazron <[email protected]> wrote:

> Sounds good for #1. For traceability purposes, will the sha hash be in
> the .js still as a comment, etc?
> ```git describe```
>
> On Thu, Aug 7, 2014 at 7:07 AM, Andrew Grieve <[email protected]>
> wrote:
> > That all sounds good to me Steve!
> >
> >
> > On Wed, Aug 6, 2014 at 6:48 PM, Steven Gill <[email protected]>
> wrote:
> >
> >> I have started to look into adding cordova.platformVersion to
> cordova.js.
> >> Issue [1].
> >>
> >> Trying to figure out the best way to implement this.
> >>
> >> One potential method.
> >>
> >> 1) Pass in platformVersion to cordova.js during build step. Make coho
> >> handle it during release process:
> >> First off, cordovajs doesn't get branched and tagged on its own anymore
> >>
> >> Non-Browserify use case
> >> A) coho prepare-release-branch --version 3.5.0 -r android
> >>    - This step would build cordova-js, inject cordova.platformVersion
> based
> >> on the --version being passed in.
> >>    - Updating version in package.json files
> >>     - Updating version numbers
> >>    - Creating release branches
> >>
> >> B) coho tag-release --version 3.5.0 -r android
> >>     - Tag cordova-android "3.5.0"
> >>     - Tag cordova-js "android-3.5.0"
> >>
> >> Browserify use case
> >> Currently, cordova.version is set to 'N/A' in browserify builds of
> >> cordova.js.
> >>
> >> We would probably have to create a utility function that grabs the
> >> cordova.platformVersion property from "platform_www/cordova.js" files.
> >> These are the cordova.js files that get bundled at release with the
> >> platforms from the non-browserify use case.
> >>
> >> Use the version when building cordova.js in prepare-browserify.js.
> >>
> >>
> >> Thoughts? Other suggestions?
> >>
> >>
> >> [1] https://issues.apache.org/jira/browse/CB-7219
> >>
>

Reply via email to