closing as https://github.com/jclouds/jclouds-examples/pull/92 supercedes it
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/88#issuecomment-374141186
Closed #88.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/88#event-1527961463
Agreed @andrewgaul, jclouds-examples as-is is not useful.
I'll try to open several subsequent PR to fix the project, maybe one maven
module for each PR?
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jcl
jclouds-examples is a dumpster fire; thanks for looking into this! This pull
request has a lot going on -- can you break out repairing bit rot into separate
smaller commits? You can also make this smaller by consistently using the
parent pom `jclouds.version` property. I made some progress in
> @@ -20,34 +20,34 @@
> http://maven.apache.org/POM/4.0.0";
> xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
> xsi:schemaLocation="http://maven.apache.org/POM/4.0.0
> http://maven.apache.org/xsd/maven-4.0.0.xsd";>
>4.0.0
> +
> +org.apache.jclouds.examples
> +jclou
> Should this be added as post-step in the release guide?
Sounds like a good idea. Better still, how about a "small shell script" to
automatically upgrade things? ;-)
Or more seriously, a step in the script that at least replaces the version
number in the examples POM and _attempts_ to compile
Yes, +1 to keep all examples using the latest stable release.
Should this be added as post-step in the release guide?
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-examples/pull/88#issueco
> can be to try to keep jclouds-examples in sync with latest release instead of
> the bleeding edge master. wdyt?
I think that was exactly the intention, and that we've simply fallen behind ;-)
So definitely a +1 on that from me.
Many thanks for helping get this back up-to-date!
---
You are re
thx @demobox I didn't know about the original intention.
Although I think it is a good idea, I think it doesn't work terribly well long
term, most of the examples are referencing very old jclouds releases and they
are not really helpful for new users.
I think a half way solution can be to try to
> @@ -20,34 +20,34 @@
> http://maven.apache.org/POM/4.0.0";
> xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
> xsi:schemaLocation="http://maven.apache.org/POM/4.0.0
> http://maven.apache.org/xsd/maven-4.0.0.xsd";>
>4.0.0
> +
> +org.apache.jclouds.examples
> +jclou
first stab at the maven structure
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds-examples/pull/88
-- Commit Summary --
* refactor maven structure
-- File Changes --
M blobstore-basics/pom.xml (18)
M
blobstore-basics/src/main/java
11 matches
Mail list logo