search

georgemarshall / django-cryptography

Easily encrypt data in Django
https://django-cryptography.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
371 stars 69 forks source link

Make sure subpackages are included in wheel #88

Open apeschar opened 1 year ago

apeschar commented 1 year ago

With the current setup.cfg, a build done using pip wheel does not actually include the core/ and utils/ directories, and when installed, the package is broken.

To test, try this:

rm -rf dist
pip wheel . --no-deps -w dist
unzip -l dist/*.whl

Before this PR:

Archive:  dist/django_cryptography-2.0-py2.py3-none-any.whl
  Length      Date    Time    Name
---------  ---------- -----   ----
      114  11-04-2022 07:52   django_cryptography/__init__.py
     1184  11-04-2022 07:52   django_cryptography/conf.py
     7903  11-04-2022 07:52   django_cryptography/fields.py
        0  11-04-2022 07:52   django_cryptography/py.typed
     1109  11-04-2022 07:52   django_cryptography/typing.py
     1492  11-04-2022 07:56   django_cryptography-2.0.dist-info/LICENSE
     3434  11-04-2022 07:56   django_cryptography-2.0.dist-info/METADATA
      110  11-04-2022 07:56   django_cryptography-2.0.dist-info/WHEEL
       20  11-04-2022 07:56   django_cryptography-2.0.dist-info/top_level.txt
      862  11-04-2022 07:56   django_cryptography-2.0.dist-info/RECORD
---------                     -------
    16228                     10 files

After this PR:

Archive:  dist/django_cryptography-2.0-py2.py3-none-any.whl
  Length      Date    Time    Name
---------  ---------- -----   ----
      114  11-04-2022 07:52   django_cryptography/__init__.py
     1184  11-04-2022 07:52   django_cryptography/conf.py
     7903  11-04-2022 07:52   django_cryptography/fields.py
        0  11-04-2022 07:52   django_cryptography/py.typed
     1109  11-04-2022 07:52   django_cryptography/typing.py
        0  11-04-2022 07:39   django_cryptography/core/__init__.py
    10363  11-04-2022 07:52   django_cryptography/core/signing.py
        0  11-04-2022 07:39   django_cryptography/utils/__init__.py
     6966  11-04-2022 07:52   django_cryptography/utils/crypto.py
     1492  11-04-2022 07:56   django_cryptography-2.0.dist-info/LICENSE
     3434  11-04-2022 07:56   django_cryptography-2.0.dist-info/METADATA
      110  11-04-2022 07:56   django_cryptography-2.0.dist-info/WHEEL
       20  11-04-2022 07:56   django_cryptography-2.0.dist-info/top_level.txt
     1228  11-04-2022 07:56   django_cryptography-2.0.dist-info/RECORD
---------                     -------
    33923                     14 files
centromere commented 1 year ago

@georgemarshall, would you be willing to merge this?