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

Put examples in another repo #88

Closed
anacrolix opened this issue Aug 23, 2015 · 7 comments
Closed

Put examples in another repo #88

anacrolix opened this issue Aug 23, 2015 · 7 comments

Comments

@anacrolix
Copy link

The go-fuzz repository is abnormally large, and takes some time to fetch on poor connections. Perhaps you could move the non-essential assets to another repository?

@Thomasdezeeuw
Copy link
Contributor

Maybe the corpus should be removed, but I think the code in the examples are quite usefull.

@dgryski
Copy link
Contributor

dgryski commented Aug 24, 2015

Removing them from the repo will only shrink the download size if we rewrote history.

@Thomasdezeeuw
Copy link
Contributor

True, but I think it will be worth it. Currently the full repo is about 300 MB on disk for me which seems rather big. Remove the examples and the complete git history it's only 1.2 MB.

@dgryski
Copy link
Contributor

dgryski commented Aug 24, 2015

Indeed. It looks like the majority of on-disk space is taken up because the corpus files are small, but still take up an entire disk block.

<dgryski@kaepora[go-fuzz] \ʕ◔ϖ◔ʔ/ > du -sh .
749M    .
<dgryski@kaepora[go-fuzz] \ʕ◔ϖ◔ʔ/ > git rm -q -r examples
<dgryski@kaepora[go-fuzz] \ʕ◔ϖ◔ʔ/ > git commit -q -m "examples: delete"
<dgryski@kaepora[go-fuzz] \ʕ◔ϖ◔ʔ/ > du -sh .
31M .


@anacrolix
Copy link
Author

sparseness more apparent this way

gopath/src/github.com/dvyukov/go-fuzz$ gdu --apparent-size -sh .
73M .
gopath/src/github.com/dvyukov/go-fuzz$ gdu -sh .
287M    .

But it was originally the transfer size I am concerned with, which is in the order of 22-24MB. https://api.github.com/repos/dvyukov/go-fuzz

@dgryski
Copy link
Contributor

dgryski commented Dec 26, 2017

Will be obsoleted by #154

@AlekSi
Copy link
Contributor

AlekSi commented Feb 6, 2018

@dvyukov This is done too.

@dvyukov dvyukov closed this as completed Feb 6, 2018
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

No branches or pull requests

5 participants