Skip to content

bpo-41089: Filters and other issues in Visual Studio projects #21070

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

Merged
merged 1 commit into from
Jun 23, 2020
Merged

bpo-41089: Filters and other issues in Visual Studio projects #21070

merged 1 commit into from
Jun 23, 2020

Conversation

nnemkin
Copy link
Contributor

@nnemkin nnemkin commented Jun 23, 2020

  • Add missing header files to pythoncore.
  • Add missing file filters ("Resource Files" in particular) to all projects.
  • Add new sub-filters for private headers in pythoncore and for 3rd party source files.
  • Add missing _zoneinfo configurations in pcbuild.sln.
  • Update bdist_wininst with the new zlib location.

Note: apart from bdist_wininst fix and added pythoncore headers, all changes are to .vcxproj.filters.

https://bugs.python.org/issue41089

* Add missing header files to pythoncore.
* Add missing file filters ("Resource Files" in particular) to
  all projects.
* Add new sub-filters for private headers in pythoncore and
  for 3rd party source files.
* Add missing _zoneinfo configurations in pcbuild.sln.
* Update bdist_wininst with the new zlib location.
@nnemkin nnemkin requested review from rhettinger, skrah and a team as code owners June 23, 2020 11:18
@zooba zooba added needs backport to 3.9 only security fixes skip news labels Jun 23, 2020
@zooba zooba merged commit fe2a48c into python:master Jun 23, 2020
@miss-islington
Copy link
Contributor

Thanks @nnemkin for the PR, and @zooba for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry, @nnemkin and @zooba, I could not cleanly backport this to 3.9 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker fe2a48c605d98ac02ab2b9593cb87ce364aeae2d 3.9

zooba pushed a commit to zooba/cpython that referenced this pull request Jun 23, 2020
…GH-21070)

* Add missing header files to pythoncore.
* Add missing file filters ("Resource Files" in particular) to
  all projects.
* Add new sub-filters for private headers in pythoncore and
  for 3rd party source files.
* Add missing _zoneinfo configurations in pcbuild.sln.
* Update bdist_wininst with the new zlib location.
@bedevere-bot
Copy link

GH-21090 is a backport of this pull request to the 3.9 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Jun 23, 2020
@nnemkin nnemkin deleted the vs_filters branch June 23, 2020 18:58
zooba added a commit that referenced this pull request Jun 23, 2020
* Add missing header files to pythoncore.
* Add missing file filters ("Resource Files" in particular) to
  all projects.
* Add new sub-filters for private headers in pythoncore and
  for 3rd party source files.
* Add missing _zoneinfo configurations in pcbuild.sln.
* Update bdist_wininst with the new zlib location.

Co-authored-by: Nikita Nemkin <nikita@nemkin.ru>
fasih pushed a commit to fasih/cpython that referenced this pull request Jun 29, 2020
…GH-21070)

* Add missing header files to pythoncore.
* Add missing file filters ("Resource Files" in particular) to
  all projects.
* Add new sub-filters for private headers in pythoncore and
  for 3rd party source files.
* Add missing _zoneinfo configurations in pcbuild.sln.
* Update bdist_wininst with the new zlib location.
arun-mani-j pushed a commit to arun-mani-j/cpython that referenced this pull request Jul 21, 2020
…GH-21070)

* Add missing header files to pythoncore.
* Add missing file filters ("Resource Files" in particular) to
  all projects.
* Add new sub-filters for private headers in pythoncore and
  for 3rd party source files.
* Add missing _zoneinfo configurations in pcbuild.sln.
* Update bdist_wininst with the new zlib location.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants