Skip to content

Support .NET Framework 4.6.1#2701

Merged
filmor merged 3 commits into
pythonnet:masterfrom
Metadorius:feature/net461
May 5, 2026
Merged

Support .NET Framework 4.6.1#2701
filmor merged 3 commits into
pythonnet:masterfrom
Metadorius:feature/net461

Conversation

@Metadorius
Copy link
Copy Markdown
Contributor

@Metadorius Metadorius commented Mar 27, 2026

What does this implement/fix? Explain your changes.

  • Bundles all required by net461 assemblies
  • Adds an AssemblyResolver to resolve them, because something prevents that and .config is not really accessible
  • Adds a PyInstaller hook to copy all those files properly, overshadowing the contrib repo hook which only copied Python.Runtime.dll

Does this close any currently open issues?

Closes #2695

Any other comments?

Requires merge of pythonnet/clr-loader#114

Checklist

Check all those that are applicable and complete.

  • Make sure to include one or more tests for your change
  • If an enhancement PR, please create docs and at best an example
  • Ensure you have signed the .NET Foundation CLA
  • Add yourself to AUTHORS
  • Updated the CHANGELOG

@Metadorius
Copy link
Copy Markdown
Contributor Author

@filmor tested this with my end users, it indeed works just fine

@Metadorius Metadorius marked this pull request as ready for review March 30, 2026 20:48
@Metadorius
Copy link
Copy Markdown
Contributor Author

Are any tests expected for this PR?

@Metadorius
Copy link
Copy Markdown
Contributor Author

My bad, I should've checked the behavior on .NET Core too. @filmor could you please approve the workflows to run again?

@Metadorius
Copy link
Copy Markdown
Contributor Author

Bump, just need tests to be approved to run.

@Metadorius
Copy link
Copy Markdown
Contributor Author

Is there any way I can help speed up the review/merge?

@Metadorius
Copy link
Copy Markdown
Contributor Author

@filmor sorry for bothering, any ETA on the merge?

@filmor
Copy link
Copy Markdown
Member

filmor commented May 5, 2026

Two issues:

  1. The new csproj is missing from the manifest, I'll fix that.
  2. This increases the wheel size substantially. Seeing that you seem to be the only one who needs this, would it work for you if our distributed wheels were still built directly and you could build your own wheel with an environment variable for internal distribution? Do you even need the wheels?

@filmor filmor force-pushed the feature/net461 branch from 2d829a0 to 92e5912 Compare May 5, 2026 15:50
@Metadorius
Copy link
Copy Markdown
Contributor Author

Metadorius commented May 5, 2026

  1. Thanks for the help, overlooked this one.

  2. The wheels are a preferred choice. If it's a critical thing, I can of course point uv to git, however, it's more trouble than just pre-built wheels, so I'd be happy if it was available on pypi. For example, uv caching caused issues with tag-based versioning when I was building the packages from source and did a partial fork, which cost me some hours figuring out what the issue was. I am sure there are other hidden nuances like this one.

    (I was sure that there's an ability to ship "variants" of a wheel, but turns out it was just extra packages via package[extra] syntax. Not sure if it's possible to have somehting like pythonnet[net461], it looks like not).

    Initialize() crash from .NET Framework application #2205 SerializationException when PythonEngine.Shutdown() #2221 Cannot import pandas in pythonnet #2222 python lambda doesn't match expression lambda #2425 seem all to be mentioning 4.6.1 in environment and are past 3.0. Likely not every user submits an issue, yet alone a PR to fix the issue, so I don't think I am the only one; there are probably other users who would benefit from this.

    As for the size increase - I checked, and the size seems to be increased from 0.2 MB to 1.6 MB. I feel like on an absolute scale even after increase that's negligible in 2026. Popular packages like numpy, pandas, cryptography are all multple megabytes worth, like 2-10 MB, and I've never heard this to be a problem.

@filmor
Copy link
Copy Markdown
Member

filmor commented May 5, 2026

Well, I don't want to join the large-package-size cohort if not necessary. Wheel variants don't exist, yet (there is an open PEP on this), otherwise that would have been the preferred way.

I think I will simply add a Windows wheel build that includes the .NET 4.6, then at least we don't carry all of the shim DLLs around on other platforms.

@lostmsu, what is your take on this?

@Metadorius
Copy link
Copy Markdown
Contributor Author

I feel like I am missing something, is 1.6 MB really that large? to me it looks like an increase from microscopic to just small 🤔

That works too, of course, just wondering if there's really a point in keeping it separate.

@filmor filmor force-pushed the feature/net461 branch from ddbb54f to 19dc42e Compare May 5, 2026 19:53
@filmor filmor merged commit 0238387 into pythonnet:master May 5, 2026
25 of 28 checks passed
Comment thread doc/source/python.rst
Default on Windows and also only supported there. Must be at least version
4.6.1, with 4.7.2 or later recommended.
4.6.1, with 4.7.2 or later recommended. For .NET 4.6 support, the wheel has
to be built with the environment variable `PYTHONNET_BUILD_NET46_SUPPORT=1`.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems to be outdated now, judging by uv build --wheel -C="--global-option=--net46-support" below

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

Successfully merging this pull request may close these issues.

Possible support for older framework versions

2 participants