Pushed fix https://github.com/apache/cordova-wp8/pull/36
Thx! Sergey -----Original Message----- From: Sergey Grebnov (Akvelon) [mailto:[email protected]] Sent: Thursday, May 22, 2014 8:46 AM To: [email protected] 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:[email protected]] Sent: Thursday, May 22, 2014 7:30 AM To: [email protected] 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 <[email protected]>: > 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 <[email protected]> > 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 <[email protected]> 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 <[email protected]> > 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 > > > > <[email protected]> > > wrote: > > > > > > > >> reviewing/testing > > > >> > > > >> @purplecabbage > > > >> risingj.com > > > >> > > > >> > > > >> On Wed, May 21, 2014 at 12:37 PM, Steven Gill < > [email protected] > > > >wrote: > > > >> > > > >>> Bump! Need one more PMC vote for this to pass. > > > >>> > > > >>> -Steve > > > >>> > > > >>> > > > >>> On Tue, May 20, 2014 at 6:50 AM, Ian Clelland < > > [email protected] > > > >>> >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 < > [email protected] > > > > > > >>> 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:[email protected]] > > > >>> > > Sent: Wednesday, May 14, 2014 3:16 PM > > > >>> > > To: [email protected] > > > >>> > > 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 > > > >>> > > > > > >>> > > > > >>> > > > >> > > > >> > > > > > > > > > >
