-
Notifications
You must be signed in to change notification settings - Fork 42
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
Create scyther_on_windows.md #42
base: master
Are you sure you want to change the base?
Conversation
detailed steps to build scyther and run it through the terminal
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thanks for this. Please double check the comments.
scyther_on_windows.md
Outdated
“git clone https://github.com/cascremers/scyther.git” | ||
6. Download the dependencies | ||
“pacman -S python3 flex bison” | ||
7. Run the build using the following command: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check: running make
from the main directory should just work here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this case I prefer 'make' (this is also more future-proof in case we change the build process)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this case I would prefer make
, because it is more future-proof.
scyther_on_windows.md
Outdated
“./scyther-w32.exe”, | ||
If we get an output like so, | ||
“Try 'scyther --help' for more information, or visit: | ||
https://cispa.saarland/group/cremers/scyther/index.html”, it’d mean that scyther works well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"It'd mean [...] well." -> "then Scyther is working as expected."
scyther_on_windows.md
Outdated
If we get an output like so, | ||
“Try 'scyther --help' for more information, or visit: | ||
https://cispa.saarland/group/cremers/scyther/index.html”, it’d mean that scyther works well. | ||
12. To check the working of security protocols, we can run the following command: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"the working of security protocols" -> "security protocol analysis".
modified guide
updated guide
modified changes
I think there is a bug in the description: we need to either
|
detailed steps to build scyther and run it through the terminal