[GitHub] [incubator-nuttx] acassis commented on a change in pull request #4564: Doc: Update download version to 10.1 and fix tar command

[GitBox <gi...@apache.org>]

acassis commented on a change in pull request #4564:
URL: https://github.com/apache/incubator-nuttx/pull/4564#discussion_r710385050



##########
File path: Documentation/quickstart/install.rst
##########
@@ -165,22 +165,22 @@ Apache NuttX is actively developed on GitHub. There are two main repositories, `
        $ cd nuttx
        $ curl -L https://github.com/apache/incubator-nuttx/tarball/master -o nuttx.tar.gz 
        $ curl -L https://github.com/apache/incubator-nuttx-apps/tarball/master -o apps.tar.gz
-       $ tar zxf nuttx.tar.gz
-       $ tar zxf apps.tar.gz
+       $ tar zxf nuttx.tar.gz --one-top-level=nuttx --strip-components 1
+       $ tar zxf apps.tar.gz --one-top-level=apps --strip-components 1
        
     There are also ``.zip`` archives available (useful for Windows users): just replace ``tarball`` with
     ``zipball``.
 
   .. tab:: Download stable release
   
     Go to `releases <https://nuttx.apache.org/download/>`_ and choose a version to download. The following
-    example uses version 9.1.0:
+    example uses version 10.1.0:
 
     .. code-block:: console
     
        $ mkdir nuttx
        $ cd nuttx
-       $ curl -L https://downloads.apache.org/incubator/nuttx/9.1.0/apache-nuttx-9.1.0-incubating.tar.gz -o nuttx.tar.gz 
-       $ curl -L https://downloads.apache.org/incubator/nuttx/9.1.0/apache-nuttx-apps-9.1.0-incubating.tar.gz -o apps.tar.gz
-       $ tar zxf nuttx.tar.gz
-       $ tar zxf apps.tar.gz
+       $ curl -L https://downloads.apache.org/incubator/nuttx/10.1.0/apache-nuttx-10.1.0-incubating.tar.gz -o nuttx.tar.gz 
+       $ curl -L https://downloads.apache.org/incubator/nuttx/10.1.0/apache-nuttx-apps-10.1.0-incubating.tar.gz -o apps.tar.gz
+       $ tar zxf nuttx.tar.gz --one-top-level=nuttx --strip-components 1

Review comment:
       > Is this actually needed for the apache releases or just the GitHub generated ones? The release validation script does not use this, so I am fairly certain this change is not needed here.
   
   Done!




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org