Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sdist for 0.4.3 is missing pyx files #70

Closed
isuruf opened this issue Sep 7, 2023 · 4 comments
Closed

sdist for 0.4.3 is missing pyx files #70

isuruf opened this issue Sep 7, 2023 · 4 comments

Comments

@isuruf
Copy link
Member

isuruf commented Sep 7, 2023

Running python setup.py sdist locally packages the pyx files though.

@oscarbenjamin
Copy link
Collaborator

Running python setup.py sdist locally packages the pyx files though.

Are you sure? When I run locally they are not included.

@oscarbenjamin
Copy link
Collaborator

Building locally I have this:

$ tree python-flint-0.4.3
python-flint-0.4.3
├── doc
│   ├── Makefile
│   └── source
│       ├── acb_mat.rst
│       ├── acb_poly.rst
│       ├── acb.rst
│       ├── acb_series.rst
│       ├── arb_mat.rst
│       ├── arb_poly.rst
│       ├── arb.rst
│       ├── arb_series.rst
│       ├── conf.py
│       ├── dirichlet.rst
│       ├── fmpq_mat.rst
│       ├── fmpq_poly.rst
│       ├── fmpq.rst
│       ├── fmpq_series.rst
│       ├── fmpz_mat.rst
│       ├── fmpz_poly.rst
│       ├── fmpz.rst
│       ├── fmpz_series.rst
│       ├── general.rst
│       ├── index.rst
│       ├── nmod_mat.rst
│       ├── nmod_poly.rst
│       ├── nmod.rst
│       ├── nmod_series.rst
│       └── setup.rst
├── LICENSE
├── MANIFEST.in
├── PKG-INFO
├── README.md
├── setup.cfg
├── setup.py
└── src
    ├── flint
    │   ├── flint_base
    │   │   ├── flint_base.c
    │   │   └── flint_context.c
    │   ├── _flint.pxd
    │   ├── functions.pyx
    │   ├── __init__.py
    │   ├── pyflint.c
    │   ├── pyflint.pxd
    │   ├── pyflint.pyx
    │   ├── test
    │   │   ├── __init__.py
    │   │   ├── __main__.py
    │   │   └── test.py
    │   └── types
    │       ├── acb.c
    │       ├── acb_mat.c
    │       ├── acb_poly.c
    │       ├── acb_series.c
    │       ├── arb.c
    │       ├── arb_mat.c
    │       ├── arb_poly.c
    │       ├── arb_series.c
    │       ├── arf.c
    │       ├── dirichlet.c
    │       ├── fmpq.c
    │       ├── fmpq_mat.c
    │       ├── fmpq_poly.c
    │       ├── fmpq_series.c
    │       ├── fmpz.c
    │       ├── fmpz_mat.c
    │       ├── fmpz_mpoly.c
    │       ├── fmpz_poly.c
    │       ├── fmpz_series.c
    │       ├── nmod.c
    │       ├── nmod_mat.c
    │       ├── nmod_poly.c
    │       └── nmod_series.c
    └── python_flint.egg-info
        ├── dependency_links.txt
        ├── PKG-INFO
        ├── SOURCES.txt
        └── top_level.txt

8 directories, 70 files

So the .pyx and .pxd files from top level inside flint are included but not from the packages like flint.types. One problem is the packages are not listed:

packages=['flint', 'flint.test'],

Fixing that only includes the __init__.py files though.

@oscarbenjamin
Copy link
Collaborator

After #71 I've just pushed 0.4.4 to PyPI. New sdist contents are:

$ tree -v python-flint-0.4.4
python-flint-0.4.4
├── LICENSE
├── MANIFEST.in
├── PKG-INFO
├── README.md
├── doc
│   ├── Makefile
│   └── source
│       ├── acb.rst
│       ├── acb_mat.rst
│       ├── acb_poly.rst
│       ├── acb_series.rst
│       ├── arb.rst
│       ├── arb_mat.rst
│       ├── arb_poly.rst
│       ├── arb_series.rst
│       ├── conf.py
│       ├── dirichlet.rst
│       ├── fmpq.rst
│       ├── fmpq_mat.rst
│       ├── fmpq_poly.rst
│       ├── fmpq_series.rst
│       ├── fmpz.rst
│       ├── fmpz_mat.rst
│       ├── fmpz_poly.rst
│       ├── fmpz_series.rst
│       ├── general.rst
│       ├── index.rst
│       ├── nmod.rst
│       ├── nmod_mat.rst
│       ├── nmod_poly.rst
│       ├── nmod_series.rst
│       └── setup.rst
├── setup.cfg
├── setup.py
└── src
    ├── flint
    │   ├── __init__.py
    │   ├── _flint.pxd
    │   ├── flint_base
    │   │   ├── __init__.py
    │   │   ├── flint_base.c
    │   │   ├── flint_base.pxd
    │   │   ├── flint_base.pyx
    │   │   ├── flint_context.c
    │   │   ├── flint_context.pxd
    │   │   └── flint_context.pyx
    │   ├── flintlib
    │   │   ├── __init__.pxd
    │   │   ├── __init__.py
    │   │   ├── acb.pxd
    │   │   ├── acb_calc.pxd
    │   │   ├── acb_dft.pxd
    │   │   ├── acb_dirichlet.pxd
    │   │   ├── acb_elliptic.pxd
    │   │   ├── acb_hypgeom.pxd
    │   │   ├── acb_mat.pxd
    │   │   ├── acb_modular.pxd
    │   │   ├── acb_poly.pxd
    │   │   ├── arb.pxd
    │   │   ├── arb_fmpz_poly.pxd
    │   │   ├── arb_hypgeom.pxd
    │   │   ├── arb_mat.pxd
    │   │   ├── arb_poly.pxd
    │   │   ├── arf.pxd
    │   │   ├── arith.pxd
    │   │   ├── bernoulli.pxd
    │   │   ├── dirichlet.pxd
    │   │   ├── fmpq.pxd
    │   │   ├── fmpq_mat.pxd
    │   │   ├── fmpq_poly.pxd
    │   │   ├── fmpz.pxd
    │   │   ├── fmpz_factor.pxd
    │   │   ├── fmpz_lll.pxd
    │   │   ├── fmpz_mat.pxd
    │   │   ├── fmpz_mpoly.pxd
    │   │   ├── fmpz_poly.pxd
    │   │   ├── fmpz_poly_factor.pxd
    │   │   ├── mag.pxd
    │   │   ├── mpoly.pxd
    │   │   ├── nmod_mat.pxd
    │   │   ├── nmod_poly.pxd
    │   │   ├── nmod_vec.pxd
    │   │   ├── partitions.pxd
    │   │   └── ulong_extras.pxd
    │   ├── functions.pyx
    │   ├── pyflint.c
    │   ├── pyflint.pxd
    │   ├── pyflint.pyx
    │   ├── test
    │   │   ├── __init__.py
    │   │   ├── __main__.py
    │   │   └── test.py
    │   ├── types
    │   │   ├── __init__.py
    │   │   ├── acb.c
    │   │   ├── acb.pxd
    │   │   ├── acb.pyx
    │   │   ├── acb_mat.c
    │   │   ├── acb_mat.pxd
    │   │   ├── acb_mat.pyx
    │   │   ├── acb_poly.c
    │   │   ├── acb_poly.pxd
    │   │   ├── acb_poly.pyx
    │   │   ├── acb_series.c
    │   │   ├── acb_series.pxd
    │   │   ├── acb_series.pyx
    │   │   ├── arb.c
    │   │   ├── arb.pxd
    │   │   ├── arb.pyx
    │   │   ├── arb_mat.c
    │   │   ├── arb_mat.pxd
    │   │   ├── arb_mat.pyx
    │   │   ├── arb_poly.c
    │   │   ├── arb_poly.pxd
    │   │   ├── arb_poly.pyx
    │   │   ├── arb_series.c
    │   │   ├── arb_series.pxd
    │   │   ├── arb_series.pyx
    │   │   ├── arf.c
    │   │   ├── arf.pxd
    │   │   ├── arf.pyx
    │   │   ├── dirichlet.c
    │   │   ├── dirichlet.pxd
    │   │   ├── dirichlet.pyx
    │   │   ├── fmpq.c
    │   │   ├── fmpq.pxd
    │   │   ├── fmpq.pyx
    │   │   ├── fmpq_mat.c
    │   │   ├── fmpq_mat.pxd
    │   │   ├── fmpq_mat.pyx
    │   │   ├── fmpq_poly.c
    │   │   ├── fmpq_poly.pxd
    │   │   ├── fmpq_poly.pyx
    │   │   ├── fmpq_series.c
    │   │   ├── fmpq_series.pxd
    │   │   ├── fmpq_series.pyx
    │   │   ├── fmpz.c
    │   │   ├── fmpz.pxd
    │   │   ├── fmpz.pyx
    │   │   ├── fmpz_mat.c
    │   │   ├── fmpz_mat.pxd
    │   │   ├── fmpz_mat.pyx
    │   │   ├── fmpz_mpoly.c
    │   │   ├── fmpz_mpoly.pxd
    │   │   ├── fmpz_mpoly.pyx
    │   │   ├── fmpz_poly.c
    │   │   ├── fmpz_poly.pxd
    │   │   ├── fmpz_poly.pyx
    │   │   ├── fmpz_series.c
    │   │   ├── fmpz_series.pxd
    │   │   ├── fmpz_series.pyx
    │   │   ├── nmod.c
    │   │   ├── nmod.pxd
    │   │   ├── nmod.pyx
    │   │   ├── nmod_mat.c
    │   │   ├── nmod_mat.pxd
    │   │   ├── nmod_mat.pyx
    │   │   ├── nmod_poly.c
    │   │   ├── nmod_poly.pxd
    │   │   ├── nmod_poly.pyx
    │   │   ├── nmod_series.c
    │   │   ├── nmod_series.pxd
    │   │   └── nmod_series.pyx
    │   └── utils
    │       ├── __init__.py
    │       ├── conversion.pxd
    │       └── typecheck.pxd
    └── python_flint.egg-info
        ├── PKG-INFO
        ├── SOURCES.txt
        ├── dependency_links.txt
        └── top_level.txt

10 directories, 162 files

Does that look okay to you @isuruf ?

@isuruf
Copy link
Member Author

isuruf commented Sep 7, 2023

Yes, it works for me. Next issue: #74

@isuruf isuruf closed this as completed Sep 7, 2023
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

No branches or pull requests

2 participants