cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Grebnov (Akvelon)" <v-seg...@microsoft.com>
Subject RE: [Vote] 3.5.0 WP8 Release
Date Thu, 22 May 2014 05:59:56 GMT
Pushed fix https://github.com/apache/cordova-wp8/pull/36 

Thx!
Sergey
-----Original Message-----
From: Sergey Grebnov (Akvelon) [mailto:v-segreb@microsoft.com] 
Sent: Thursday, May 22, 2014 8:46 AM
To: dev@cordova.apache.org
Subject: RE: [Vote] 3.5.0 WP8 Release

Created issue for this, working on fix..

https://issues.apache.org/jira/browse/CB-6732
[WP8] Fix "MSBuild 64 bit is not supported" build error

Thx!
Sergey
-----Original Message-----
From: Stefan Dobrev [mailto:stefan.dobrev@gmail.com]
Sent: Thursday, May 22, 2014 7:30 AM
To: dev@cordova.apache.org
Subject: Re: [Vote] 3.5.0 WP8 Release

The problem comes from the bitness of the process you are executing the command from. I guess
that you have 64bit Node.js installation and that's why `cordova build wp8` does not works
for you, but calling directly the platform script works. It all boils down to the registry
key setting for MSBuildToolsPath which is:

   - for 64-bit process (
   *HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\MSBuild\ToolsVersions\12.0*) -
   C:\Program Files (x86)\MSBuild\12.0\bin\amd64\
   - and shimmed for 32bit process on 64bit architectures (
   *HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Microsoft\MSBuild\ToolsVersions\12.0*)
   - C:\Program Files (x86)\MSBuild\12.0\bin\

I have VS 2013 Update 2 on my machine. I do not have any other VS installations.

Hope this helps.




2014-05-22 4:56 GMT+03:00 Jesse <purplecabbage@gmail.com>:

> I think I have narrowed this down to something in superspawn, not sure 
> what though.
>
> When calling ```reg query``` we are getting different values based on 
> the way the .bat file is called
>
> c:\Test350> platforms\wp8\cordova\build
> MSBuildToolsPath: C:\Program Files (x86)\MSBuild\12.0\bin\ <= GOOD
>
> c:\Test350> cordova build wp8
> MSBuildToolsPath: C:\Program Files (x86)\MSBuild\12.0\bin\amd64\ <== 
> NOT GOOD, the x64 version will not work.
>
>
>
>
> @purplecabbage
> risingj.com
>
>
> On Wed, May 21, 2014 at 4:27 PM, Steven Gill <stevengill97@gmail.com>
> wrote:
>
> > Another update,
> >
> > Works for me:
> > cordova build wp8
> >
> > this does not:
> > platforms/wp8/cordova/build
> >
> > Yes, this is the opposite of what Jesse is experiencing. I am using
> windows
> > 8 not 8.1. I get the same error as Jesse posted above.
> >
> >
> > On Wed, May 21, 2014 at 3:46 PM, Jesse <purplecabbage@gmail.com> wrote:
> >
> > > Update:
> > > the regression is in cordova-cli or cordova-lib
> > >
> > > This works:
> > > c:\Test350> platforms\wp8\cordova\build
> > >
> > > and this does not:
> > > c:\Test350> cordova build wp8
> > >
> > >
> > >
> > > @purplecabbage
> > > risingj.com
> > >
> > >
> > > On Wed, May 21, 2014 at 3:26 PM, Jesse <purplecabbage@gmail.com>
> wrote:
> > >
> > > > Windows Phone 8 has a regression, I am not sure how to resolve.
> > > > Our change to not require msbuild in the user environment path 
> > > > is
> > failing
> > > > on my machine.
> > > >
> > > > This could be the result of using windows 8.1 update 1, or WP8.1 
> > > > SDK installer changed the msbuild reg-key path, or VS2013 did ...
> > > >
> > > > Joel, what is your setup?  You said you were able to build/run 
> > > > ... I
> am
> > > > not.
> > > > What is in the reg for :
> > > >
> > HKLM\\SOFTWARE\\Microsoft\\MSBuild\\ToolsVersions\\12.0\\MSBuildTool
> > sPath
> > > > and
> > > >
> > HKLM\\SOFTWARE\\Microsoft\\MSBuild\\ToolsVersions\\4.0\\MSBuildTools
> > Path
> > > >
> > > > My system returns (respectively) :
> > > > C:\Program Files (x86)\MSBuild\12.0\bin\amd64\ and 
> > > > C:\Windows\Microsoft.NET\Framework64\v4.0.30319\
> > > >
> > > > Which results in the following error message:
> > > >
> > > > Building Windows Phone application using MSBuild 64 bit is not
> > supported.
> > > > If you are using TFS build definitions, change the MSBuild 
> > > > platform
> to
> > > x86.
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > @purplecabbage
> > > > risingj.com
> > > >
> > > >
> > > > On Wed, May 21, 2014 at 12:42 PM, Jesse 
> > > > <purplecabbage@gmail.com>
> > wrote:
> > > >
> > > >> reviewing/testing
> > > >>
> > > >> @purplecabbage
> > > >> risingj.com
> > > >>
> > > >>
> > > >> On Wed, May 21, 2014 at 12:37 PM, Steven Gill <
> stevengill97@gmail.com
> > > >wrote:
> > > >>
> > > >>> Bump! Need one more PMC vote for this to pass.
> > > >>>
> > > >>> -Steve
> > > >>>
> > > >>>
> > > >>> On Tue, May 20, 2014 at 6:50 AM, Ian Clelland <
> > iclelland@chromium.org
> > > >>> >wrote:
> > > >>>
> > > >>> > +1
> > > >>> >
> > > >>> > * Verified archive signature and hashes
> > > >>> > * Confirmed that archive contents match repository at the
> > > >>> > 3.5.0
> tag
> > > >>> > * Verified that the release self-identifies as 3.5.0 :)
> > > >>> >
> > > >>> >
> > > >>> > On Fri, May 16, 2014 at 6:43 PM, Joel Kucera <
> joelku@microsoft.com
> > >
> > > >>> wrote:
> > > >>> >
> > > >>> > > +1
> > > >>> > >
> > > >>> > > * verified zip with coho verify-archive
> > > >>> > > * verified repo tag at 3.5.0 matches what is listed
below
> > > >>> > > * manually tested cordova wp8 platform with the CLI,
build 
> > > >>> > > and
> > run
> > > on
> > > >>> > both
> > > >>> > > emulator and wp8 devices, verified platform functionality
> > > >>> > >
> > > >>> > > Joel Kucera
> > > >>> > >
> > > >>> > > -----Original Message-----
> > > >>> > > From: Steven Gill [mailto:stevengill97@gmail.com]
> > > >>> > > Sent: Wednesday, May 14, 2014 3:16 PM
> > > >>> > > To: dev@cordova.apache.org
> > > >>> > > Subject: [Vote] 3.5.0 WP8 Release
> > > >>> > >
> > > >>> > > Please review and vote on this 3.5.0 WP8 Release.
> > > >>> > >
> > > >>> > > Release issue: 
> > > >>> > > https://issues.apache.org/jira/browse/CB-6586
> > > >>> > >
> > > >>> > > Repos ready to be released have been published to dist/dev:
> > > >>> > > https://dist.apache.org/repos/dist/dev/cordova/CB-6586/fin
> > > >>> > > al
> > > >>> > >
> > > >>> > > The package was published from the corresponding git
tag:
> > > >>> > > cordova-wp8: 3.5.0 (59184cbaa0)
> > > >>> > >
> > > >>> > > Upon a successful vote I will upload the archives to

> > > >>> > > dist/,
> > publish
> > > >>> them
> > > >>> > > to NPM, and post the corresponding blog post.
> > > >>> > >
> > > >>> > > Voting guidelines:
> > > >>> > >
> > > >>> >
> > > >>>
> > >
> >
> https://github.com/apache/cordova-coho/blob/master/docs/release-voting
> .md
> > > >>> > >
> > > >>> > > Voting will go on for a minimum of 48 hours.
> > > >>> > >
> > > >>> > > I vote +1:
> > > >>> > > * Verified archive
> > > >>> > > * Ensured continuous build was green when repos were

> > > >>> > > tagged
> > > >>> > >
> > > >>> >
> > > >>>
> > > >>
> > > >>
> > > >
> > >
> >
>
Mime
View raw message