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

Distribute minimal source archives #131

Closed
wants to merge 4 commits into from
Closed

Distribute minimal source archives #131

wants to merge 4 commits into from

Conversation

nbren12
Copy link
Contributor

@nbren12 nbren12 commented Feb 8, 2021

It takes a while to check out this repo, which is quite large. Uncompressed the repo size is 2 GB, which makes it a bit slow to build dockerfiles with. To cut down on the initial download for users, this PR sets git attributes such that only the files within the enzyme subdirectory are exported with git-archive. The enzyme/benchmarks folder is also ignored since it is quite large. The resulting tar file is <1 MB in size.

See #119.

@nbren12 nbren12 mentioned this pull request Feb 8, 2021
@wsmoses
Copy link
Member

wsmoses commented Feb 9, 2021

It looks like your cmake change isn't finding the benchmark directory?

@nbren12
Copy link
Contributor Author

nbren12 commented Feb 9, 2021

Thanks @wsmoses. I'll look into it. Nice set of tests!

The v2 action seems to ignore files marked with `export-ignore` in the .gitattributes file.

See https://github.community/t/using-container-checkout-doesnt-checkout-hidden-files/18429
Base automatically changed from master to main February 10, 2021 20:07
@vchuravy
Copy link
Member

Closed in favour of #135 thanks @nbren12

@vchuravy vchuravy closed this Feb 10, 2021
@nbren12
Copy link
Contributor Author

nbren12 commented Feb 10, 2021

Sounds good. Thanks @vchuravy. This "CI apocalypse" sounds grim haha.

@nbren12 nbren12 deleted the small-tars branch February 11, 2021 00:58
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