Closed rtzoeller closed 4 years ago
@buckd trying to figure out the build changes here... it seems like dependencies are copied based on the existence of the dependency_target
key, but here we effectively have two different dependency steps: the execution unit implementation depends on the interface PPL, and the custom device depends on both PPLs. Not sure how to make that work with the current build tools implementation.
Confirmed with @niphilj and @buckd that we are okay merging the PPLs for now.
I've consolidated the PPLs into a single project, but left the file structure intact for now. Let me know if we should consolidate the files more than they are now.
Bleep bloop!
LabVIEW Diff Robot here with some diffs served up hot for your pull request.
Notice something funny? Help fix me on my GitHub repo.
Bleep bloop!
LabVIEW Diff Robot here with some diffs served up hot for your pull request.
Notice something funny? Help fix me on my GitHub repo.
What does this Pull Request accomplish?
Provide a stub implementation of the execution unit and execution unit factory interfaces. These currently just log to the system event log when they are called.
Why should this Pull Request be merged?
We need to be able to implement these interfaces eventually, and having a stub implementation now lets us sort out things like PPL loading in the engine and evaluate the interface design more critically.
What testing has been done?
Confirmed that I can deploy the custom device and that the system event log gets populated with messages from the custom device.