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

reduce submodule clone time and size using depth option #1616

Closed
wants to merge 2 commits into from

Conversation

karaketir16
Copy link
Contributor

This will decrease the required clone size and so reduce the overall build time.

@karaketir16
Copy link
Contributor Author

Let's wait for the workflow completion for comparision

@karaketir16
Copy link
Contributor Author

Some remotes don’t support cloning with the --depth 1 option. I’ll investigate.

some remotes do not allow cloning with --depth 1
@karaketir16
Copy link
Contributor Author

Comparison of Cloning the binutils Submodule

Without --depth 1:
image

With --depth 1:
image

Results:

  • Object Count: Reduced drastically from 1,239,461 to 40,643.
  • Download Size: Dropped from approximately 450 MiB (full clone) to 78.15 MiB (shallow clone).

Note: I did not wait this time for cloning of the full history.
here an old log from workflow:
image

@TommyMurphyTM1234
Copy link
Collaborator

I presume that you've seen this existing open PR already?

Some remotes don’t support cloning with the --depth 1 option. I’ll investigate.

I thought that uClibc was the only one here that had a problem with it?

@karaketir16
Copy link
Contributor Author

No, I did not see other pull request.
I wrote a script, thats try full clone, if shallow fails.

@mickflemm
Copy link
Contributor

This is already done here, for all modules including uclibc:
efc7cc9

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