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

Update install instructions #10

Merged
merged 8 commits into from
Jun 3, 2017
Merged

Update install instructions #10

merged 8 commits into from
Jun 3, 2017

Conversation

tsroten
Copy link
Member

@tsroten tsroten commented May 26, 2017

This:

  • adds a note about Python 2.6
  • adds Fedora install instructions
  • updates wording on the page


I haven't built an RPM package for mycli yet. So please use `pip` to install `mycli`. You can install pip on your system using:
We don't have an RPM package for mycli, yet. So, please use `pip` to install `mycli`. You can install pip on your system using:
Copy link
Member

Choose a reason for hiding this comment

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

If people are able to install it on Fedora doesn't that mean there is an RPM package for it?

/cc @meeuw

Copy link
Member Author

Choose a reason for hiding this comment

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

Good question. Maybe you or @meeuw can help me figure out what better wording for that is.

Is there an easy, non-pip way to install mycli on RHEL or Centos?

If not, is it better to simply say "We don't have packages for RHEL or Centos, yet. So, please use pip..."?

Copy link
Contributor

Choose a reason for hiding this comment

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

The RPM can be used on Centos/RHEL but most of the dependencies are missing. I've created a repository with all the dependencies on https://copr.fedorainfracloud.org/coprs/meeuw/mycli-epel/

I'd like to get this into Fedora-EPEL but haven't found the time to do this.

Copy link
Member Author

Choose a reason for hiding this comment

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

@meeuw Great, no problem.

So, for now, does this sound ok? "We don't have packages for RHEL or Centos, yet. Instead, use pip..."

@amjith

tsroten added 4 commits May 31, 2017 07:19
…ll_instructions

* 'master' of github.com:dbcli/mycli.net:
  Remove prompt from sample .my.cnf.
  Add mysql section note to config docs.
  Add pager instructions.
  Add note about editing previous command.
  Add introduction and headings.
  Update config and add mysql option file.
@meeuw
Copy link
Contributor

meeuw commented May 31, 2017

Hmm, I though I've added a comment this morning, but I don't see it anymore.

I'm working on getting the dependencies in EPEL, I've already packaged them:
https://copr.fedorainfracloud.org/coprs/meeuw/mycli-epel/
(this can already be used as a copr yum/dnf repository)

I'm not sure if we should publish it right now.

@tsroten
Copy link
Member Author

tsroten commented May 31, 2017

@meeuw - Thanks for being so thorough. Your comment is there, you have to click on "Show Outdated", though.

I reworded the docs in this pull request. Then, whenever it changes in the future, we can update it then. Thanks!

@meeuw
Copy link
Contributor

meeuw commented May 31, 2017

👍 This is fine for now

### RHEL, Centos, Fedora:
### Fedora

:::bash
Copy link
Member

Choose a reason for hiding this comment

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

This triple colon is not rendering properly in md.

Is that intentional?

Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm, yeah, I think that's how Pelican does code blocks. Github-flavored Markdown is different.

To be honest, I was imitating the triple-colons that were already in the file. Looking at Pelican's docs, it seem like what they recommend (and when you run the development server, it renders).

screen shot 2017-06-02 at 21 18 16

@amjith
Copy link
Member

amjith commented Jun 3, 2017

🦊

@amjith amjith merged commit 9478d8e into master Jun 3, 2017
@amjith amjith deleted the update_install_instructions branch June 3, 2017 03:32
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.

3 participants