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.

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

Make a Markdown version of INSTALL #579

Closed
wants to merge 1 commit into from
Closed

Make a Markdown version of INSTALL #579

wants to merge 1 commit into from

Conversation

larsnaesbye
Copy link
Contributor

First version. Probably needs some cleanup afterwards.

@larsnaesbye larsnaesbye mentioned this pull request Jan 9, 2020
Copy link
Member

@375gnu 375gnu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not every issue, just some i noticed

Here's the table which provides executable file names accordingly to interface:

Interface | Executable name
- | :-
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

either an error or non-standard md extension

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non-standard extension. But GitHub supports tables like that.
Else we need another way to make tables. Markdown supports HTML tables, but that's usually not elegant IMHO.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No sorry, it is an error.

| NATIVE_OSX_APP | bool | OFF | Support native OSX data paths

Notes:
\* /usr/local on Unix-like systems, c:\Program Files or c:\Program Files (x86)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in this block
should be added at the end of lines


Configure and build:

`mkdir build
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

``` should be used here


Configure and build:

`mkdir build`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

``` should be used here

Development snapshots installation on Unix-like systems:
### On Debian 10 and derived systems:

`curl https://download.opensuse.org/repositories/home:/munix9:/unstable/Debian_10/Release.key`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

must be

  curl https://download.opensuse.org/repositories/home:/munix9:/unstable/Debian_10/Release.key | sudo apt-key add -
  echo "deb https://download.opensuse.org/repositories/home:/munix9:/unstable/Debian_10/ ./" | sudo tee /etc/apt/sources.list.d/celestia-obs.list
  sudo apt update && sudo apt install celestia

* Check https://celestia.space/download.html.
* Check https://bintray.com/celestia if it contains packages for your system.

Stable version installation on Windows and OSX:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OSX should be replaced with macOS

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, althought it isn't in the text-only version either.

@levinli303
Copy link
Collaborator

Looks good to me, but please squash your commits

@larsnaesbye
Copy link
Contributor Author

Dammit. I picked wrong before the push. Sorry. Closing this and opening another instead :-/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants