Skip to content

Make clib build reproducible by sorting sources#3679

Merged
abravalheri merged 2 commits into
pypa:mainfrom
danigm:reproducible-build-clib
Nov 23, 2022
Merged

Make clib build reproducible by sorting sources#3679
abravalheri merged 2 commits into
pypa:mainfrom
danigm:reproducible-build-clib

Conversation

@danigm

@danigm danigm commented Nov 14, 2022

Copy link
Copy Markdown
Contributor

Summary of changes

Sort sources in build_clib command.

Closes #3678

Pull Request Checklist

  • Changes have tests
  • News fragment added in [changelog.d/].

@abravalheri

abravalheri commented Nov 14, 2022

Copy link
Copy Markdown
Contributor

Hi @danigm thank you very much for the contribution.

Please note that any changes in the setuptools/_distutils directory need to be proposed in the https://github.com/pypa/distutils repository.

Would you like to propose this change to pypa/distutils?

@danigm danigm force-pushed the reproducible-build-clib branch from e2a115b to 95234bb Compare November 15, 2022 07:20
@danigm

danigm commented Nov 15, 2022

Copy link
Copy Markdown
Contributor Author

Hi @danigm thank you very much for the contribution.

Please note that any changes in the setuptools/_distutils directory need to be proposed in the https://github.com/pypa/distutils repository.

Would you like to propose this change to pypa/distutils?

Thanks for the info. I've removed that change from the PR, that's not "needed" because the build_libraries method overrides the base class functionality, so it should work as expected just with this change.

@abravalheri

Copy link
Copy Markdown
Contributor

I think this looks great.

If no one beats me, I can merge the PR next time I am releasing a version.

Comment thread changelog.d/3678.change.rst Outdated
Co-authored-by: Anderson Bravalheri <andersonbravalheri+github@gmail.com>
@abravalheri abravalheri merged commit d805c53 into pypa:main Nov 23, 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.

[FR] Reproducible build of clib libraries

2 participants