Skip to content

Conversation

cdunn2001
Copy link
Contributor

re: #242

Test auto appveyor run...

@cdunn2001 cdunn2001 mentioned this pull request Apr 17, 2015
@cdunn2001
Copy link
Contributor Author

Seems to work! But it takes 1/2 hour.

I guess we should try to fix these someday:

Build succeeded.

"C:\projects\jsoncpp\jsoncpp.sln" (default target) (1) ->
"C:\projects\jsoncpp\ALL_BUILD.vcxproj.metaproj" (default target) (2) ->
"C:\projects\jsoncpp\src\lib_json\jsoncpp_lib_static.vcxproj.metaproj" (default target) (4) ->
"C:\projects\jsoncpp\src\lib_json\jsoncpp_lib_static.vcxproj" (default target) (5) ->
(ClCompile target) -> 
  C:\projects\jsoncpp\src\lib_json\json_value.cpp(128): warning C4267: '=' : conversion from 'size_t' to 'unsigned int', possible loss of data [C:\projects\jsoncpp\src\lib_json\jsoncpp_lib_static.vcxproj]
  C:\projects\jsoncpp\src\lib_json\json_value.cpp(162): warning C4512: 'Json::Exception' : assignment operator could not be generated [C:\projects\jsoncpp\src\lib_json\jsoncpp_lib_static.vcxproj]
  C:\projects\jsoncpp\src\lib_json\json_value.cpp(166): warning C4512: 'Json::RuntimeError' : assignment operator could not be generated [C:\projects\jsoncpp\src\lib_json\jsoncpp_lib_static.vcxproj]
  C:\projects\jsoncpp\src\lib_json\json_value.cpp(170): warning C4512: 'Json::LogicError' : assignment operator could not be generated [C:\projects\jsoncpp\src\lib_json\jsoncpp_lib_static.vcxproj]
  C:\projects\jsoncpp\src\lib_json\json_writer.cpp(349): warning C4267: 'argument' : conversion from 'size_t' to 'unsigned int', possible loss of data [C:\projects\jsoncpp\src\lib_json\jsoncpp_lib_static.vcxproj]
  C:\projects\jsoncpp\src\lib_json\json_writer.cpp(909): warning C4267: 'argument' : conversion from 'size_t' to 'unsigned int', possible loss of data [C:\projects\jsoncpp\src\lib_json\jsoncpp_lib_static.vcxproj]

    6 Warning(s)
    0 Error(s)

@marklakata had a fix for some of those already.

cdunn2001 added a commit that referenced this pull request Apr 19, 2015
New `appveyor.yml`: All tests pass, in both Appveyor and Travis!

Henceforth, GitHub will run both for any pull-request, so this file will be needed in the `0.y.z` branch too.
@cdunn2001 cdunn2001 merged commit 441f8cd into open-source-parsers:master Apr 19, 2015
@cdunn2001 cdunn2001 deleted the appveyor branch April 19, 2015 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants