Rest of fix for gPTP intervals on ARM #646
Conversation
The previous fix correctly identified the problem but was incomplete--this one has actually been verified to work. More `char` fields had to be changed to `signed char` to avoid problems with negative numbers.
|
Sorry, my testing procedure for the last patch was flawed; I hadn't removed the command line parameters from the correct launch script, so it launched with command line parameters and worked correctly due to that instead of the incomplete fix. I got the right script edited this time, so I'm pretty sure it's actually working to set the right default parameters now. The Travis failure is unrelated to the code here; I'm not sure why it's doing that. |
|
Here's the Travis issue log for the apt repository causing the build fail: https://www.traviscistatus.com/incidents/z1317rvdkbsj |
|
@pinealservo - thanks for the updated commit. @vitorroriz - thanks for testing. Merging. |
|
Looks like there is something wrong with travis at the moment, but I'm still going to merge this pull request. |
The previous fix correctly identified the problem but was incomplete--this
one has actually been verified to work. More
charfields had to be changedto
signed charto avoid problems with negative numbers.