| msg185660 - (view) |
Author: Roumen Petrov (rpetrov) * |
Date: 2013-03-31 19:59 |
| split of . |
|
|
| msg266050 - (view) |
Author: Martin Panter (martin.panter) *  |
Date: 2016-05-22 05:15 |
| Would it be better to define all the things in "winreparse.h" with names that are less likely to conflict, say Py_REPARSE_DATA_BUFFER etc? This would be a more general way to avoid conflicts, and also avoid hacking the configure script. Also, is the “#ifndef MAXIMUM_REPARSE_DATA_BUFFER_SIZE” condition necessary? |
|
|
| msg271432 - (view) |
Author: Martin Panter (martin.panter) * |
Date: 2016-07-27 04:18 |
| Here is a patch implementing my suggestion to unconditionally define everything, prefixed with Py_ instead. Not tested on a normal Windows build. |
|
|
| msg271449 - (view) |
Author: Steve Dower (steve.dower) * |
Date: 2016-07-27 13:42 |
| Worth checking if the headers in VC14 have the structure now, as it may simplify the whole thing. If not, I'd add a leading underscore to the name so nobody confuses it for public API in the future. |
|
|
| msg271602 - (view) |
Author: Martin Panter (martin.panter) * |
Date: 2016-07-29 04:28 |
| Leading underscore is a good idea. I have no idea if VC14 includes the structure, but I suspect if it was added with this name, we would know because it would cause the same conflict that Min GW caused. |
|
|
| msg271605 - (view) |
Author: Eryk Sun (eryksun) *  |
Date: 2016-07-29 05:46 |
| REPARSE_DATA_BUFFER is defined in the WDK filesystem header, km\ntifs.h. It's not defined in the user-mode SDK. |
|
|
| msg271632 - (view) |
Author: Steve Dower (steve.dower) * |
Date: 2016-07-29 14:54 |
| I think the WDK is still separate from the Windows SDK, so probably best to keep the custom definition with a comment specifying which file it came from (so whoever diagnoses an obscure issue in the future can find it quickly). With the comment, I'm fine with the patch. If you haven't built on MSVC then keep an eye on the build bots until it goes through in case something unpredictable happens. |
|
|
| msg271941 - (view) |
Author: Roundup Robot (python-dev) |
Date: 2016-08-04 03:03 |
| New changeset e4ddfa61199c by Martin Panter in branch 'default': Issue #17599: Use unique _Py_REPARSE_DATA_BUFFER etc names to avoid conflict https://hg.python.org/cpython/rev/e4ddfa61199c |
|
|
| msg271957 - (view) |
Author: Martin Panter (martin.panter) * |
Date: 2016-08-04 08:33 |
| Thanks for your feedback. I add a reference to km\ntifs.h in the comment. The buildbots seem generally happy with the change. (One timed out in the tests, but that seems to be an existing intermittent problem.) |
|
|