Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update VS projects #47

Closed
wants to merge 16 commits into from
Closed

Update VS projects #47

wants to merge 16 commits into from

Conversation

eugeneopalev
Copy link
Contributor

@eugeneopalev eugeneopalev commented Mar 26, 2019

Changes summary:

  • Merge libogg_static.sln and libogg_dynamic.sln solution files into single libogg.sln
  • Create proper configurations for DLL builds (DebugDLL and ReleaseDLL)
  • Also create same solutions for VS2017 and VS2019

Now you can build libogg on Windows out of the box without any problems.

@eugeneopalev eugeneopalev changed the title Update vs projects Update VS projects Mar 26, 2019
@eugeneopalev eugeneopalev marked this pull request as ready for review March 30, 2019 19:45
@rillian
Copy link
Contributor

rillian commented Aug 12, 2019

Wow, thanks for all the work!

I don't use Visual Studio, so a couple of quick questions to better understand your intent here:

  1. We have cmake support now, which some people say is their preferred way of building unix packages for Windows. Do we still need project files?

  2. Visual Studio can generally open older project files. What's the advantage of maintaining three sets of project files? Can they all be tested on appveyor?

@willson-chen
Copy link
Contributor

@rillian agree with your opinion. CMake is a better way, while win32 folder is out of date. Why not consider about removing win32 folder?

@eugeneopalev
Copy link
Contributor Author

@rillian Maintaining three project files is a little too much, I agree. They basically all the same. Should I just leave VS2015 project or remove win32 folder completely and rely on CMake? I think the second variant is more preferable.

@rillian
Copy link
Contributor

rillian commented Aug 23, 2019 via email

@willson-chen
Copy link
Contributor

Ok great. How about:

- For this PR, just leave the VS2015 project file. I'll merge that for the 1.3.4 release next week.
- Prepare another PR which removes the win32 folder entirely. I'll merge that after 1.3.4 is out.

That way there's at least one release where the Visual Studio project files are marked deprecated.

Sound reasonable?

Agree. It is a reasonable way of leaving some time for vs develops who use win32 to transfer to cmake,

@eugeneopalev
Copy link
Contributor Author

@rillian Removed VS2017 and VS2019 projects. Also removed DLL configurations from appveyor.yml. Don't think that setup complex build matrix with DLLs for VS project is a good idea if it will be removed anyway.

@rillian
Copy link
Contributor

rillian commented Aug 30, 2019

I've rebased and and pushed as 6d55ddf. Thanks for the cleanup!

@rillian rillian closed this Aug 30, 2019
@eugeneopalev eugeneopalev deleted the update-vs-projects branch September 24, 2019 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants