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

There doesn't seem to be an easy way to import existing corpora #79

Open
evverx opened this issue Jan 21, 2022 · 3 comments
Open

There doesn't seem to be an easy way to import existing corpora #79

evverx opened this issue Jan 21, 2022 · 3 comments

Comments

@evverx
Copy link
Contributor

evverx commented Jan 21, 2022

I'm not sure it can be fixed easily in the sense that it seems to be hard to upload artifacts using curl or something like that but having spent about an hour trying to import a corpus I think the easiest way is to open a PR where the corpus is put in $OUT/fuzz-target_seed_corpus.zip and temporarily unblock cflite_batch to run it on PRs. This way, the corpus is uploaded and can be used by PRs before cflite_batch itself is merged.

@jonathanmetzman
Copy link
Collaborator

I'm not sure it can be fixed easily in the sense that it seems to be hard to upload artifacts using curl or something like that but having spent about an hour trying to import a corpus I think the easiest way is to open a PR where the corpus is put in $OUT/fuzz-target_seed_corpus.zip and temporarily unblock cflite_batch to run it on PRs. This way, the corpus is uploaded and can be used by PRs before cflite_batch itself is merged.

I suppose that would add to the repo size.
If you're using the git filestore then I think it's actually not so hard to add to an existing corpus right?

@evverx
Copy link
Contributor Author

evverx commented Jan 26, 2022

If you're using the git filestore then I think it's actually not so hard to add to an existing corpus right?

I didn't consider using a git repository to keep corpora because as far as I understand it isn't compatible with forks in the sense that every fork would have to create PERSONAL_ACCESS_TOKEN manually and change storage-repo accordingly.

@jonathanmetzman
Copy link
Collaborator

Interesting point.

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

2 participants