Tierion / pymerkletools

Python tools for creating Merkle trees, generating Merkle proofs, and verification of Merkle proofs
MIT License
166 stars 53 forks source link

"tests" package erroneously installed by "setup.py" #19

Closed leycec closed 5 years ago

leycec commented 5 years ago

Greetings and cryptographic salutations. I'm currently packaging merkletools for Gentoo and briefly faceplanted into a minor installation-time issue.

wat is bad?

Specifically, the top-level setup.py script erroneously installs the tests package. This is bad – not simply under Gentoo but under any Python environment. Why? Coupla reasons:

how to make good?

We've trivially patched this on our end. Here's how you can, too. In setup.py:

    # Replace this bad line...
    packages=find_packages(),

    # ...with this good line.
    packages=find_packages(exclude=["tests"]),

That's it. Easy peasy. Thanks for all the fruitful Merkle trees, all! :deciduous_tree:

jacohend commented 5 years ago

Thanks @leycec, we've made this change.