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

on cygwin build error #477

Open
pedoc opened this issue Jun 4, 2023 · 1 comment
Open

on cygwin build error #477

pedoc opened this issue Jun 4, 2023 · 1 comment

Comments

@pedoc
Copy link

pedoc commented Jun 4, 2023

version:

cat /proc/version
CYGWIN_NT-10.0-25375 version 3.4.6-1.x86_64 (corinna@calimero) (gcc version 11.3.0 20220421 (Fedora Cygwin 11.3.0-1) (GCC) ) 2023-02-14 13:23 UTC

On recent versions of cygwin, the icmpv6_echo_reply_lh typedef already exists.

$ grep -r icmpv6_echo_reply_lh /usr/include
/usr/include/w32api/ipexport.h:typedef struct icmpv6_echo_reply_lh {

ipexport.h(L105-102)

typedef struct icmpv6_echo_reply_lh {
  IPV6_ADDRESS_EX Address;
  ULONG Status;
  unsigned int RoundTripTime;
} ICMPV6_ECHO_REPLY_LH, *PICMPV6_ECHO_REPLY_LH;

typedef ICMPV6_ECHO_REPLY_LH ICMPV6_ECHO_REPLY;
typedef ICMPV6_ECHO_REPLY_LH *PICMPV6_ECHO_REPLY;

This leads to duplication of definitions in packet\probe_cygwin.h

If you delete the relevant definitions in /usr/include/w32api/ipexport.h, it can be compiled normally, but if you delete the definitions in packet\probe_cygwin.h, it fails.

@apschultz
Copy link

Slightly easier change that doesn't involve hacking standard headers:

#465 (comment)

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

No branches or pull requests

2 participants