import clr clashes with pythonnet package

[lmittmann]

import clr clashes with pythonnet package

[den-run-ai]

This does not affect our build or installation, but new pythonnet users will keep bumping into this issue by incorrectly assuming that pip install clr is necessary before import clr.

Would you mind renaming your package before it is too late? I see that it is only about one month old?

[vmuriart]

The name clashes with IronPython which uses import clr as its import hook.

[lmittmann]

I noticed the package name conflict after publishing. Does this affect your build or installation process?

My understanding of pythonnet/pythonnet#388 is, that pip install clr was executed instead of pip install pythonnet, right?

[lmittmann]

It is actually really hard to find a good 3 letter name. How about a little compromise:

• We update the pythonnet/README.md to show some installation instructions, since people who wrongly install clr instead of pythonnet just got a different error, before I published my package.
• I will update the AttributeError to show a more explanatory error message, pointing to pythonnet

[vmuriart]

The problem is for users that install both like the original issue that made us detect it.

[lmittmann]

I thought the original issue was that a new pythonnet user installed the wrong package.

[den-run-ai]

import clr import System How would you report error on this?

…

On Sat, Feb 18, 2017, 12:34 PM Victor Uriarte ***@***.***> wrote: The problem is for users that install both like the original issue that made us detect it. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHgZ5fvbkhjJmnJD3GV9yk83C3bc561Kks5rdzm0gaJpZM4MDzqB> .

[lmittmann]

System is a toplevel module of pythonnet?

[den-run-ai]

"import clr" hooks up CPython with .NET runtime. "import System" imports main .NET assembly namespace into CPython. Same imports are used in IronPython.

…

On Sat, Feb 18, 2017, 12:46 PM lmittmann ***@***.***> wrote: System is a toplevel module of pythonnet? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHgZ5SgV0u4XgfJJi9mSll8L4r-kfSMLks5rdzyEgaJpZM4MDzqB> .

[lmittmann]

The AttributeError caused by clr.AddReference can be easily detected. Detecting the import System error might be more complicated. I will have a look on how this "hook up" is done, it might be also catchable.

[vmuriart]

We modify the import hook to inject .net libraries. In this case it would be difficult to decide if it's an import error because of clr vs pythonnet or if the user is trying to import libraries from Python or from GAC

…

On Sat, Feb 18, 2017 at 11:27 lmittmann ***@***.***> wrote: The AttributeError caused by clr.AddReference can be easily detected. Detecting the import System error might be more complicated. I will have a look on how this "hook up" is done, it might be also catchable. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AMr87O6qvkmOIo7_aTyIzr5_L-E6hF9aks5rd0YtgaJpZM4MDzqB> .

[vmuriart]

You can do a quick Google lookup of "import clr" to see the amount of usage it has.

…

On Sat, Feb 18, 2017 at 12:47 Victor Uriarte ***@***.***> wrote: We modify the import hook to inject .net libraries. In this case it would be difficult to decide if it's an import error because of clr vs pythonnet or if the user is trying to import libraries from Python or from GAC On Sat, Feb 18, 2017 at 11:27 lmittmann ***@***.***> wrote: > The AttributeError caused by clr.AddReference can be easily detected. > Detecting the import System error might be more complicated. I will have > a look on how this "hook up" is done, it might be also catchable. > > — > You are receiving this because you commented. > Reply to this email directly, view it on GitHub > <#1 (comment)>, or mute > the thread > <https://github.com/notifications/unsubscribe-auth/AMr87O6qvkmOIo7_aTyIzr5_L-E6hF9aks5rd0YtgaJpZM4MDzqB> > . >

[filmor]

@lmittmann Please take a stance on this now.

Are you going to rename your library? You are clashing with both Python.NET and IronPython and are causing confusion for people using pip directly. If not, please at least say so now and we will put the corresponding information into our Readme and entry page.

Please don't try to hack around this by capturing import System or clr.AddReference.

[lmittmann]

I will rename the library later today. I am not sure about the best way to transfer ownership of a pypi package though. Any ideas @denfromufa @vmuriart @filmor?

[filmor]

Does this link work? https://pypi.python.org/pypi?:action=role_form&package_name=clr

You can add one of us as owners and we will handle it from there.

Many thanks for the renaming :)

[lmittmann]

That worked, you should have access now.

[koubaa]

@lmittmann could you please unlist clr from pypi?