RE: [DISCUSS] Dropping the -dev suffix
As a committer working on the master branch using npm link, it has been useful to have '-dev' prefix and do a "cordova --version". I vote for (3). Thanks, Nikhil -Original Message- From: Vladimir Kotikov (Akvelon) [mailto:v-vlk...@microsoft.com] Sent: Wednesday, July 8, 2015 7:28 AM To: dev@cordova.apache.org Subject: RE: [DISCUSS] Dropping the -dev suffix Hey, guys. I've found another reason to drop this while testing cordova-lib with updated dependencies (see https://issues.apache.org/jira/browse/CB-9332) It appears, that starting from 4.x.x semver treats prerelease versions in a very different way. For example: require('semver').satisfies('5.1.0-dev', '>=0.10.0') // true with semver@2 require('semver').satisfies('5.1.0-dev', '>=0.10.0') // false with semver@4 For more details see https://github.com/npm/node-semver#prerelease-tags and http://blog.npmjs.org/post/98131109725/npm-2-0-0 As a result of updating semver without removing -dev we'll get a lot of issues with plugins that has engine tags, referencing our tools. So I think there are 3 options for us: 1. migrate to semver@4 and drop -dev suffix 2. leave all as is and don't update semver dependency 3. update semver, leave -dev suffix, and add a workaround for engines check (for example strip out -dev suffix while checking engines) Any thoughts? --- Best regards, Vladimir -Original Message- From: Jesse [mailto:purplecabb...@gmail.com] Sent: Thursday, 14 May, 2015 20:42 To: dev@cordova.apache.org Subject: Re: [DISCUSS] Dropping the -dev suffix I agree we should keep it, at least for the time being. There is code in coho that manipulates all the versions in different branches when we do releases, changing it would mean we need to update all of that coho logic also. Granted, coho releases would be simpler without the suffix, however, I don't think it is currently worth the change. @purplecabbage risingj.com On Thu, May 14, 2015 at 10:35 AM, Nikhil Khandelwal wrote: > Now that we know how to fix the plugin to work with 'dev' versions of > platforms, we could keep the 'dev' suffix. It's somewhat useful for > anyone using our 'master' branch (I'm hoping its mostly the dev team) > and adding the platform and have the dev prefix listed here: > > λ cordova platform ls > Installed platforms: android 4.1.0-dev, ios 3.8.0, windows 4.0.0-dev, > wp8 Available platforms: amazon-fireos, blackberry10, browser, > firefoxos, > windows8 > > Thanks, > Nikhil > > > -Original Message- > From: Jesse [mailto:purplecabb...@gmail.com] > Sent: Wednesday, May 13, 2015 4:39 PM > To: dev@cordova.apache.org > Subject: Re: [DISCUSS] Dropping the -dev suffix > > So back to the initial discussion, should we keep our -dev suffixes? > If so we need to be sure to use engine tags like '>=4.0' or if you > specifically want to force a specific min patch, and still allow the > dev suffix, you can do '>=4.0.0--' > > And of course this should be documented. > > > @purplecabbage > risingj.com > > On Wed, May 13, 2015 at 4:21 PM, Jesse wrote: > > > Yes, I just verified the same. > > > > Changing the plugin.xml engine to : > > cordova-wp8 >=4.0.0-- will allow install for both wp8@4.0.0 and > > wp8@4.0.0-dev > > > > @purplecabbage > > risingj.com > > > > On Wed, May 13, 2015 at 4:18 PM, Tim Barham > > > > wrote: > > > >> No, you were correct - any sort of pre-release identifier makes the > >> version less than the version without the pre-release identifier > >> (since it signifies a pre-release). > >> > >> Hence '4.0.0-dev', '4.0.0-alpha', '4.0.0-0.0.1' and > >> '4.0.0-iwishiwasarealreease' are all < 4.0.0. > >> > >> It seems to me the problem is with the plugin since it requires > >> cordova-wp8 >=4.0.0, and there is no cordova-wp8 release (or yet to > >> be released version on github) that satisfies that requirement. > >> > >> ________________ > >> From: So, Byoungro > >> Sent: Thursday, May 14, 2015 3:39 AM > >> To: dev@cordova.apache.org > >> Subject: RE: [DISCUSS] Dropping the -dev suffix > >> > >> Sorry, I got the other way around. :) 4.0.0-dev is > 4.0.0. > >> > >> Byoungro So > >> SSG / DPD / Mobile Computing and Compilers Intel Corporation > >> > >> -Original Message- > >> From: So, Byoungro > >> Sent: Wednesday, May 13, 2015 10:37 AM > >
RE: [DISCUSS] Dropping the -dev suffix
Hey, guys. I've found another reason to drop this while testing cordova-lib with updated dependencies (see https://issues.apache.org/jira/browse/CB-9332) It appears, that starting from 4.x.x semver treats prerelease versions in a very different way. For example: require('semver').satisfies('5.1.0-dev', '>=0.10.0') // true with semver@2 require('semver').satisfies('5.1.0-dev', '>=0.10.0') // false with semver@4 For more details see https://github.com/npm/node-semver#prerelease-tags and http://blog.npmjs.org/post/98131109725/npm-2-0-0 As a result of updating semver without removing -dev we'll get a lot of issues with plugins that has engine tags, referencing our tools. So I think there are 3 options for us: 1. migrate to semver@4 and drop -dev suffix 2. leave all as is and don't update semver dependency 3. update semver, leave -dev suffix, and add a workaround for engines check (for example strip out -dev suffix while checking engines) Any thoughts? --- Best regards, Vladimir -Original Message- From: Jesse [mailto:purplecabb...@gmail.com] Sent: Thursday, 14 May, 2015 20:42 To: dev@cordova.apache.org Subject: Re: [DISCUSS] Dropping the -dev suffix I agree we should keep it, at least for the time being. There is code in coho that manipulates all the versions in different branches when we do releases, changing it would mean we need to update all of that coho logic also. Granted, coho releases would be simpler without the suffix, however, I don't think it is currently worth the change. @purplecabbage risingj.com On Thu, May 14, 2015 at 10:35 AM, Nikhil Khandelwal wrote: > Now that we know how to fix the plugin to work with 'dev' versions of > platforms, we could keep the 'dev' suffix. It's somewhat useful for > anyone using our 'master' branch (I'm hoping its mostly the dev team) > and adding the platform and have the dev prefix listed here: > > λ cordova platform ls > Installed platforms: android 4.1.0-dev, ios 3.8.0, windows 4.0.0-dev, > wp8 Available platforms: amazon-fireos, blackberry10, browser, > firefoxos, > windows8 > > Thanks, > Nikhil > > > -----Original Message- > From: Jesse [mailto:purplecabb...@gmail.com] > Sent: Wednesday, May 13, 2015 4:39 PM > To: dev@cordova.apache.org > Subject: Re: [DISCUSS] Dropping the -dev suffix > > So back to the initial discussion, should we keep our -dev suffixes? > If so we need to be sure to use engine tags like '>=4.0' or if you > specifically want to force a specific min patch, and still allow the > dev suffix, you can do '>=4.0.0--' > > And of course this should be documented. > > > @purplecabbage > risingj.com > > On Wed, May 13, 2015 at 4:21 PM, Jesse wrote: > > > Yes, I just verified the same. > > > > Changing the plugin.xml engine to : > > cordova-wp8 >=4.0.0-- will allow install for both wp8@4.0.0 and > > wp8@4.0.0-dev > > > > @purplecabbage > > risingj.com > > > > On Wed, May 13, 2015 at 4:18 PM, Tim Barham > > > > wrote: > > > >> No, you were correct - any sort of pre-release identifier makes the > >> version less than the version without the pre-release identifier > >> (since it signifies a pre-release). > >> > >> Hence '4.0.0-dev', '4.0.0-alpha', '4.0.0-0.0.1' and > >> '4.0.0-iwishiwasarealreease' are all < 4.0.0. > >> > >> It seems to me the problem is with the plugin since it requires > >> cordova-wp8 >=4.0.0, and there is no cordova-wp8 release (or yet to > >> be released version on github) that satisfies that requirement. > >> > >> ________________ > >> From: So, Byoungro > >> Sent: Thursday, May 14, 2015 3:39 AM > >> To: dev@cordova.apache.org > >> Subject: RE: [DISCUSS] Dropping the -dev suffix > >> > >> Sorry, I got the other way around. :) 4.0.0-dev is > 4.0.0. > >> > >> Byoungro So > >> SSG / DPD / Mobile Computing and Compilers Intel Corporation > >> > >> -Original Message- > >> From: So, Byoungro > >> Sent: Wednesday, May 13, 2015 10:37 AM > >> To: dev@cordova.apache.org > >> Subject: RE: [DISCUSS] Dropping the -dev suffix > >> > >> Semver should be able to understand '4.0.0-dev' as the valid > >> version string. > >> There could be some other requirement problems, like <=, >. > >> For example, 4.0.0-dev is < 4.0.0. > >> So, if the requirement
Re: [DISCUSS] Dropping the -dev suffix
I agree we should keep it, at least for the time being. There is code in coho that manipulates all the versions in different branches when we do releases, changing it would mean we need to update all of that coho logic also. Granted, coho releases would be simpler without the suffix, however, I don't think it is currently worth the change. @purplecabbage risingj.com On Thu, May 14, 2015 at 10:35 AM, Nikhil Khandelwal wrote: > Now that we know how to fix the plugin to work with 'dev' versions of > platforms, we could keep the 'dev' suffix. It's somewhat useful for anyone > using our 'master' branch (I'm hoping its mostly the dev team) and adding > the platform and have the dev prefix listed here: > > λ cordova platform ls > Installed platforms: android 4.1.0-dev, ios 3.8.0, windows 4.0.0-dev, wp8 > Available platforms: amazon-fireos, blackberry10, browser, firefoxos, > windows8 > > Thanks, > Nikhil > > > -Original Message- > From: Jesse [mailto:purplecabb...@gmail.com] > Sent: Wednesday, May 13, 2015 4:39 PM > To: dev@cordova.apache.org > Subject: Re: [DISCUSS] Dropping the -dev suffix > > So back to the initial discussion, should we keep our -dev suffixes? > If so we need to be sure to use engine tags like '>=4.0' or if you > specifically want to force a specific min patch, and still allow the dev > suffix, you can do '>=4.0.0--' > > And of course this should be documented. > > > @purplecabbage > risingj.com > > On Wed, May 13, 2015 at 4:21 PM, Jesse wrote: > > > Yes, I just verified the same. > > > > Changing the plugin.xml engine to : > > cordova-wp8 >=4.0.0-- will allow install for both wp8@4.0.0 and > > wp8@4.0.0-dev > > > > @purplecabbage > > risingj.com > > > > On Wed, May 13, 2015 at 4:18 PM, Tim Barham > > wrote: > > > >> No, you were correct - any sort of pre-release identifier makes the > >> version less than the version without the pre-release identifier > >> (since it signifies a pre-release). > >> > >> Hence '4.0.0-dev', '4.0.0-alpha', '4.0.0-0.0.1' and > >> '4.0.0-iwishiwasarealreease' are all < 4.0.0. > >> > >> It seems to me the problem is with the plugin since it requires > >> cordova-wp8 >=4.0.0, and there is no cordova-wp8 release (or yet to > >> be released version on github) that satisfies that requirement. > >> > >> > >> From: So, Byoungro > >> Sent: Thursday, May 14, 2015 3:39 AM > >> To: dev@cordova.apache.org > >> Subject: RE: [DISCUSS] Dropping the -dev suffix > >> > >> Sorry, I got the other way around. :) 4.0.0-dev is > 4.0.0. > >> > >> Byoungro So > >> SSG / DPD / Mobile Computing and Compilers Intel Corporation > >> > >> -Original Message- > >> From: So, Byoungro > >> Sent: Wednesday, May 13, 2015 10:37 AM > >> To: dev@cordova.apache.org > >> Subject: RE: [DISCUSS] Dropping the -dev suffix > >> > >> Semver should be able to understand '4.0.0-dev' as the valid version > >> string. > >> There could be some other requirement problems, like <=, >. > >> For example, 4.0.0-dev is < 4.0.0. > >> So, if the requirement is >=4.0.0, then the version string '4.0.0-dev' > >> will fail to satisfy the requirement. > >> > >> Byoungro So > >> SSG / DPD / Mobile Computing and Compilers Intel Corporation > >> > >> -Original Message- > >> From: Steven Gill [mailto:stevengil...@gmail.com] > >> Sent: Wednesday, May 13, 2015 10:21 AM > >> To: dev@cordova.apache.org > >> Subject: [DISCUSS] Dropping the -dev suffix > >> > >> Recently, a issue (CB-9021) popped up where mobile spec was failing > >> because the semver check failed on a platform due to `-dev` being in > >> the version and not valid. > >> > >> I think we should remove -dev from our versions. Right now, after we > >> do a release, we add -dev to show that the repo is in development and > >> hasn't been released. I think we can accomplish the same by just > >> bumping the version (minor?) in the package.json and just know it > >> isn't released by seeing what is published on npm. > >> > >> Thoughts? > >> > >> - > >> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > >> For additional commands, e-mail: dev-h...@cordova.apache.org > >> > >> > >> - > >> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > >> For additional commands, e-mail: dev-h...@cordova.apache.org > >> > >> > > > > - > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > For additional commands, e-mail: dev-h...@cordova.apache.org >
RE: [DISCUSS] Dropping the -dev suffix
Now that we know how to fix the plugin to work with 'dev' versions of platforms, we could keep the 'dev' suffix. It's somewhat useful for anyone using our 'master' branch (I'm hoping its mostly the dev team) and adding the platform and have the dev prefix listed here: λ cordova platform ls Installed platforms: android 4.1.0-dev, ios 3.8.0, windows 4.0.0-dev, wp8 Available platforms: amazon-fireos, blackberry10, browser, firefoxos, windows8 Thanks, Nikhil -Original Message- From: Jesse [mailto:purplecabb...@gmail.com] Sent: Wednesday, May 13, 2015 4:39 PM To: dev@cordova.apache.org Subject: Re: [DISCUSS] Dropping the -dev suffix So back to the initial discussion, should we keep our -dev suffixes? If so we need to be sure to use engine tags like '>=4.0' or if you specifically want to force a specific min patch, and still allow the dev suffix, you can do '>=4.0.0--' And of course this should be documented. @purplecabbage risingj.com On Wed, May 13, 2015 at 4:21 PM, Jesse wrote: > Yes, I just verified the same. > > Changing the plugin.xml engine to : > cordova-wp8 >=4.0.0-- will allow install for both wp8@4.0.0 and > wp8@4.0.0-dev > > @purplecabbage > risingj.com > > On Wed, May 13, 2015 at 4:18 PM, Tim Barham > wrote: > >> No, you were correct - any sort of pre-release identifier makes the >> version less than the version without the pre-release identifier >> (since it signifies a pre-release). >> >> Hence '4.0.0-dev', '4.0.0-alpha', '4.0.0-0.0.1' and >> '4.0.0-iwishiwasarealreease' are all < 4.0.0. >> >> It seems to me the problem is with the plugin since it requires >> cordova-wp8 >=4.0.0, and there is no cordova-wp8 release (or yet to >> be released version on github) that satisfies that requirement. >> >> >> From: So, Byoungro >> Sent: Thursday, May 14, 2015 3:39 AM >> To: dev@cordova.apache.org >> Subject: RE: [DISCUSS] Dropping the -dev suffix >> >> Sorry, I got the other way around. :) 4.0.0-dev is > 4.0.0. >> >> Byoungro So >> SSG / DPD / Mobile Computing and Compilers Intel Corporation >> >> -Original Message- >> From: So, Byoungro >> Sent: Wednesday, May 13, 2015 10:37 AM >> To: dev@cordova.apache.org >> Subject: RE: [DISCUSS] Dropping the -dev suffix >> >> Semver should be able to understand '4.0.0-dev' as the valid version >> string. >> There could be some other requirement problems, like <=, >. >> For example, 4.0.0-dev is < 4.0.0. >> So, if the requirement is >=4.0.0, then the version string '4.0.0-dev' >> will fail to satisfy the requirement. >> >> Byoungro So >> SSG / DPD / Mobile Computing and Compilers Intel Corporation >> >> -Original Message- >> From: Steven Gill [mailto:stevengil...@gmail.com] >> Sent: Wednesday, May 13, 2015 10:21 AM >> To: dev@cordova.apache.org >> Subject: [DISCUSS] Dropping the -dev suffix >> >> Recently, a issue (CB-9021) popped up where mobile spec was failing >> because the semver check failed on a platform due to `-dev` being in >> the version and not valid. >> >> I think we should remove -dev from our versions. Right now, after we >> do a release, we add -dev to show that the repo is in development and >> hasn't been released. I think we can accomplish the same by just >> bumping the version (minor?) in the package.json and just know it >> isn't released by seeing what is published on npm. >> >> Thoughts? >> >> - >> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org >> For additional commands, e-mail: dev-h...@cordova.apache.org >> >> >> - >> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org >> For additional commands, e-mail: dev-h...@cordova.apache.org >> >> > - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
Re: [DISCUSS] Dropping the -dev suffix
So back to the initial discussion, should we keep our -dev suffixes? If so we need to be sure to use engine tags like '>=4.0' or if you specifically want to force a specific min patch, and still allow the dev suffix, you can do '>=4.0.0--' And of course this should be documented. @purplecabbage risingj.com On Wed, May 13, 2015 at 4:21 PM, Jesse wrote: > Yes, I just verified the same. > > Changing the plugin.xml engine to : > cordova-wp8 >=4.0.0-- will allow install for both wp8@4.0.0 and > wp8@4.0.0-dev > > @purplecabbage > risingj.com > > On Wed, May 13, 2015 at 4:18 PM, Tim Barham > wrote: > >> No, you were correct - any sort of pre-release identifier makes the >> version less than the version without the pre-release identifier (since it >> signifies a pre-release). >> >> Hence '4.0.0-dev', '4.0.0-alpha', '4.0.0-0.0.1' and >> '4.0.0-iwishiwasarealreease' are all < 4.0.0. >> >> It seems to me the problem is with the plugin since it requires >> cordova-wp8 >=4.0.0, and there is no cordova-wp8 release (or yet to be >> released version on github) that satisfies that requirement. >> >> ____ >> From: So, Byoungro >> Sent: Thursday, May 14, 2015 3:39 AM >> To: dev@cordova.apache.org >> Subject: RE: [DISCUSS] Dropping the -dev suffix >> >> Sorry, I got the other way around. :) >> 4.0.0-dev is > 4.0.0. >> >> Byoungro So >> SSG / DPD / Mobile Computing and Compilers >> Intel Corporation >> >> -Original Message- >> From: So, Byoungro >> Sent: Wednesday, May 13, 2015 10:37 AM >> To: dev@cordova.apache.org >> Subject: RE: [DISCUSS] Dropping the -dev suffix >> >> Semver should be able to understand '4.0.0-dev' as the valid version >> string. >> There could be some other requirement problems, like <=, >. >> For example, 4.0.0-dev is < 4.0.0. >> So, if the requirement is >=4.0.0, then the version string '4.0.0-dev' >> will fail to satisfy the requirement. >> >> Byoungro So >> SSG / DPD / Mobile Computing and Compilers Intel Corporation >> >> -Original Message- >> From: Steven Gill [mailto:stevengil...@gmail.com] >> Sent: Wednesday, May 13, 2015 10:21 AM >> To: dev@cordova.apache.org >> Subject: [DISCUSS] Dropping the -dev suffix >> >> Recently, a issue (CB-9021) popped up where mobile spec was failing >> because the semver check failed on a platform due to `-dev` being in the >> version and not valid. >> >> I think we should remove -dev from our versions. Right now, after we do a >> release, we add -dev to show that the repo is in development and hasn't >> been released. I think we can accomplish the same by just bumping the >> version (minor?) in the package.json and just know it isn't released by >> seeing what is published on npm. >> >> Thoughts? >> >> - >> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org >> For additional commands, e-mail: dev-h...@cordova.apache.org >> >> >> - >> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org >> For additional commands, e-mail: dev-h...@cordova.apache.org >> >> >
Re: [DISCUSS] Dropping the -dev suffix
switching engine to >=4.0 (no patch version) seems to make the most sense for this problem. On Wed, May 13, 2015 at 4:21 PM, Jesse wrote: > Yes, I just verified the same. > > Changing the plugin.xml engine to : > cordova-wp8 >=4.0.0-- will allow install for both wp8@4.0.0 and > wp8@4.0.0-dev > > @purplecabbage > risingj.com > > On Wed, May 13, 2015 at 4:18 PM, Tim Barham > wrote: > > > No, you were correct - any sort of pre-release identifier makes the > > version less than the version without the pre-release identifier (since > it > > signifies a pre-release). > > > > Hence '4.0.0-dev', '4.0.0-alpha', '4.0.0-0.0.1' and > > '4.0.0-iwishiwasarealreease' are all < 4.0.0. > > > > It seems to me the problem is with the plugin since it requires > > cordova-wp8 >=4.0.0, and there is no cordova-wp8 release (or yet to be > > released version on github) that satisfies that requirement. > > > > ________________ > > From: So, Byoungro > > Sent: Thursday, May 14, 2015 3:39 AM > > To: dev@cordova.apache.org > > Subject: RE: [DISCUSS] Dropping the -dev suffix > > > > Sorry, I got the other way around. :) > > 4.0.0-dev is > 4.0.0. > > > > Byoungro So > > SSG / DPD / Mobile Computing and Compilers > > Intel Corporation > > > > -Original Message- > > From: So, Byoungro > > Sent: Wednesday, May 13, 2015 10:37 AM > > To: dev@cordova.apache.org > > Subject: RE: [DISCUSS] Dropping the -dev suffix > > > > Semver should be able to understand '4.0.0-dev' as the valid version > > string. > > There could be some other requirement problems, like <=, >. > > For example, 4.0.0-dev is < 4.0.0. > > So, if the requirement is >=4.0.0, then the version string '4.0.0-dev' > > will fail to satisfy the requirement. > > > > Byoungro So > > SSG / DPD / Mobile Computing and Compilers Intel Corporation > > > > -Original Message- > > From: Steven Gill [mailto:stevengil...@gmail.com] > > Sent: Wednesday, May 13, 2015 10:21 AM > > To: dev@cordova.apache.org > > Subject: [DISCUSS] Dropping the -dev suffix > > > > Recently, a issue (CB-9021) popped up where mobile spec was failing > > because the semver check failed on a platform due to `-dev` being in the > > version and not valid. > > > > I think we should remove -dev from our versions. Right now, after we do a > > release, we add -dev to show that the repo is in development and hasn't > > been released. I think we can accomplish the same by just bumping the > > version (minor?) in the package.json and just know it isn't released by > > seeing what is published on npm. > > > > Thoughts? > > > > - > > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > > For additional commands, e-mail: dev-h...@cordova.apache.org > > > > > > - > > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > > For additional commands, e-mail: dev-h...@cordova.apache.org > > > > >
Re: [DISCUSS] Dropping the -dev suffix
Yes, I just verified the same. Changing the plugin.xml engine to : cordova-wp8 >=4.0.0-- will allow install for both wp8@4.0.0 and wp8@4.0.0-dev @purplecabbage risingj.com On Wed, May 13, 2015 at 4:18 PM, Tim Barham wrote: > No, you were correct - any sort of pre-release identifier makes the > version less than the version without the pre-release identifier (since it > signifies a pre-release). > > Hence '4.0.0-dev', '4.0.0-alpha', '4.0.0-0.0.1' and > '4.0.0-iwishiwasarealreease' are all < 4.0.0. > > It seems to me the problem is with the plugin since it requires > cordova-wp8 >=4.0.0, and there is no cordova-wp8 release (or yet to be > released version on github) that satisfies that requirement. > > > From: So, Byoungro > Sent: Thursday, May 14, 2015 3:39 AM > To: dev@cordova.apache.org > Subject: RE: [DISCUSS] Dropping the -dev suffix > > Sorry, I got the other way around. :) > 4.0.0-dev is > 4.0.0. > > Byoungro So > SSG / DPD / Mobile Computing and Compilers > Intel Corporation > > -----Original Message----- > From: So, Byoungro > Sent: Wednesday, May 13, 2015 10:37 AM > To: dev@cordova.apache.org > Subject: RE: [DISCUSS] Dropping the -dev suffix > > Semver should be able to understand '4.0.0-dev' as the valid version > string. > There could be some other requirement problems, like <=, >. > For example, 4.0.0-dev is < 4.0.0. > So, if the requirement is >=4.0.0, then the version string '4.0.0-dev' > will fail to satisfy the requirement. > > Byoungro So > SSG / DPD / Mobile Computing and Compilers Intel Corporation > > -Original Message- > From: Steven Gill [mailto:stevengil...@gmail.com] > Sent: Wednesday, May 13, 2015 10:21 AM > To: dev@cordova.apache.org > Subject: [DISCUSS] Dropping the -dev suffix > > Recently, a issue (CB-9021) popped up where mobile spec was failing > because the semver check failed on a platform due to `-dev` being in the > version and not valid. > > I think we should remove -dev from our versions. Right now, after we do a > release, we add -dev to show that the repo is in development and hasn't > been released. I think we can accomplish the same by just bumping the > version (minor?) in the package.json and just know it isn't released by > seeing what is published on npm. > > Thoughts? > > - > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > For additional commands, e-mail: dev-h...@cordova.apache.org > > > - > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > For additional commands, e-mail: dev-h...@cordova.apache.org > >
Re: [DISCUSS] Dropping the -dev suffix
No, you were correct - any sort of pre-release identifier makes the version less than the version without the pre-release identifier (since it signifies a pre-release). Hence '4.0.0-dev', '4.0.0-alpha', '4.0.0-0.0.1' and '4.0.0-iwishiwasarealreease' are all < 4.0.0. It seems to me the problem is with the plugin since it requires cordova-wp8 >=4.0.0, and there is no cordova-wp8 release (or yet to be released version on github) that satisfies that requirement. From: So, Byoungro Sent: Thursday, May 14, 2015 3:39 AM To: dev@cordova.apache.org Subject: RE: [DISCUSS] Dropping the -dev suffix Sorry, I got the other way around. :) 4.0.0-dev is > 4.0.0. Byoungro So SSG / DPD / Mobile Computing and Compilers Intel Corporation -Original Message- From: So, Byoungro Sent: Wednesday, May 13, 2015 10:37 AM To: dev@cordova.apache.org Subject: RE: [DISCUSS] Dropping the -dev suffix Semver should be able to understand '4.0.0-dev' as the valid version string. There could be some other requirement problems, like <=, >. For example, 4.0.0-dev is < 4.0.0. So, if the requirement is >=4.0.0, then the version string '4.0.0-dev' will fail to satisfy the requirement. Byoungro So SSG / DPD / Mobile Computing and Compilers Intel Corporation -Original Message- From: Steven Gill [mailto:stevengil...@gmail.com] Sent: Wednesday, May 13, 2015 10:21 AM To: dev@cordova.apache.org Subject: [DISCUSS] Dropping the -dev suffix Recently, a issue (CB-9021) popped up where mobile spec was failing because the semver check failed on a platform due to `-dev` being in the version and not valid. I think we should remove -dev from our versions. Right now, after we do a release, we add -dev to show that the repo is in development and hasn't been released. I think we can accomplish the same by just bumping the version (minor?) in the package.json and just know it isn't released by seeing what is published on npm. Thoughts? - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
Re: [DISCUSS] Dropping the -dev suffix
It is and it is failing currently. In the file-transfer plugin [1] ... Failed to install 'cordova-plugin-file-transfer':CordovaError: Plugin doesn't support this project's cordova-wp8 version. cordova-wp8: 4.0.0-dev, failed version requirement: 1.0.0 ... which fails in plugman install [2] On closer inspection, this appears to be an issue with plugman, and not our use of semver. I'll look more. [1] https://github.com/apache/cordova-plugin-file-transfer/blob/master/plugin.xml#L36 [2] https://github.com/apache/cordova-lib/blob/master/cordova-lib/src/plugman/install.js#L108 @purplecabbage risingj.com On Wed, May 13, 2015 at 1:33 PM, Andrew Grieve wrote: > I think -dev is working fine and doing what it's supposed to do. The logic > of -label is subtle: > > > var sem = require('semver') > undefined > > sem.satisfies('4.0.0-dev', '>=4.0.0') > false > > sem.satisfies('4.0.0-dev', '>=4.0') > true > > So, the engine check should just to >=4.0, or >=4.0.0-dev > > On Wed, May 13, 2015 at 1:39 PM, So, Byoungro > wrote: > > > Sorry, I got the other way around. :) > > 4.0.0-dev is > 4.0.0. > > > > Byoungro So > > SSG / DPD / Mobile Computing and Compilers > > Intel Corporation > > > > -Original Message- > > From: So, Byoungro > > Sent: Wednesday, May 13, 2015 10:37 AM > > To: dev@cordova.apache.org > > Subject: RE: [DISCUSS] Dropping the -dev suffix > > > > Semver should be able to understand '4.0.0-dev' as the valid version > > string. > > There could be some other requirement problems, like <=, >. > > For example, 4.0.0-dev is < 4.0.0. > > So, if the requirement is >=4.0.0, then the version string '4.0.0-dev' > > will fail to satisfy the requirement. > > > > Byoungro So > > SSG / DPD / Mobile Computing and Compilers Intel Corporation > > > > -Original Message- > > From: Steven Gill [mailto:stevengil...@gmail.com] > > Sent: Wednesday, May 13, 2015 10:21 AM > > To: dev@cordova.apache.org > > Subject: [DISCUSS] Dropping the -dev suffix > > > > Recently, a issue (CB-9021) popped up where mobile spec was failing > > because the semver check failed on a platform due to `-dev` being in the > > version and not valid. > > > > I think we should remove -dev from our versions. Right now, after we do a > > release, we add -dev to show that the repo is in development and hasn't > > been released. I think we can accomplish the same by just bumping the > > version (minor?) in the package.json and just know it isn't released by > > seeing what is published on npm. > > > > Thoughts? > > > > - > > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > > For additional commands, e-mail: dev-h...@cordova.apache.org > > > > >
Re: [DISCUSS] Dropping the -dev suffix
I think -dev is working fine and doing what it's supposed to do. The logic of -label is subtle: > var sem = require('semver') undefined > sem.satisfies('4.0.0-dev', '>=4.0.0') false > sem.satisfies('4.0.0-dev', '>=4.0') true So, the engine check should just to >=4.0, or >=4.0.0-dev On Wed, May 13, 2015 at 1:39 PM, So, Byoungro wrote: > Sorry, I got the other way around. :) > 4.0.0-dev is > 4.0.0. > > Byoungro So > SSG / DPD / Mobile Computing and Compilers > Intel Corporation > > -Original Message- > From: So, Byoungro > Sent: Wednesday, May 13, 2015 10:37 AM > To: dev@cordova.apache.org > Subject: RE: [DISCUSS] Dropping the -dev suffix > > Semver should be able to understand '4.0.0-dev' as the valid version > string. > There could be some other requirement problems, like <=, >. > For example, 4.0.0-dev is < 4.0.0. > So, if the requirement is >=4.0.0, then the version string '4.0.0-dev' > will fail to satisfy the requirement. > > Byoungro So > SSG / DPD / Mobile Computing and Compilers Intel Corporation > > -Original Message- > From: Steven Gill [mailto:stevengil...@gmail.com] > Sent: Wednesday, May 13, 2015 10:21 AM > To: dev@cordova.apache.org > Subject: [DISCUSS] Dropping the -dev suffix > > Recently, a issue (CB-9021) popped up where mobile spec was failing > because the semver check failed on a platform due to `-dev` being in the > version and not valid. > > I think we should remove -dev from our versions. Right now, after we do a > release, we add -dev to show that the repo is in development and hasn't > been released. I think we can accomplish the same by just bumping the > version (minor?) in the package.json and just know it isn't released by > seeing what is published on npm. > > Thoughts? > > - > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > For additional commands, e-mail: dev-h...@cordova.apache.org > >
RE: [DISCUSS] Dropping the -dev suffix
Sorry, I got the other way around. :) 4.0.0-dev is > 4.0.0. Byoungro So SSG / DPD / Mobile Computing and Compilers Intel Corporation -Original Message- From: So, Byoungro Sent: Wednesday, May 13, 2015 10:37 AM To: dev@cordova.apache.org Subject: RE: [DISCUSS] Dropping the -dev suffix Semver should be able to understand '4.0.0-dev' as the valid version string. There could be some other requirement problems, like <=, >. For example, 4.0.0-dev is < 4.0.0. So, if the requirement is >=4.0.0, then the version string '4.0.0-dev' will fail to satisfy the requirement. Byoungro So SSG / DPD / Mobile Computing and Compilers Intel Corporation -Original Message- From: Steven Gill [mailto:stevengil...@gmail.com] Sent: Wednesday, May 13, 2015 10:21 AM To: dev@cordova.apache.org Subject: [DISCUSS] Dropping the -dev suffix Recently, a issue (CB-9021) popped up where mobile spec was failing because the semver check failed on a platform due to `-dev` being in the version and not valid. I think we should remove -dev from our versions. Right now, after we do a release, we add -dev to show that the repo is in development and hasn't been released. I think we can accomplish the same by just bumping the version (minor?) in the package.json and just know it isn't released by seeing what is published on npm. Thoughts? - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
RE: [DISCUSS] Dropping the -dev suffix
Semver should be able to understand '4.0.0-dev' as the valid version string. There could be some other requirement problems, like <=, >. For example, 4.0.0-dev is < 4.0.0. So, if the requirement is >=4.0.0, then the version string '4.0.0-dev' will fail to satisfy the requirement. Byoungro So SSG / DPD / Mobile Computing and Compilers Intel Corporation -Original Message- From: Steven Gill [mailto:stevengil...@gmail.com] Sent: Wednesday, May 13, 2015 10:21 AM To: dev@cordova.apache.org Subject: [DISCUSS] Dropping the -dev suffix Recently, a issue (CB-9021) popped up where mobile spec was failing because the semver check failed on a platform due to `-dev` being in the version and not valid. I think we should remove -dev from our versions. Right now, after we do a release, we add -dev to show that the repo is in development and hasn't been released. I think we can accomplish the same by just bumping the version (minor?) in the package.json and just know it isn't released by seeing what is published on npm. Thoughts? - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org