Builds broken: "Platform" environment no longer exists

Brian Reavis's Avatar

Brian Reavis

30 Jul, 2014 03:16 AM

In the last day or two, the Platform environment variable disappeared... and this is breaking builds. This needs to exist to test for x64, x86 build configurations.

Affected repos:
https://github.com/naturalatlas/node-gdal
https://github.com/conda/conda
https://github.com/mapnik/node-mapnik

Current ENV variables: https://gist.github.com/brianreavis/c9350994a5a4e5bfb6f9

  1. 1 Posted by micah on 30 Jul, 2014 04:08 AM

    micah's Avatar

    I am experiencing the same problem. My appveyor.yml looks something like:

    platform:
      - AnyCPU
    
    configuration:
      - Release
    

    When I attempt to build https://ci.appveyor.com/project/Zoltu/zoltu-wrapper (only change was to readme text) I receive error MSB4126: The specified solution configuration "Release|AnyCPU" is invalid. It would appear that the configuration and platform are both being supplied to MSBuild as the configuration, concatenated with a |.

    Again looking at the build output, it appears that the platform/configuration isn't supplied via command line, so I am not sure how appveyor is providing these variables so I can't really debug further.

    "C:\Program Files (x86)\MSBuild\12.0\bin\msbuild.exe" "Zoltu.Wrapper.sln" /logger:"C:\Program Files\AppVeyor\BuildAgent\Appveyor.MSBuildLogger.dll"

  2. Support Staff 2 Posted by Feodor Fitsner on 30 Jul, 2014 05:14 AM

    Feodor Fitsner's Avatar

    Will fix that shortly, sorry for that regression.

  3. Support Staff 3 Posted by Feodor Fitsner on 30 Jul, 2014 05:51 AM

    Feodor Fitsner's Avatar

    @micah, AppVeyor supplies those values through environment variables. You can override any variable within MSBuild process through environment variables.

  4. Support Staff 4 Posted by Feodor Fitsner on 30 Jul, 2014 05:54 AM

    Feodor Fitsner's Avatar

    ...and there is no bug in relation to MSBuild. it's just AnyCPU is the platform that doesn't exist in your solution. Correct one is Any CPU (with space):

    platform:
      - Any CPU
    
    configuration:
      - Release
    
  5. Support Staff 5 Posted by Feodor Fitsner on 30 Jul, 2014 12:33 PM

    Feodor Fitsner's Avatar

    It's back - original behavior of platform and configuration parameters for non-MSBuild scenarios.

  6. 6 Posted by Daniele E. Dome... on 30 Jul, 2014 02:25 PM

    Daniele E. Domenichelli's Avatar

    I have a CMake project that I am building with a configuration similar to this:

    platform:
      - Win32
      - x64
    
    before_build:
      - cmd: if "%platform%"=="Win32" cmake -G"Visual Studio 12"  ..
      - cmd: if "%platform%"=="x64" cmake -G"Visual Studio 12 Win64" ..
    

    It was working before, but now (last build 22 minutes ago) none of the above cmake command is executed, and therefore the build fails after a few seconds since the solution file is not created.

  7. Support Staff 7 Posted by Feodor Fitsner on 30 Jul, 2014 02:28 PM

    Feodor Fitsner's Avatar

    Will fix that shortly.

  8. Support Staff 8 Posted by Feodor Fitsner on 30 Jul, 2014 03:33 PM

    Feodor Fitsner's Avatar

    Should be fine now.

  9. 9 Posted by Daniele E. Dome... on 30 Jul, 2014 04:31 PM

    Daniele E. Domenichelli's Avatar

    Thanks!

    Unfortunately now it looks like that the before_build step is no longer performed

  10. Support Staff 10 Posted by Feodor Fitsner on 30 Jul, 2014 04:45 PM

    Feodor Fitsner's Avatar
  11. 11 Posted by Daniele E. Dome... on 30 Jul, 2014 04:57 PM

    Daniele E. Domenichelli's Avatar

    Failed for both Win32 and x64 builds:

    https://ci.appveyor.com/project/drdanz/ycm/build/job/brw96b623wij26ua
    https://ci.appveyor.com/project/drdanz/ycm/build/job/sc26eonasufh0xmv

    The before_build step seems to be skipped, this was the output of the previous build
    https://ci.appveyor.com/project/drdanz/ycm/build/0.0.0.169/job/wgb7...
    showing in cyan the before_build commands.

    the cmake to go straight to the build step...

  12. 12 Posted by Daniele E. Dome... on 30 Jul, 2014 04:59 PM

    Daniele E. Domenichelli's Avatar
  13. Support Staff 13 Posted by Feodor Fitsner on 30 Jul, 2014 05:38 PM

    Feodor Fitsner's Avatar

    OK, I see. Just deployed a fix.

  14. 14 Posted by Daniele E. Dome... on 30 Jul, 2014 05:57 PM

    Daniele E. Domenichelli's Avatar

    The build started now.
    Thanks a lot!

  15. 15 Posted by Brian Reavis on 30 Jul, 2014 06:02 PM

    Brian Reavis's Avatar

    Thanks Feodor!

  16. Ilya Finkelshteyn closed this discussion on 25 Aug, 2018 01:46 AM.

Comments are currently closed for this discussion. You can start a new one.

Keyboard shortcuts

Generic

? Show this help
ESC Blurs the current field

Comment Form

r Focus the comment reply box
^ + ↩ Submit the comment

You can use Command ⌘ instead of Control ^ on Mac