Wireshark-bugs: [Wireshark-bugs] [Bug 3500] Add Python embedding to Windows makefiles
Date: Fri, 10 Jun 2011 20:23:29 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3500

--- Comment #16 from Michael Mann <mmann78@xxxxxxxxxxxx> 2011-06-10 20:23:28 PDT ---
(In reply to comment #15)
> OK, try rev 37650 without the changes to link dumpcap with the Python library;
> they change version_info.c not to include <Python.h>.
> Then check whether, for Wireshark or TShark, the version information includes
> the Python version number; epan/epan.c, which adds the Python information,
> didn't, and still doesn't, include <Python.h> - perhaps it needs to.

That works!   I did a "rebuild all" ("distclean", then "all") and didn't get
any errors.  

My patch can be ignored as I don't think its worth updating for a single line
of code (uncommenting out PYTHON_EMBED).  There are comments around 
PYTHON_EMBED that seem unnecessary at this point as long as Python 2.7 is the
standard, but I'll let core developers decide if they should be removed.

The wiki needs updating to note that Python scripting is now possible in
Windows, but I'd otherwise consider this resolved.

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