Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 1292] Win32: confusing setup target warning if cl.exe is not in path

[bugzilla-daemon@xxxxxxxxxxxxx]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1292

Chris Maynard <christopher.maynard@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #4579|                            |review_for_checkin?
               Flag|                            |

--- Comment #1 from Chris Maynard <christopher.maynard@xxxxxxxxx> 2010-04-27 13:00:51 PDT ---
Created an attachment (id=4579)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=4579)
Patch to differentiate between native and cygwin apps.

With this patch, the verify_tools (or setup) targets will now complain about
either missing native or missing cygwin tools instead of grouping them
together.

As a test, I renamed cl.exe so it wouldn't be found.  Here's the result (with
my actual $PATH replaced with path1:path2:...

C:\wireshark\vs2008ee>nmake -f Makefile.nmake verify_tools

Microsoft (R) Program Maintenance Utility Version 9.00.30729.01
Copyright (C) Microsoft Corporation.  All rights reserved.

Checking for required native applications:
        link: /cygdrive/c/Program Files/Microsoft Visual Studio 9.0/VC/BIN/link

        mt: /cygdrive/c/Program Files/Microsoft SDKs/Windows/v6.0A/bin/mt
        nmake: /cygdrive/c/Program Files/Microsoft Visual Studio
9.0/VC/BIN/nmak
e

Can't find:  cl

ERROR: These application(s) are either not installed or simply can't be found
in
 the current PATH: path1:path2:...

NMAKE : fatal error U1077: 'C:\cygwin\bin\bash.EXE' : return code '0x1'
Stop.

C:\wireshark\vs2008ee>

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.