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

Volume Table Requires User Modification #53

Closed
skliper opened this issue Sep 25, 2019 · 4 comments
Closed

Volume Table Requires User Modification #53

skliper opened this issue Sep 25, 2019 · 4 comments
Labels
bug Something isn't working
Milestone

Comments

@skliper
Copy link
Contributor

skliper commented Sep 25, 2019

The volume table definition is source file that is included with each PSP implementation. The implementation source code files are not intended to be modified. The volume table however, requires user modification to specify the volumes and their physical locations specific to a project/mission.

Adding a volume table macro is a simple solution to allow volume information to be specified at build time.

@skliper
Copy link
Contributor Author

skliper commented Sep 25, 2019

Imported from trac issue 49. Created by sstrege on 2016-02-10T15:19:51, last modified: 2019-04-02T14:49:34

@skliper skliper added this to the psp-1.3.0 milestone Sep 25, 2019
@skliper skliper self-assigned this Sep 25, 2019
@skliper skliper added the bug Something isn't working label Sep 25, 2019
@skliper
Copy link
Contributor Author

skliper commented Sep 25, 2019

Trac comment by sstrege on 2016-02-10 15:23:01:

commit ID: [changeset:d423ce6]

Adds VOLTAB macro to cfe_psp_voltab.c
Rather than only being able to use the default /cf physical location or having each developer make local changes to the voltab.c file, this modification allows the location of the /cf directory to be specified at build time(it could be provided on the command line or as we use the option at JSC, set in the top level makefile).
If the default location is acceptable, then developers do nothing new at build time.

@skliper
Copy link
Contributor Author

skliper commented Sep 25, 2019

Trac comment by jphickey on 2016-02-10 15:36:21:

Pushed commit [changeset:023719e] as discussed at the 2016-02-10 CCB, containing this change isolated from the rest of #14

Branch trac-49-voltab-macros contains this change set for merge.

@skliper
Copy link
Contributor Author

skliper commented Sep 25, 2019

Trac comment by jhageman on 2019-04-02 14:49:34:

Milestone renamed

@skliper skliper closed this as completed Sep 25, 2019
@skliper skliper removed their assignment Sep 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant