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

Sv comp script java 8 #102

Open
wants to merge 2 commits into
base: gradle-build
Choose a base branch
from

Conversation

Ao-senXiong
Copy link

No description provided.

@sohah
Copy link

sohah commented Aug 11, 2024

@Ao-senXiong , this pull request needs a few changes:

  1. the sv-comp script needs to use relative paths, you cannot use your username in the script.
  2. the instructions on the readme is not accurate, and detailed. Please, have more detailed instructions about where we get the benchmarks we test, the directory structure between the sv-comp benchmarks and SPF, and please use relative directory structure in your command to run the script, i.e., avoid using your username, and use a relative path, that allows the user to just copy and paste your command if he/she got the directory structure in the right way.

@Ao-senXiong
Copy link
Author

@Ao-senXiong , this pull request needs a few changes:

  1. the sv-comp script needs to use relative paths, you cannot use your username in the script.
  2. the instructions on the readme is not accurate, and detailed. Please, have more detailed instructions about where we get the benchmarks we test, the directory structure between the sv-comp benchmarks and SPF, and please use relative directory structure in your command to run the script, i.e., avoid using your username, and use a relative path, that allows the user to just copy and paste your command if he/she got the directory structure in the right way.

Sure, thanks! I will make the changes after the script is stable and usable.

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.

2 participants