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.

Sign up for GitHub

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

Jump to bottom

Clarify pipe name scope in documentation #665

Closed
skliper opened this issue May 1, 2020 · 0 comments · Fixed by #1191
Closed

Clarify pipe name scope in documentation #665

skliper opened this issue May 1, 2020 · 0 comments · Fixed by #1191
Assignees
@zanzaben
Labels
docs This change only affects documentation. enhancement
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented May 1, 2020

Is your feature request related to a problem? Please describe.
From #650 (comment)

Although the Pipe names will not collide with other Application Pipe names in the cFE

This not correct, because pipe names WILL conflict with other application pipe names within the cFE. It is actually OSAL that enforces uniqueness of queue names which underpin the SB pipes.

Describe the solution you'd like
Update the line in the documentation.

Describe alternatives you've considered
None

Additional context
See discussion on #650

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added docs This change only affects documentation. enhancement labels May 1, 2020
@skliper skliper added this to the 6.8.0 milestone May 1, 2020
@skliper skliper modified the milestones: 6.8.0, 7.0.0 Jun 5, 2020
zanzaben added a commit to zanzaben/cFE that referenced this issue Feb 25, 2021
@zanzaben
Fix nasa#665, update pipe name documentation.
8a1894c
@zanzaben zanzaben mentioned this issue Mar 1, 2021
Merged
@skliper skliper closed this as completed in 80fca94 Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zanzaben zanzaben

Labels
docs This change only affects documentation. enhancement
Projects
None yet
Milestone
Caelum
Development

Successfully merging a pull request may close this issue.

Fix #665, update pipe name documentation. zanzaben/cFE
2 participants
@zanzaben @skliper