You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ariatosca.apache.org by vishwanathj <gi...@git.apache.org> on 2017/08/18 16:48:46 UTC
[GitHub] incubator-ariatosca pull request #192: ARIA-352 Package name "aria[ssh]" inc...
GitHub user vishwanathj opened a pull request:
https://github.com/apache/incubator-ariatosca/pull/192
ARIA-352 Package name "aria[ssh]" incorrect in the installation docum…
…ent link
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vishwanathj/incubator-ariatosca master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-ariatosca/pull/192.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #192
----
commit 21d4de6ace881c9e44370eb294cb45695b0e90ea
Author: Vishwanath Jayaraman <vi...@hotmail.com>
Date: 2017-08-18T16:40:44Z
ARIA-352 Package name "aria[ssh]" incorrect in the installation document link
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-ariatosca pull request #192: ARIA-352 Package name "aria[ssh]" inc...
Posted by vishwanathj <gi...@git.apache.org>.
Github user vishwanathj commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/192#discussion_r134246108
--- Diff: README.rst ---
@@ -53,7 +53,7 @@ To install ARIA from source, download the source tarball from
**Ubuntu/Debian** (tested on Ubuntu14.04, Ubuntu16.04)::
apt-get install -y python-dev gcc libffi-dev libssl-dev
- pip install aria[ssh]
+ pip install apache-ariatosca[ssh]
--- End diff --
Please go ahead and merge. Thanks for your guidance and help.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-ariatosca issue #192: ARIA-352 Package name "aria[ssh]" incorrect ...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit commented on the issue:
https://github.com/apache/incubator-ariatosca/pull/192
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-ariatosca pull request #192: ARIA-352 Package name "aria[ssh]" inc...
Posted by ran-z <gi...@git.apache.org>.
Github user ran-z commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/192#discussion_r134171829
--- Diff: README.rst ---
@@ -53,7 +53,7 @@ To install ARIA from source, download the source tarball from
**Ubuntu/Debian** (tested on Ubuntu14.04, Ubuntu16.04)::
apt-get install -y python-dev gcc libffi-dev libssl-dev
- pip install aria[ssh]
+ pip install apache-ariatosca[ssh]
--- End diff --
Thanks. Now I just have to ask that you squash the two commits into a single one, and make sure you're rebased on top of master once you do that - then I'll be able to merge this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-ariatosca pull request #192: ARIA-352 Package name "aria[ssh]" inc...
Posted by vishwanathj <gi...@git.apache.org>.
Github user vishwanathj commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/192#discussion_r134239981
--- Diff: README.rst ---
@@ -53,7 +53,7 @@ To install ARIA from source, download the source tarball from
**Ubuntu/Debian** (tested on Ubuntu14.04, Ubuntu16.04)::
apt-get install -y python-dev gcc libffi-dev libssl-dev
- pip install aria[ssh]
+ pip install apache-ariatosca[ssh]
--- End diff --
I am seeing the below failure in AppVeyor build after squishing the commits, is there a way to restart the AppVeyor build?
git clone -q https://github.com/apache/incubator-ariatosca.git C:\projects\incubator-ariatosca
fatal: unable to access 'https://github.com/apache/incubator-ariatosca.git/': Failed to connect to github.com port 443: Timed out
Command exited with code 128
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-ariatosca pull request #192: ARIA-352 Package name "aria[ssh]" inc...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-ariatosca/pull/192
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-ariatosca pull request #192: ARIA-352 Package name "aria[ssh]" inc...
Posted by ran-z <gi...@git.apache.org>.
Github user ran-z commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/192#discussion_r134244356
--- Diff: README.rst ---
@@ -53,7 +53,7 @@ To install ARIA from source, download the source tarball from
**Ubuntu/Debian** (tested on Ubuntu14.04, Ubuntu16.04)::
apt-get install -y python-dev gcc libffi-dev libssl-dev
- pip install aria[ssh]
+ pip install apache-ariatosca[ssh]
--- End diff --
Yeah, it seems like Github has been malfunctioning today..
I don't have permissions for the AppVeyor account, only Apache Infra does; Supposedly it's easiest to simply re-push (after "modifying" the commit, e.g. by empty rebase).
Since this is only a README change, though, we could simply merge despite the tests having failed. It's simply less pretty to see the red X :)
Up to you, let me know whether you'd like to push again or I should merge right now
Thanks :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-ariatosca pull request #192: ARIA-352 Package name "aria[ssh]" inc...
Posted by vishwanathj <gi...@git.apache.org>.
Github user vishwanathj commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/192#discussion_r134123326
--- Diff: README.rst ---
@@ -53,7 +53,7 @@ To install ARIA from source, download the source tarball from
**Ubuntu/Debian** (tested on Ubuntu14.04, Ubuntu16.04)::
apt-get install -y python-dev gcc libffi-dev libssl-dev
- pip install aria[ssh]
+ pip install apache-ariatosca[ssh]
--- End diff --
Sure, will update the CentOS, ArchLinux and Windows section as well
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-ariatosca pull request #192: ARIA-352 Package name "aria[ssh]" inc...
Posted by ran-z <gi...@git.apache.org>.
Github user ran-z commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/192#discussion_r134108006
--- Diff: README.rst ---
@@ -53,7 +53,7 @@ To install ARIA from source, download the source tarball from
**Ubuntu/Debian** (tested on Ubuntu14.04, Ubuntu16.04)::
apt-get install -y python-dev gcc libffi-dev libssl-dev
- pip install aria[ssh]
+ pip install apache-ariatosca[ssh]
--- End diff --
This change should also be applied to the other platforms - i.e. it should be "apache-ariatosca" on centos/windows/.. as well.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---