Fix #766, Add to table search path#767
Conversation
|
@jphickey @astrogeco could we add this one in? stakeholder request. Ping @excaliburtb |
jphickey
left a comment
jphickey
left a comment
There was a problem hiding this comment.
I'm fine with this, it's simple enough and shouldn't break anything.
Once PR #751 is merged then I'd suggest adding a table search path to mission_defaults.cmake to allow customization of this table search path. It is a little more complicated than the app search path as it needs to be per-cpu but I think it can be done.
The table search path in mission_defaults.cmake would be the ideal solution in my mind. I really think moving the various search paths to mission_defaults.cmake is a significant improvement in general. |
CDKnightNASA
left a comment
CDKnightNASA
left a comment
There was a problem hiding this comment.
I like the idea of ${TGT}/ as it is easier to create addl. cpu's without having to spend time renaming long filenames that start with ${TGT}_.
|
Are you waiting on me for anything? I'd like to get this in to at least IC branch and the solution looks fine to me. |
|
Whoops, I thought this was in IC... @astrogeco could we get this added? |
astrogeco
changed the title
5 participants
Describe the contribution
Adds to table search path
Fix #766
Testing performed
CI
Expected behavior changes
Just adds to search path
System(s) tested on
CI
Additional context
None
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC