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

Update README.adoc #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.adoc #2

wants to merge 1 commit into from

Conversation

requenag
Copy link

Update to help everyone to install this on Centos7
Please note this doesn't work on Centos8 (I wasn't able to solve the libpcap.h (libpcap-devel dependencies)

Update to help everyone to install this on Centos7
Please note this doesn't work on Centos8 (I wasn't able to solve the libpcap.h (libpcap-devel dependencies)
@mlichvar
Copy link
Owner

On Centos8 it should be the same, except maybe you need to enable the codeready-builder repository?

@requenag
Copy link
Author

Indeed. I should have check rpmfinder and there is this package on Centos8 (PowerTools repo).
I will give a try.

As well, I did a quick diagram to understand how this works as I am a bit slow. :)
Here is the diagram attached.

ntpperf_001

@mlichvar
Copy link
Owner

Looks good. I'd just replace "NTP client" with ntpperf and move the 172.18.0.0/16 box inside to me it clear it's not a single client and that it is simulated (the packets are not routed outside). Also, there is a typo in "192.168.1.0/24".

@requenag
Copy link
Author

Updated.

ntpperf

@requenag
Copy link
Author

My bad. I fixed the Client/Server @ip here is the last one :)
ntpperf

Happy to send the drawio via email if you prefer to let you change it.

@mlichvar
Copy link
Owner

Great. Please add it to the document and commit.

@requenag
Copy link
Author

Upload are disable. I can't add the picture to the readme

@mlichvar
Copy link
Owner

Why not add it as a new file to the repo and reference it in the document?

@requenag
Copy link
Author

requenag commented May 13, 2021 via email

@jhalldn
Copy link

jhalldn commented Apr 2, 2023

How do you build this with NTS support? Edit Makefile or add some parameter to make command?

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.

3 participants