Azure CLI wheels includes __init__.py files that should not be there

A given wheel should only be responsible for one file, and only one. It means no other packages should contain this file too: if two wheels contains the same file, installing them side by side will break the uninstallation scenario.

Concrete example:

• azure/__init__.py is owned by azure-nspkg, and only this wheel should contain it
• azure/cli/__init__.py is owned by azure-cli-nspkg, and only this wheel should contain it

This means the current wheel content of a package like azure-cli is wrong, since it contains both copy of these files.

To make things worse, the copies of these files use a “pkg_resource” format that is a performance killer. See also https://github.com/Azure/azure-cli/issues/13293

CLI packages MUST NOT contains files that are owned by any nspkg (applies to azure-cli-core, etc.)

[lmazuel]

That’s correct. You remove those lines from your wheel using this: https://github.com/Azure/azure-sdk-for-python/blob/7aecf6ffa4f886df25f40cc2cd2ea451dfea7e08/sdk/storage/azure-mgmt-storage/setup.py#L78-L80

See also: https://github.com/Azure/azure-sdk-for-python/blob/master/doc/dev/packaging.md

[lmazuel]

@arrownj

• it should not be in the wheel ever, whatever py2 or py3
• sdist is more interesting, I’m not sure it matters anymore that much if you target just py3 (keep it or leave it, same). Most of all the complex easy-install/dev install/etc. scenarios are gone. If you remove it you should be fine.