Skip to content

Do not install the *.c sources #73

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.

Sign up for GitHub

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

Jump to bottom
Merged
fantix merged 3 commits into from
Oct 16, 2023
Merged

Do not install the *.c sources #73

fantix merged 3 commits into from
Oct 16, 2023

Conversation

@hroncok
Copy link
Contributor

@hroncok hroncok commented Sep 9, 2021

They should not be required for anything.

@hroncok
Do not install the *.c sources
19d565e 
They should not be required for anything.
@Kludex
Copy link

Kludex commented Jul 3, 2022

The license file is being included through it, jfyk.

@fantix
Copy link
Member

fantix commented Oct 16, 2023 •
edited
Loading

We can use something like this to exclude only the C files:

https://github.com/MagicStack/asyncpg/blob/d7faaff57a7a9c0029a31f09564d30ab35007907/pyproject.toml#L63-L64

@fantix fantix merged commit b2fc5bd into MagicStack:master Oct 16, 2023
fantix added a commit that referenced this pull request Oct 16, 2023
@fantix
httptools 0.6.1
8a4a152 
Changes
=======

* Explicit Python 3.12 support and build wheels, change min version to 3.8 (#95)
  (by @tiptenbrink in 25f412b for #95)

* Do not install the *.c sources in wheels (#73)
  (by @hroncok in b2fc5bd for #73)
@fantix fantix mentioned this pull request Oct 16, 2023
Merged
fantix added a commit that referenced this pull request Oct 16, 2023
@fantix
httptools 0.6.1
de13727 
Changes
=======

* Explicit Python 3.12 support and build wheels, change min version to 3.8 (#95)
  (by @tiptenbrink in 25f412b for #95)

* Do not install the *.c sources in wheels (#73)
  (by @hroncok in b2fc5bd for #73)
@hroncok hroncok deleted the patch-1 branch October 17, 2023 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fantix fantix fantix approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hroncok @Kludex @fantix