[ptxdist] [PATCH] doc: environment: make toolchain directory user writeable as root

[ptxdist] [PATCH] doc: environment: make toolchain directory user writeable as root

[Philipp Zabel]

Creating the toolchain directory under /opt and making it writeable to
the user should be done as root. Make this very obvious by prepending
the commans with sudo.

Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
 doc/environment.rst | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/doc/environment.rst b/doc/environment.rst
index 18aecec4eb0b..7b0da683b10e 100644
--- a/doc/environment.rst
+++ b/doc/environment.rst
@@ -484,9 +484,9 @@ enter:
 
 ::
 
-   $ mkdir /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
-   $ chown <username> /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
-   $ chmod a+rwx /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
+   $ sudo mkdir /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
+   $ sudo chown <username> /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
+   $ sudo chmod a+rwx /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
 
 We recommend to keep this installation path as PTXdist expects the
 toolchains in ``/opt``. Whenever we go to select a platform in a
-- 
2.20.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

Re: [ptxdist] [PATCH] doc: environment: make toolchain directory user writeable as root

[Michael Olbrich]

On Thu, May 28, 2020 at 10:05:14AM +0200, Philipp Zabel wrote:
> Creating the toolchain directory under /opt and making it writeable to
> the user should be done as root. Make this very obvious by prepending
> the commans with sudo.

Hmm, it does say that you probably need root permissions to do this in the
paragraph above. I'm not sure if adding sudo here helps any.

Also, I need to rewrite the whole section because the latest toolchains are
handled differently.

Michael

> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> ---
>  doc/environment.rst | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/doc/environment.rst b/doc/environment.rst
> index 18aecec4eb0b..7b0da683b10e 100644
> --- a/doc/environment.rst
> +++ b/doc/environment.rst
> @@ -484,9 +484,9 @@ enter:
>  
>  ::
>  
> -   $ mkdir /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
> -   $ chown <username> /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
> -   $ chmod a+rwx /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
> +   $ sudo mkdir /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
> +   $ sudo chown <username> /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
> +   $ sudo chmod a+rwx /opt/OSELAS.Toolchain-|oselasTCNVendorVersion|
>  
>  We recommend to keep this installation path as PTXdist expects the
>  toolchains in ``/opt``. Whenever we go to select a platform in a
> -- 
> 2.20.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

Re: [ptxdist] [PATCH] doc: environment: make toolchain directory user writeable as root

[Philipp Zabel]

On Fri, May 29, 2020 at 10:14:44AM +0200, Michael Olbrich wrote:
> On Thu, May 28, 2020 at 10:05:14AM +0200, Philipp Zabel wrote:
> > Creating the toolchain directory under /opt and making it writeable to
> > the user should be done as root. Make this very obvious by prepending
> > the commans with sudo.
> 
> Hmm, it does say that you probably need root permissions to do this in the
> paragraph above. I'm not sure if adding sudo here helps any.

It's just a suggestion, this way people who like to copy & paste code
snippets would be less likely to miss the comment about required root
permissions above.

> Also, I need to rewrite the whole section because the latest toolchains are
> handled differently.

Ok, no worries.

regards
Philipp

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

Re: [ptxdist] [PATCH] doc: environment: make toolchain directory user writeable as root

[Christian Hermann]

Hello list,

see below.

Kind regards

On 29.05.20 10:28, Philipp Zabel wrote:
> On Fri, May 29, 2020 at 10:14:44AM +0200, Michael Olbrich wrote:
>> On Thu, May 28, 2020 at 10:05:14AM +0200, Philipp Zabel wrote:
>>> Creating the toolchain directory under /opt and making it writeable to
>>> the user should be done as root. Make this very obvious by prepending
>>> the commans with sudo.
>>
>> Hmm, it does say that you probably need root permissions to do this in the
>> paragraph above. I'm not sure if adding sudo here helps any.There's a convention to prefix commands requiring root's privilege with
'#' instead of '$'.
> 
> It's just a suggestion, this way people who like to copy & paste code
> snippets would be less likely to miss the comment about required root
> permissions above.
> 
You should never copy/paste text obtained from a browser into your
terminal emulator as many don't provide proper paste protection[1].
Or at least be very sure to know how your emulator works in this regard.
[1] https://lwn.net/Articles/749992/

>> Also, I need to rewrite the whole section because the latest toolchains are
>> handled differently.
> 
> Ok, no worries.
> 
> regards
> Philipp
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de