Skip to content

Watcom Portability Improvements - #509

Merged
bradh352 merged 2 commits into
c-ares:mainfrom
renodr:master
Feb 8, 2023
Merged

Watcom Portability Improvements#509
bradh352 merged 2 commits into
c-ares:mainfrom
renodr:master

Conversation

@renodr

@renodr renodr commented Feb 7, 2023

Copy link
Copy Markdown
Contributor
  • Modify the Watcom Makefile for the source code reorganization (Makefile.Watcom (WIN16?) broken with source tree reorganization #352)
  • Add *.map files into .gitignore
  • Fix build errors with Watcom's builtin Windows SDK (which is rather outdated). It's smart enough to understand Windows Vista, but doesn't have PMIB_UNICASTIPADDRESS_TABLE or MIB_IPFORWARD_ROW2.

It may be possible to use a different Windows SDK with the Watcom compiler, such as the most recent Windows 10 SDK. Alternatively the SDK in OpenWatcom 2.0 (which is in development) should fix this.

I have no problems testing this Makefile prior to releases, just give me a ping.

Tested with Windows Vista, Windows 7, and Windows 10 using 'adig', 'acountry', and 'ahost'. This also seems to work on Windows XP, though this is likely due to the compiler in use.

- Modify the Watcom Makefile for the source code reorganization (c-ares#352)
- Add *.map files into .gitignore
- Fix build errors with Watcom's builtin Windows SDK (which is rather
  outdated). It's smart enough to understand Windows Vista, but doesn't
  have PMIB_UNICASTIPADDRESS_TABLE or MIB_IPFORWARD_ROW2.

It may be possible to use a different Windows SDK with the Watcom
compiler, such as the most recent Windows 10 SDK. Alternatively the SDK
in OpenWatcom 2.0 (which is in development) should fix this.

I have no problems testing this Makefile prior to releases, just give me
a ping.

Tested with Windows Vista, Windows 7, and Windows 10 using 'adig',
'acountry', and 'ahost'. This also seems to work on Windows XP, though
this is likely due to the compiler in use.
@bradh352
bradh352 merged commit 761c618 into c-ares:main Feb 8, 2023
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.

2 participants