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

Revert "move back to setuptools (#367)" #536

Closed
wants to merge 2 commits into from
Closed

Conversation

vishwin
Copy link

@vishwin vishwin commented Apr 11, 2022

Fixes #340, again

flit_core and installer are now mature enough to bootstrap themselves. Needed to enable a setuptools-free bootstrap environment and avoid circular dependencies.

This reverts commit 14ad352.

Signed-off-by: Charlie Li vishwin@FreeBSD.org

vishwin and others added 2 commits April 10, 2022 21:08
flit_core and installer are now mature enough to bootstrap themselves. Needed to enable a setuptools-free bootstrap environment and avoid circular dependencies.

This reverts commit 14ad352.

Signed-off-by: Charlie Li <vishwin@FreeBSD.org>
@brettcannon
Copy link
Member

@pradyunsg should we accept this and do a separate port over to PEP 621? And where do you want the version to live: __init__.py or pyproject.toml?

@pradyunsg
Copy link
Member

I'm fine with doing it in one shot or two. I have a mild preference for putting the version in __init__.py but don't feel strongly either way. :)

@brettcannon
Copy link
Member

@vishwin would you mind addressing the linting errors?

@vishwin
Copy link
Author

vishwin commented Apr 14, 2022

I'll try, and probably just PEP-621 it in the process

@rossburton
Copy link

Is there any progress on this or #546?

@brettcannon
Copy link
Member

@rossburton I'm waiting to see if #546 goes anywhere before considering this PR.

@nanonyme
Copy link

nanonyme commented Oct 6, 2022

@brettcannon seems #546 is reasonably mature at this point, pending final review. Probabaly worth focusing on that at this point.

@pradyunsg
Copy link
Member

Given that #546 is approved and this one has merge conflicts, I'm gonna go ahead and close this out.

Thanks for filing this @vishwin! ^>^

@pradyunsg pradyunsg closed this Oct 7, 2022
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

Successfully merging this pull request may close these issues.

Transition to using flit
5 participants