Skip to content
This repository has been archived by the owner on Jun 14, 2024. It is now read-only.

v0.2.0 + remove all traces of Gradle Software Model #62

Merged
merged 3 commits into from
Sep 26, 2016

Conversation

loosebazooka
Copy link
Contributor

It appears that the Gradle Software Model for java projects is
not moving forward. Change code back to classic plugin format -
using extensions and project callbacks.

This code change affects how to configure the plugin now,
the "appengine" configuration block is now under project, not model.

OLD :
model {
appengine {
}
}

NEW :
appengine {
}

@loosebazooka
Copy link
Contributor Author

@patflynn @meltsufin plugin rewrite.

It appears that the Gradle Software Model for java projects is
not moving forward. Change code back to classic plugin format -
using extensions and project callbacks.

This code change affects how to configure the plugin now,
the "appengine" configuration block is now under project, not model.

OLD :
model {
  appengine {
  }
}

NEW :
appengine {
}
1. Signing is handled by the nexus plugin
2. Make sure you run --no-daemon when doing an upload
@@ -18,7 +18,7 @@ sourceCompatibility = JavaVersion.VERSION_1_7
targetCompatibility = JavaVersion.VERSION_1_7

group = 'com.google.cloud.tools'
version = '0.1.2-SNAPSHOT'
version = '0.2.0-SNAPSHOT'
Copy link
Contributor

Choose a reason for hiding this comment

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

@meltsufin @loosebazooka

Why are we still on 0.2.0 if these are 1.0 RC ready?

Copy link
Member

Choose a reason for hiding this comment

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

For Maven, I was thinking next release would be 1.0.0.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

mostly just because I change a lot of the internals and would like to play around with it for a week before switching to 1.0.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

or does 1.0.0-rc basically imply the same thing?

@meltsufin
Copy link
Member

install comes up with this warning:

warning: [options] bootstrap class path not set in conjunction with -source 1.7
1 warning

@loosebazooka
Copy link
Contributor Author

Yeah cause you're probably building with java8. You can build with java7 or bypass that warning by specifying the bootstrap class in your build.gradle

compileJavaTask.options.bootClasspath = (new File(java7Home, "/jre/lib/rt.jar")).canonicalPath

@meltsufin
Copy link
Member

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants