Skip to content

Update some Python libraries - #3586

Open
peterebden wants to merge 5 commits into
thought-machine:masterfrom
peterebden:python-updates-2
Open

Update some Python libraries#3586
peterebden wants to merge 5 commits into
thought-machine:masterfrom
peterebden:python-updates-2

Conversation

@peterebden

Copy link
Copy Markdown
Collaborator

Fixing a warning about a missing licence and bringing some things more up to date. Tried a script or two locally and they still work.

I do wonder a little if we should drop all Python here and just do it all in Go, but it is sort of nice to illustrate that this is a multi-language build system.

@SpangleLabs

Copy link
Copy Markdown
Contributor

Should we be marking the licences for the rest of these?

@peterebden

Copy link
Copy Markdown
Collaborator Author

Should we be marking the licences for the rest of these?

We don't generally in this repo. I only added it for that one because pip_library is emitting a warning about it.

@peterebden

Copy link
Copy Markdown
Collaborator Author

Should we be marking the licences for the rest of these?

We don't generally in this repo. I only added it for that one because pip_library is emitting a warning about it.

Hmmmm I've changed it to python_wheel (which seems better) but it still emits the warning. So I guess it's not pip_library but some property of the wheel metadata.

@peterebden

Copy link
Copy Markdown
Collaborator Author

Should we be marking the licences for the rest of these?

We don't generally in this repo. I only added it for that one because pip_library is emitting a warning about it.

Hmmmm I've changed it to python_wheel (which seems better) but it still emits the warning. So I guess it's not pip_library but some property of the wheel metadata.

For posterity, this is a newer format that we'll start supporting with please-build/python-rules#322

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.

3 participants