You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@jclouds.apache.org by Jeremy Daggett <no...@github.com> on 2014/05/20 21:50:39 UTC

[jclouds-labs] Fixes jclouds-representations pom artifactId to use correct spelling (#64)

This PR fixes a simple typo in the jclouds-representations project poms.

Please refer to the [Maven Central Repository](http://search.maven.org/#artifactdetails%7Corg.apache.jclouds.labs%7Cjclouds-represetations%7C1.7.2%7Cpom)

The incorrect spelling of the artifactId has been present since [1.6.1-incubating](http://search.maven.org/#search%7Cgav%7C1%7Cg%3A%22org.apache.jclouds.labs%22%20AND%20a%3A%22jclouds-represetations%22)
You can merge this Pull Request by running:

  git pull https://github.com/rackerlabs/jclouds-labs fix-pom-typo

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

  https://github.com/jclouds/jclouds-labs/pull/64

-- Commit Summary --

  * Fixed artifactId typo

-- File Changes --

    M jclouds-representations/pom.xml (4)
    M jclouds-representations/representations-codec/pom.xml (2)
    M jclouds-representations/representations-core/pom.xml (2)

-- Patch Links --

https://github.com/jclouds/jclouds-labs/pull/64.patch
https://github.com/jclouds/jclouds-labs/pull/64.diff

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

Re: [jclouds-labs] Fixes jclouds-representations pom artifactId to use correct spelling (#64)

Posted by BuildHive <no...@github.com>.

[jclouds » jclouds-labs #1063](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs/1063/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/64#issuecomment-43676156

Re: [jclouds-labs] Fixes jclouds-representations pom artifactId to use correct spelling (#64)

Posted by Jeremy Daggett <no...@github.com>.

Closed #64.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/64#event-124657471

Re: [jclouds-labs] Fixes jclouds-representations pom artifactId to use correct spelling (#64)

Posted by Ignasi Barrera <no...@github.com>.

> Do we need downstream changes in Karaf to match this?

I've had a look and [jclouds-karaf](https://github.com/jclouds/jclouds-karaf/blob/master/feature/src/main/resources/feature.xml#L53-L54) already used the good name. I'm wondering why its builds didn't fail...
Anyway, I think it's good to merge.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/64#issuecomment-43852696

Re: [jclouds-labs] Fixes jclouds-representations pom artifactId to use correct spelling (#64)

Posted by CloudBees pull request builder plugin <no...@github.com>.

[jclouds-labs-pull-requests #157](https://jclouds.ci.cloudbees.com/job/jclouds-labs-pull-requests/157/) SUCCESS
This pull request looks good

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/64#issuecomment-43676166

Re: [jclouds-labs] Fixes jclouds-representations pom artifactId to use correct spelling (#64)

Posted by Andrew Phillips <no...@github.com>.

Change itself looks fine. Do we need downstream changes in Karaf to match this?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/64#issuecomment-43775949

Re: [jclouds-labs] Fixes jclouds-representations pom artifactId to use correct spelling (#64)

Posted by Jeremy Daggett <no...@github.com>.

@demobox Thanks! I just successfully built Karaf and the dependency was updated to correctly match this change.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/64#issuecomment-43794379

Re: [jclouds-labs] Fixes jclouds-representations pom artifactId to use correct spelling (#64)

Posted by Andrew Phillips <no...@github.com>.

> I just successfully built Karaf and the dependency was updated to correctly match this change.

Thanks for checking, @jdaggett!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/64#issuecomment-44152685

Re: [jclouds-labs] Fixes jclouds-representations pom artifactId to use correct spelling (#64)

Posted by Jeremy Daggett <no...@github.com>.

Merged to master 9f4b28a

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/64#issuecomment-44152605

Re: [jclouds-labs] Fixes jclouds-representations pom artifactId to use correct spelling (#64)

Posted by Ignasi Barrera <no...@github.com>.

LGTM. Thanks @jdaggett !

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/64#issuecomment-43722149