Thanks Stefan, and Sergey. That did the trick, all pulled and merged. I have re-tagged the 3.5.x branch 3.5.0 https://git-wip-us.apache.org/repos/asf?p=cordova-wp8.git;a=commit;h=aa2bfb157d1c992de5fd0593eda959284a870cfa
@purplecabbage risingj.com On Wed, May 21, 2014 at 10:59 PM, Sergey Grebnov (Akvelon) < v-seg...@microsoft.com> wrote: > Pushed fix https://github.com/apache/cordova-wp8/pull/36 > > Thx! > Sergey > -----Original Message----- > From: Sergey Grebnov (Akvelon) [mailto:v-seg...@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.dob...@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 <purplecabb...@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 <stevengil...@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 <purplecabb...@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 <purplecabb...@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 > > > > > <purplecabb...@gmail.com> > > > wrote: > > > > > > > > > >> reviewing/testing > > > > >> > > > > >> @purplecabbage > > > > >> risingj.com > > > > >> > > > > >> > > > > >> On Wed, May 21, 2014 at 12:37 PM, Steven Gill < > > stevengil...@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 < > > > iclell...@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 < > > joe...@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:stevengil...@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 > > > > >>> > > > > > > >>> > > > > > >>> > > > > >> > > > > >> > > > > > > > > > > > > > > >
