-
Notifications
You must be signed in to change notification settings - Fork 33
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
docs(linux): Security: Add SELinux documentation #93
docs(linux): Security: Add SELinux documentation #93
Conversation
New warnings found with rstcheck:
|
New warnings found with rstcheck:
|
1ce835a
to
c92b6cc
Compare
New warnings found with rstcheck:
|
c92b6cc
to
1ac781b
Compare
New warnings found with rstcheck:
|
1ac781b
to
d04aa2f
Compare
New warnings found with rstcheck:
|
source/linux/Foundational_Components/System_Security/SELinux.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/System_Security/SELinux.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/System_Security/SELinux.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/System_Security/SELinux.rst
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@praneethbajjuri Are we confident enough to callout selinux support in AM62X, AM62A and AM62P device specific Release Notes under new features in SDK 10.1? [and eventually point it to source/linux/Foundational_Components/System_Security/SELinux.rst
]
Similar to what we did for other set of features in https://software-dl.ti.com/processor-sdk-linux/esd/AM62X/latest/exports/docs/devices/AM62X/linux/Release_Specific_Release_Notes.html#what-s-new
@ti-kamlesh please review. |
source/linux/Foundational_Components/System_Security/SELinux.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/System_Security/SELinux.rst
Outdated
Show resolved
Hide resolved
d04aa2f
to
8169ddf
Compare
New warnings found with rstcheck:
|
8169ddf
to
75dd813
Compare
New warnings found with rstcheck:
|
Thanks for the reviews @cshilwant @StaticRocket. @ti-kamlesh please review |
source/linux/Foundational_Components/System_Security/SELinux.rst
Outdated
Show resolved
Hide resolved
source/linux/Foundational_Components/System_Security/SELinux.rst
Outdated
Show resolved
Hide resolved
New warnings found with rstcheck:
|
1 similar comment
New warnings found with rstcheck:
|
@AashvijShenai run rstchecker and fix delta warnings reported. |
New warnings found with rstcheck:
|
850de56
to
cb36fc7
Compare
New warnings found with rstcheck:
|
cb36fc7
to
ba8b461
Compare
Updated the section depths and added the label |
New warnings found with rstcheck:
|
ba8b461
to
c17980f
Compare
New warnings found with rstcheck:
|
Evidentially, a section cannot have the same name as a file. This cannot be bypassed through the use of labels. |
@StaticRocket any idea why the above is happening? |
source/linux/Foundational_Components/System_Security/SELinux.rst
Outdated
Show resolved
Hide resolved
This document details how to build and use SELinux on Yocto and Debian distributions. In order to prepare for the upcoming system security documentation, this guide has been separated into another directory: System_Security. Signed-off-by: Aashvij Shenai <[email protected]>
c17980f
to
7e3ac14
Compare
I think we should go ahead and add in release notes |
@praneethbajjuri Let me know your take on this. Based on which we can decide either this PR needs works or is good to be pulled in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Depending how the release note is addressed, this is fine for now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pulling this for now.
@AashvijShenai touchbase with @praneethbajjuri to see if this has to be highlighted in release notes (which can go in as a seperate PR)
@AashvijShenai I don't see your section under left navigation bar under - https://texasinstruments.github.io/processor-sdk-doc/processor-sdk-linux-AM62X/esd/docs/master/linux/Foundational_Components.html Was this intentional? If not, you might have to update https://github.com/TexasInstruments/processor-sdk-doc/blob/master/source/linux/Foundational_Components.rst for the same |
This document details how to build and use SELinux on Yocto and Debian distributions.
In order to prepare for the upcoming system security documentation, this guide has been separated into another directory: System_Security.