Skip to content
This repository has been archived by the owner on Apr 17, 2022. It is now read-only.

Compile natively with MSVC2010 #1881

Closed
wzdev-ci opened this issue May 27, 2010 · 12 comments
Closed

Compile natively with MSVC2010 #1881

wzdev-ci opened this issue May 27, 2010 · 12 comments
Labels
closed component: Build System Issues normal
Milestone
unspecified

Comments

@wzdev-ci
Copy link
Contributor

resolution_fixed type_patch (an actual patch, not a request for one) | by SonOfBerg <dmarney@...>

Attached is a patch that will allow WZ2100 to be compiled natively in visual studio 2010. Please note that only the 'release' build works at this time.

Several changes had to be made to math headers in order to allow it to compile natively, I have tested this and can confirm that it still compiles under Linux, I have not tested it on MacOSx.

I have also recompiled several of the devpkg libraries to allow this to work.

Issue migrated from trac:1881 at 2022-04-15 21:59:07 -0700
@wzdev-ci
Copy link
Contributor Author

SonOfBerg <dmarney@...> uploaded file msvc2010.diff (71.5 KiB)

svn diff file

@wzdev-ci
Copy link
Contributor Author

SonOfBerg <dmarney@...> uploaded file devpkg-trunk-msvc2010-20102705.rar (4534.4 KiB)

msvc2010 compatible devpkg

@wzdev-ci
Copy link
Contributor Author

sonofberg uploaded file msvc2010-working.diff (69.6 KiB)

@wzdev-ci
Copy link
Contributor Author

sonofberg uploaded file lib.rar (11.2 KiB)

Fixed msvc project files.

@wzdev-ci
Copy link
Contributor Author

sonofberg uploaded file win32.rar (8.6 KiB)

@wzdev-ci
Copy link
Contributor Author

BlueMaxima uploaded file msvc2010-fixer-v2.patch (66.3 KiB)

@wzdev-ci
Copy link
Contributor Author

sonofberg uploaded file msvc2010-v3.patch (150.5 KiB)

Updated patch fixes a few bugs.

@wzdev-ci
Copy link
Contributor Author

Buginator commented

The VS2k10 file needs to be a new file, and not touch the old files at all.
(We still use VS2k5, Vs2k8)

Why the addition of the **|| defined(WIN32) ** ?

That could break our CC / mingw builds.

@wzdev-ci
Copy link
Contributor Author

Wolftrak commented

Berg,can you build a working patch that doesn't break the other builds? I've spend a day working around the errors and I still can't get it to work.

@wzdev-ci
Copy link
Contributor Author

Berg commented

Note

Reported by: SonOfBerg <dmarney@…> Sorry wolftrak but this patch was made by my son who is busy with real life and no longer involved with wz2100.net

@wzdev-ci
Copy link
Contributor Author

Per changed resolution from `` to fixed

@wzdev-ci
Copy link
Contributor Author

Per changed status from new to closed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
closed component: Build System Issues normal
Projects
None yet
Milestone
unspecified
Development

No branches or pull requests
1 participant
@wzdev-ci