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

Use chrono time instead of Posix time #230

Merged
merged 2 commits into from
Dec 4, 2024
Merged

Conversation

kevin-f-ortega
Copy link
Collaborator

Related Issue(s)
Has Unit Tests (y/n)
Documentation Included (y/n)

Change Description

Using ChronoTime instead of PosixTime.

Rationale

This switch allows non-posix platforms to run with the vanilla deployment built with fprime-util new --deployment

Testing/Review Recommendations

Build a deployment with this change and run it locally. Verify time updates as expected. I already did this test on my Linux machine.

Future Work

None

…olves the issue of building for a non-posix platform with a vanilla deployment
@kevin-f-ortega kevin-f-ortega self-assigned this Dec 4, 2024
Copy link
Contributor

@thomas-bc thomas-bc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Was this deployment tested?

@kevin-f-ortega
Copy link
Collaborator Author

LGTM! Was this deployment tested?

Yes, I ran this deployment on my Linux machine. Time of events looked good to me

@kevin-f-ortega kevin-f-ortega merged commit 477c3d5 into devel Dec 4, 2024
31 checks passed
@kevin-f-ortega kevin-f-ortega deleted the posix-to-chrono branch December 4, 2024 22:55
@thomas-bc thomas-bc restored the posix-to-chrono branch December 4, 2024 23:02
@thomas-bc thomas-bc deleted the posix-to-chrono branch December 4, 2024 23:02
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