search

cern-eos / eos

EOS Storage
Other
211 stars 39 forks source link

installation docs: fix table syntax for dependency repo, add double t… #46

Closed andresailer closed 7 months ago

andresailer commented 7 months ago

…icks to prevent unicodisation of yum commands

Dear EOS folks,

In your documentation page https://eos-docs.web.cern.ch/diopside/manual/hardware-installation.html#eos-for-client-server

The table syntax is broken for the "dependency repos" which makes it disappear on the actual page Here is a patch


 +-----------------+--------------------------------------------------------------------------------------------------------------------+
 | Platform        | Setup                                                                                                              | 
 +=================+=============================================================================================== =====================+
 | CentOS7         | yum-config-manager --add-repo "https://storage-ci.web.cern.ch/storage-ci/eos/diopside-depend/el-7/x86_64/"         |
 +-----------------+--------------------------------------------------------------------------------------------------------------------+
 | Alma 8          | yum-config-manager --add-repo "https://storage-ci.web.cern.ch/storage-ci/eos/diopside-depend/el-8/x86_64/"         |
 +-----------------+--------------------------------------------------------------------------------------------------------------------+
-| Alma 9          | yum-config-manager --add-repo "https://storage-ci.web.cern.ch/storage-ci/eos/diopside-depend/el-9/x86_64/"        |
+| Alma 9          | yum-config-manager --add-repo "https://storage-ci.web.cern.ch/storage-ci/eos/diopside-depend/el-9/x86_64/"         |
 +-----------------+--------------------------------------------------------------------------------------------------------------------+
 | Fedora 38       | https://storage-ci.web.cern.ch/storage-ci/eos/diopside-depend/fc-38/                                               |
 +-----------------+--------------------------------------------------------------------------------------------------------------------+

And the page turns the commands into unicode -- and " " which breaks copy paste, which is really annoying, I think surrounding things with double ticks should work.

Thanks for your consideration!

esindril commented 7 months ago

Thanks for your contribution, this will be released in 5.2.20.