You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jclouds.apache.org by Everett Toews <no...@github.com> on 2014/01/16 18:23:11 UTC

[jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

You can merge this Pull Request by running:

  git pull https://github.com/rackerlabs/jclouds-site openstack-pom

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds-site/pull/53

-- Commit Summary --

  * Added pom.xml file for OpenStack dependencies

-- File Changes --

    M documentation/quickstart/openstack.markdown (101)

-- Patch Links --

https://github.com/jclouds/jclouds-site/pull/53.patch
https://github.com/jclouds/jclouds-site/pull/53.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by Everett Toews <no...@github.com>.
>  
> -This is a list of providers that work with OpenStack that you can use to build your Context.
> +This pom.xml file specifies all of the dependencies you'll need to work with OpenStack. Replace the jclouds.version X.X.X with the latest version available according to the [Release Notes](http://jclouds.apache.org/documentation/releasenotes/).

Ya. Way too confusing.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53/files#r8942559

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by Everett Toews <no...@github.com>.
I think it's okay where it is for now. I purposefully put it at the bottom so it's out of the way but the user can still easily browse it. Making it a separate download makes this much more difficult. 

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53#issuecomment-32537486

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by Everett Toews <no...@github.com>.
>  
> -* `"openstack-nova"`
> -* `"openstack-keystone"`
> -* `"openstack-cinder"`
> -* `"swift-keystone"`
> +<pre>
> +{% highlight xml %}

Regarding copying, not that I'm aware of. The code samples will look better after the rebranding.

I very purposefully didn't use a Gist because it would be under my account and impossible for anyone else to update.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53/files#r8942852

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by Andrew Phillips <no...@github.com>.
>  
> -* `"openstack-nova"`
> -* `"openstack-keystone"`
> -* `"openstack-cinder"`
> -* `"swift-keystone"`
> +<pre>
> +{% highlight xml %}

OK!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53/files#r8946375

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by Andrew Phillips <no...@github.com>.
>  
> -This is a list of providers that work with OpenStack that you can use to build your Context.
> +This pom.xml file specifies all of the dependencies you'll need to work with OpenStack. Replace the jclouds.version X.X.X with the latest version available according to the [Release Notes](http://jclouds.apache.org/documentation/releasenotes/).

Link `pom.xml` to http://maven.apache.org/pom.html, or too confusing?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53/files#r8942262

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by Everett Toews <no...@github.com>.
Merged and deployed.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53#issuecomment-32491683

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by Andrew Phillips <no...@github.com>.
>  
> -* `"openstack-nova"`
> -* `"openstack-keystone"`
> -* `"openstack-cinder"`
> -* `"swift-keystone"`
> +<pre>
> +{% highlight xml %}

Does Jekyll support some way of making a block of text easy to copy to the clipboard? Or would it be worth also having a link to a Gist or something for easy downloading?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53/files#r8942652

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by Andrew Phillips <no...@github.com>.
>  
> -* `"openstack-nova"`
> -* `"openstack-keystone"`
> -* `"openstack-cinder"`
> -* `"swift-keystone"`
> +<pre>
> +{% highlight xml %}

> I very purposefully didn't use a Gist

True, true. Worth putting a downloadable file in jclouds-site?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53/files#r8944786

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-site-pull-requests #173](https://jclouds.ci.cloudbees.com/job/jclouds-site-pull-requests/173/) SUCCESS
This pull request looks good

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53#issuecomment-32490619

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by jclouds-commentator <no...@github.com>.
  Go to http://57ba79a217fcb8a72ca2-fddd3ead1441f877364b1610c17eeaf7.r63.cf1.rackcdn.com/ to review your changes.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53#issuecomment-32490835

Re: [jclouds-site] Added pom.xml file for OpenStack dependencies (#53)

Posted by Everett Toews <no...@github.com>.
>  
> -* `"openstack-nova"`
> -* `"openstack-keystone"`
> -* `"openstack-cinder"`
> -* `"swift-keystone"`
> +<pre>
> +{% highlight xml %}

I think it's okay where it is for now. I purposefully put it at the bottom so it's out of the way but the user can still easily browse it. Making it a separate download makes this much more difficult.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/53/files#r8946184