DevO2012 / gflags

Automatically exported from code.google.com/p/gflags
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Warnings in Visual Studio 2010 and unable to compile unit test #73

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Platform: Windows 7 Ultimate x64
Compiler: MSVC2010

Using the Visual Studio 2010 solution file, the project builds and I 
successfully got a dynamic linked library, but I have a shit load of warnings 
and I was unable to compile the unit test.

Nothing was changed. All I did was checkout the project from SVN (Head 
revision), and opened up the solution file in Visual Studio 2010 and pressed 
Ctrl + Shift + B (Build) on my keyboard.

I attached a file, with a complete copy of my build output. I'm going to 
research the problem to see if I can come up with a solution.

Original issue reported on code.google.com by swenko...@gmail.com on 22 Nov 2013 at 4:16

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks for the report. Would you maybe mind trying the build of the gflags 
library using CMake to generate the Visual Studio project files ? You may 
indeed still get similar errors, but I would like to continue the development 
of gflags based on this migrated source tree. If you find a fix for these 
warnings and errors that would be great. The plan is to replace the current SVN 
trunk by this new Git branch before the end of this year.

See https://github.com/schuhschuh/gflags/tree/cmake-migration

Original comment by andreas....@gmail.com on 23 Nov 2013 at 11:16

GoogleCodeExporter commented 9 years ago
Hi, I'm the same guy who contributed the visual studio fixes on Github 
(@Photonios). I actually just switched my development environment to Windows, 
but I'm setting up a VM for Windows development. I'll give the branch a try 
within a few days. I'll keep your updated.

Original comment by swenko...@gmail.com on 28 Nov 2013 at 8:49

GoogleCodeExporter commented 9 years ago
* switch development environment to Linux

Original comment by swenko...@gmail.com on 28 Nov 2013 at 8:49

GoogleCodeExporter commented 9 years ago
Yes, that would be great. I am actually at the moment without my Windows VM, so 
would have difficulties to test myself. Thus it would be helpful if you try it 
out and let me know what you find.

Original comment by andreas....@gmail.com on 28 Nov 2013 at 11:28

GoogleCodeExporter commented 9 years ago
Any updates on this issue? I have Visual Studio 2013 (12.0) on Windows 7 64-bit 
and I'm getting many build errors. The attached log file is from building the 
"gflags" project on "Release" configuration using the CMake branch.

I would love to help out in getting those issues fixed, but it would be good to 
know if someone else has made some progress so I wouldn't have to duplicate 
effort.

Thanks!

Original comment by islam.os...@gmail.com on 14 Feb 2014 at 1:14

Attachments:

GoogleCodeExporter commented 9 years ago
It seems like nobody looked into these errors and warnings further. I would 
appreciate a patch/pull request for the code in the CMake branch to resolve 
these build errors.

Cheers, Andreas.

Original comment by andreas....@gmail.com on 13 Mar 2014 at 10:02

GoogleCodeExporter commented 9 years ago
I recall looking at those issues when working on the leak fix branch. Maybe it 
was only for VS2012?

Original comment by Massimo...@gmail.com on 13 Mar 2014 at 3:57

GoogleCodeExporter commented 9 years ago
The build of the Git cmake-migration branch should be fixed now for 
Windows/Visual Studio users (see 
https://github.com/schuhschuh/gflags/commit/b2180eb52c961b20c7b79425ecb9f9773317
c7f7).

Warnings regarding the missing export of STL classes are still present. Will 
look into this next. Some information can be found at 
http://support.microsoft.com/default.aspx?scid=KB;EN-US;168958 .

Original comment by andreas....@gmail.com on 14 Mar 2014 at 3:33

GoogleCodeExporter commented 9 years ago
The missing STL export warnings are fixed in the cmake-migration branch.

See 
https://github.com/schuhschuh/gflags/commit/70eac23c1b20da94100b2c62081572d80c47
ef0b

Original comment by andreas....@gmail.com on 14 Mar 2014 at 4:10

GoogleCodeExporter commented 9 years ago
The final fix was to simply not export the CommandLineInfo data only structure 
which is fully defined in the header file. Exporting the STL classes will only 
cause more troubles.

Original comment by andreas....@gmail.com on 20 Mar 2014 at 4:08