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

Symcc runtime docsrs fix #270

Merged
merged 3 commits into from
Aug 23, 2021
Merged

Symcc runtime docsrs fix #270

merged 3 commits into from
Aug 23, 2021

Conversation

julihoh
Copy link
Collaborator

@julihoh julihoh commented Aug 20, 2021

symcc_runtime doesn't build on docs.rs, because the worker there does not have access to the network and the build.rs script attempts to download symcc from git.

the workaround to this problem is to include symcc as a submodule which will be included in the packaged crate file, so no network access is necessary to build the symcc_runtime on docs.rs.

This does add a submodule to the repository.

@julihoh
Copy link
Collaborator Author

julihoh commented Aug 20, 2021

I can't really verify that this will actually fix the problem, because I have trouble getting docs.rs to build locally.

if this is merged, it would be good to publish the symcc_runtime crate once again (it already has a new version number) to reflect changes on docs.rs

@julihoh
Copy link
Collaborator Author

julihoh commented Aug 20, 2021

Also: make sure to have cloned the submodule before publishing. otherwise it won't work

@domenukk
Copy link
Member

@andreafioraldi should we merge this now, the release 0.6.1 with the proper docs soonish? There was an Android fix too

@andreafioraldi andreafioraldi merged commit bdd1412 into main Aug 23, 2021
@domenukk domenukk deleted the symcc_runtime_docsrs_fix branch September 14, 2021 12:10
khang06 pushed a commit to khang06/LibAFL that referenced this pull request Oct 11, 2022
* add symcc as submodule

* make build script docs.rs aware

* bump version of symcc_runtime
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.

None yet

3 participants