Skip to content

Conversation

dannycjones
Copy link
Contributor

Description of change

With the latest CRT update, we would exceed the 10MiB limit imposed by crates.io.

This change takes inspiration from the CRT Python bindings exclude list to remove files that are not useful for standard CRT usage. There's a few others that we identified that we can cut also.

Does this change impact existing behavior?

No change.

Does this change need a changelog entry in any of the crates?

No change, it does not change compilation output from mountpoint-s3-crt-sys.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I agree to the terms of the Developer Certificate of Origin (DCO).

Signed-off-by: Daniel Carl Jones <djonesoa@amazon.com>
@dannycjones dannycjones temporarily deployed to PR integration tests August 22, 2024 13:35 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests August 22, 2024 13:35 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests August 22, 2024 13:35 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests August 22, 2024 13:35 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests August 22, 2024 13:35 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests August 22, 2024 13:35 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests August 22, 2024 13:35 — with GitHub Actions Inactive
@dannycjones dannycjones enabled auto-merge August 22, 2024 13:35
Signed-off-by: Daniel Carl Jones <djonesoa@amazon.com>
@dannycjones dannycjones temporarily deployed to PR integration tests September 3, 2024 08:56 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests September 3, 2024 08:56 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests September 3, 2024 08:56 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests September 3, 2024 08:56 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests September 3, 2024 08:56 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests September 3, 2024 08:56 — with GitHub Actions Inactive
@dannycjones dannycjones temporarily deployed to PR integration tests September 3, 2024 08:56 — with GitHub Actions Inactive
@dannycjones dannycjones requested a review from unexge September 3, 2024 09:00
@dannycjones dannycjones added this pull request to the merge queue Sep 3, 2024
Merged via the queue into awslabs:main with commit ae4f909 Sep 3, 2024
23 checks passed
@dannycjones dannycjones deleted the update-crt-exclude branch September 3, 2024 09:28
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.

2 participants