Skip to content

Conversation

@bastimeyer
Copy link
Member

Use build to build sdist and wheels instead of calling setuptools directly, which is deprecated.

See
https://github.com/pypa/build


I've checked the sdist tarball and the built wheels, and everything looked okay to me. The win32 and win-amd64 wheels also have the correct GUI entry scripts (streamlinkw).

Use `build` to build sdist and wheels instead of calling setuptools
directly, which is deprecated.
@gravyboat
Copy link
Member

Looks good, nice improvement!

@gravyboat gravyboat merged commit ec42682 into streamlink:master Jan 8, 2023
@bastimeyer
Copy link
Member Author

There's also

gpg ... --passphrase="${RELEASE_KEY_PASSPHRASE}" ...

which should be changed to

gpg ... --passphrase-fd 0 ... <<< "${RELEASE_KEY_PASSPHRASE}"

but I didn't want to touch this. The PyPI user+pass are also passed as parameters to twine in the deploy script, and this doesn't look like it can be changed other than env vars, which is better, but not ideal either. Doesn't matter too much though on the CI side.


Btw, I don't have the password for the encrypted signing key (462165d) and it's also not on the protonmail account. I believe only @beardypig has it. If nobody else has it, then I'll send him an email (I don't think he'll see this message)

@bastimeyer bastimeyer deleted the build/update-build-script branch January 8, 2023 02:03
@gravyboat
Copy link
Member

I double checked and I don't have the password for the signing key.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants