Add support for eclipse 4.13 #482

k-brooks · 2019-10-31T21:46:11Z

Part 1 of 2: _ext wrapper updates to support 4.13
I broke out (and ordered) the commits by subproject intentionally, although, without project linking / publishing, I have not been able to properly build/test the dependent projects.
I have not attempted to publish, wasn't clear on the process.
Part 2 will come once these libraries have published.

Would you be interested in having the ext projects cleaned up? It would be nice to have project dependencies across these, and potentially the 'root' project.


        eclipse-base update for 4.13 (#480)


        eclipse-jdt update for 4.13 (#480)


        eclipse-cdt update for 4.13 (#480)


        eclipse-wtp update for 4.13 (#480)


        eclipse-groovy update for 4.13 (#480)

k-brooks · 2019-10-31T21:50:31Z

_ext/eclipse-cdt/gradle.properties

@@ -11,11 +11,10 @@ ext_group=com.diffplug.spotless
 ext_VER_JAVA=1.8

 # Compile dependencies
-VER_ECLIPSE_CDT=9.7


I'm wondering if this was a bug in the eclipse-cdt release of 9.8, as it referenced 9.7

Good catch! I looked at it, and I believe you are correct. I should publish a 9.8.1 bugfix.

Thanks for the fix. I really missed to commit the vital change.

nedtwigg

Looks like a great start! From here, you are blocked on me publishing these new _ext jars, I'll let you know when that is complete. I'll push some small changes to this branch as well.

Re: combining build with the main project. The problem is that the builds for these are really slow because of p2, and I don't know of a good way to speed them up. I am open to a build refactor (best to do in a separate PR to prevent blocking this PR), but it needs to not slow down the current build. Everything in _ext is pretty specialized, it's important for regular java devs to not have to deal with this when contributing normal mavenCentral-based formatters.

nedtwigg · 2019-10-31T22:07:00Z

_ext/eclipse-base/gradle.properties

@@ -1,7 +1,7 @@
 # Mayor versions correspond to the supported Eclipse core version.
 # Minor version is incremented for features or incompatible changes (including changes to supported dependency versions).
 # Patch version is incremented for backward compatible patches of this library.
-ext_version=3.2.1


If we don't change the code in eclipse-base, then we don't need to publish a new eclipse-base. We only change this when something about the OSGi plumbing needs to change, which is generally rare.

I was under the impression that the compile-scoped dependencies were included in the fat jar, but if that is incorrect, then I can back this and the related changes out.
Are you OK with me rewriting these commits to suit, or would you prefer new commits?

nedtwigg · 2019-10-31T22:07:00Z

_ext/eclipse-cdt/gradle.properties

@@ -11,11 +11,10 @@ ext_group=com.diffplug.spotless
 ext_VER_JAVA=1.8

 # Compile dependencies
-VER_ECLIPSE_CDT=9.7


Good catch! I looked at it, and I believe you are correct. I should publish a 9.8.1 bugfix.


        Revert "eclipse-base update for 4.13 (#480)"

This reverts commit 8f167b8.


        Revert "eclipse-jdt update for 4.13 (#480)"

This reverts commit 1260246.


        Revert "eclipse-cdt update for 4.13 (#480)"

This reverts commit 85e2f33.

nedtwigg · 2019-10-31T22:20:59Z

@fvgh before I publish this, can you confirm that this is true:

we don't need to publish a new ext-eclipse-base
we don't need to publish a new ext-eclipse-jdt

This is because we can just update lockfiles for jdt. I do need to publish new jars for groovy, cdt, and wtp. Are we missing anything important?


        Publish eclipse-cdt 9.8.1.


        Publish eclipse-cdt 9.9.0.

nedtwigg · 2019-11-01T07:56:08Z

Can you give me push permissions on your PR branch?

https://github.com/Vestmark/spotless: git-receive-pack not permitted on 'https://github.com/Vestmark/spotless/'

If not, can you move your issue_480-ext to match the DiffPlug repo? I published the eclipse-cdt 9.8.1 bundled jar, and updated the corresponding lockfile. I also published eclipse-cdt 9.9.0, and I leave it to you to do the lockfile. When we get confirmation from @fvgh that we have done this correctly, then we can move forward with wtp, groovy, and jdt.

fvgh · 2019-11-01T09:11:07Z

Thanks for all your effort. I am really busy at the moment, hence I did not find time to contribute. So if the _ext tests are passing I am fine with the changes. If you get stuck anywhere, let me know. I'll try to answer within 24 hours.

k-brooks · 2019-11-01T12:02:00Z

Can you give me push permissions on your PR branch?
https://github.com/Vestmark/spotless: git-receive-pack not permitted on 'https://github.com/Vestmark/spotless/'
If not, can you move your issue_480-ext to match the DiffPlug repo? I published the eclipse-cdt 9.8.1 bundled jar, and updated the corresponding lockfile. I also published eclipse-cdt 9.9.0, and I leave it to you to do the lockfile. When we get confirmation from @fvgh that we have done this correctly, then we can move forward with wtp, groovy, and jdt.

My apologies @nedtwigg , I thought had checked that box, but perhaps you require permissions at the repo level? I granted you maintenance permissions for the Vestmark fork - if that does not work, then I will push this branch to the Diffplug repo, instead.

k-brooks · 2019-11-01T12:10:18Z

Looks like a great start! From here, you are blocked on me publishing these new _ext jars, I'll let you know when that is complete. I'll push some small changes to this branch as well.

Re: combining build with the main project. The problem is that the builds for these are really slow because of p2, and I don't know of a good way to speed them up. I am open to a build refactor (best to do in a separate PR to prevent blocking this PR), but it needs to not slow down the current build. Everything in _ext is pretty specialized, it's important for regular java devs to not have to deal with this when contributing normal mavenCentral-based formatters.

#483 opened to capture the build work, will comment with thoughts on that thread.

k-brooks · 2019-11-01T16:55:40Z

_ext/eclipse-cdt/CHANGES.md

+
+### Version 9.8.1 - October 31st 2019 ([artifact]([jcenter](https://bintray.com/diffplug/opensource/spotless-eclipse-cdt)))
+
+* Really publish Eclipse CDT release 9.8 for Eclipse 4.12 ([#482](https://github.com/diffplug/spotless/pull/482)).


k-brooks · 2019-11-01T16:56:44Z

_ext/eclipse-cdt/gradle.properties

 ext_artifactId=spotless-eclipse-cdt
-ext_description=Eclipse's CDT C/C++ formatter bundled for Spotless
+ext_description=Eclipse''s CDT C/C++ formatter bundled for Spotless


not a huge deal, but a typo here, I can fix it but didn't want to mess with your work

The syntax highlighter in VSCode seemed to think I needed to escape the single-quote. That surprised me, and I think it turns out to be wrong. A fix would be appreciated!

.properties spec does not need double-quote

.properties resource bundle does need double-quote

nedtwigg · 2019-11-01T17:40:59Z

I think we have published everything we need so that you can now do the lockfiles for CDT and for JDT. When those are working, I'll publish the jars for WTP and groovy, and then you can do the lockfiles for those too, and then we can merge and publish!


        revert extra quote (#480)


        remove version bump to eclipse base (#480)


        cdt/jdt eclipse 4.13 support (#480)

k-brooks · 2019-11-01T18:46:45Z

JDT & CDT are up, thanks again!


        Bump the eclips-base dependency from 3.2.0 to 3.2.1 in the 4.13.0 loc…

…kfiles for cdt/jdt.


        Publish groovy 3.5.0

nedtwigg · 2019-11-01T19:49:06Z

In the lockfiles, even though eclipse-base can be 3.2.0, we get a bugfix if we ask for 3.2.1.

ext-eclipse-groovy 3.5.0 is up and ready for lockfiles.

ext-eclipse-wtp 3.15.0 is erroring for me, so it is not up:

> gradle -b _ext/eclipse-wtp/build.gradle jar
...
Could not resolve all files for configuration ':compileClasspath'.
> Could not find any version that matches org.eclipse.platform:org.eclipse.core.filebuffers:[3.8.0,4.0.0[.
  Versions that do not match:
      3.6.700
      3.6.600
      3.6.500
      3.6.400
      3.6.300
      + 3 more
  Searched in the following locations:
      https://repo.maven.apache.org/maven2/org/eclipse/platform/org.eclipse.core.filebuffers/maven-metadata.xml
      https://jcenter.bintray.com/org/eclipse/platform/org.eclipse.core.filebuffers/maven-metadata.xml
      file:/Users/ntwigg/.m2/repository/org/eclipse/platform/org.eclipse.core.filebuffers/maven-metadata.xml
      file:/Users/ntwigg/.m2/repository/org/eclipse/platform/org.eclipse.core.filebuffers/
      file:/Users/ntwigg/Documents/dev/spotless/_ext/eclipse-wtp/build/p2asmaven/maven/org/eclipse/platform/org.eclipse.core.filebuffers/maven-metadata.xml
      file:/Users/ntwigg/Documents/dev/spotless/_ext/eclipse-wtp/build/p2asmaven/maven/org/eclipse/platform/org.eclipse.core.filebuff


        eclipse platform versioning split (#480)

k-brooks · 2019-11-01T21:35:23Z

@nedtwigg - sorry for the compile error - looks like the two artifacts (osgi services and filebuffers) diverged in their versioning, updated accordingly
Lock file(s) to follow shortly


        eclipse 4.13 groovy formatter (#480)

k-brooks · 2019-11-04T15:55:29Z

@nedtwigg - I have the wtp formatter lock file staged, ready when you are :-)

fvgh

The rest looks good to me. Thanks again for your work.

...es/com/diffplug/spotless/extra/groovy_eclipse_formatter/v4.13.0.lockfile


        Publish eclipse-wtp 3.15.0

@fvgh


        Remove SWT from the groovy lockfile, per @fvgh's advice

nedtwigg · 2019-11-06T19:30:50Z

eclipse-wtp 3.15.0 is published (sorry for delay)

k-brooks · 2019-11-06T23:03:14Z

Looks like wtp 3.15.0 has not propagated?
Caused by: org.eclipse.aether.resolution.ArtifactResolutionException: Could not find artifact com.diffplug.spotless:spotless-eclipse-wtp:jar:3.15.0 in google-maven-central


        Eclipse 4.13 support for wtp (#480)


        Remove swt dependency from cdt lockfile (#480)

nedtwigg · 2019-11-06T23:17:48Z

It claims to be published here: https://bintray.com/diffplug/opensource/spotless-eclipse-wtp#

And the mavenCentral sync also claims to be successful. I restart the CI build later today.

k-brooks · 2019-11-06T23:20:24Z

Odd - I was able to run tests locally (assuming travis is running gradlew check, or some derivative thereof), I did not see that failure.

nedtwigg · 2019-11-07T17:52:39Z

I reran the build, looks like we're passing CI now! This looks ready to merge to me, except it needs changelog entries for plugin-maven and plugin-gradle.


        Update changelogs (#480)

k-brooks · 2019-11-07T18:22:54Z

changelogs updated, thanks again for all your help @nedtwigg !

nedtwigg · 2019-11-12T05:51:10Z

Released in x.26.0

k-brooks added 5 commits Oct 31, 2019

eclipse-base update for 4.13 (#480)

8f167b8

eclipse-jdt update for 4.13 (#480)

1260246

eclipse-cdt update for 4.13 (#480)

85e2f33

eclipse-wtp update for 4.13 (#480)

02e3223

eclipse-groovy update for 4.13 (#480)

Loading status checks…

1b8ef41

k-brooks reviewed Oct 31, 2019

View changes

nedtwigg reviewed Oct 31, 2019

View changes

nedtwigg added 3 commits Oct 31, 2019

Revert "eclipse-base update for 4.13 (#480)"

ce6662f

This reverts commit 8f167b8.

Revert "eclipse-jdt update for 4.13 (#480)"

d4d01bf

This reverts commit 1260246.

Revert "eclipse-cdt update for 4.13 (#480)"

6c452fe

This reverts commit 85e2f33.

nedtwigg added 2 commits Nov 1, 2019

Publish eclipse-cdt 9.8.1.

Loading status checks…

0b00d79

Publish eclipse-cdt 9.9.0.

Loading status checks…

d1c8512

fvgh approved these changes Nov 1, 2019

View changes

k-brooks reviewed Nov 1, 2019

View changes

k-brooks added 3 commits Nov 1, 2019

revert extra quote (#480)

Loading status checks…

8c55363

remove version bump to eclipse base (#480)

1c4cb71

cdt/jdt eclipse 4.13 support (#480)

Loading status checks…

4b63e09

nedtwigg added 2 commits Nov 1, 2019

Bump the eclips-base dependency from 3.2.0 to 3.2.1 in the 4.13.0 loc…

15b33b6

…kfiles for cdt/jdt.

Publish groovy 3.5.0

Loading status checks…

e4ee903

nedtwigg changed the title ~~Issue 480 ext~~ Add support for eclipse 4.13 Nov 1, 2019

eclipse platform versioning split (#480)

Loading status checks…

1ad30a8

eclipse 4.13 groovy formatter (#480)

Loading status checks…

0309d60

fvgh requested changes Nov 4, 2019

View changes

...es/com/diffplug/spotless/extra/groovy_eclipse_formatter/v4.13.0.lockfile Outdated Show resolved Hide resolved

nedtwigg added 2 commits Nov 6, 2019

Publish eclipse-wtp 3.15.0

Loading status checks…

008ac4f

Remove SWT from the groovy lockfile, per @fvgh's advice

Loading status checks…

7b249b9

Eclipse 4.13 support for wtp (#480)

Loading status checks…

3c86339

k-brooks force-pushed the Vestmark:issue_480-ext branch from 373b738 to 3c86339 Nov 6, 2019

Remove swt dependency from cdt lockfile (#480)

Loading status checks…

52a1d95

Update changelogs (#480)

Loading status checks…

75f32f9

nedtwigg merged commit 04d5c7d into diffplug:master Nov 7, 2019
1 check passed

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

k-brooks deleted the Vestmark:issue_480-ext branch Nov 7, 2019

Aug	SEP	Oct
	16
2019	2020	2021

diffplug / spotless

Add support for eclipse 4.13 #482

Add support for eclipse 4.13 #482

k-brooks commented Oct 31, 2019

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

nedtwigg left a comment

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

nedtwigg commented Oct 31, 2019

nedtwigg commented Nov 1, 2019

fvgh commented Nov 1, 2019

k-brooks commented Nov 1, 2019 •

edited

k-brooks commented Nov 1, 2019

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

nedtwigg commented Nov 1, 2019

k-brooks commented Nov 1, 2019

nedtwigg commented Nov 1, 2019

k-brooks commented Nov 1, 2019

k-brooks commented Nov 4, 2019

fvgh left a comment

nedtwigg commented Nov 6, 2019

k-brooks commented Nov 6, 2019

nedtwigg commented Nov 6, 2019

k-brooks commented Nov 6, 2019

nedtwigg commented Nov 7, 2019

k-brooks commented Nov 7, 2019

nedtwigg commented Nov 12, 2019


		### Version 9.8.1 - October 31st 2019 ([artifact]([jcenter](https://bintray.com/diffplug/opensource/spotless-eclipse-cdt)))

		* Really publish Eclipse CDT release 9.8 for Eclipse 4.12 ([#482](https://github.com/diffplug/spotless/pull/482)).

diffplug / spotless

Join GitHub today

Add support for eclipse 4.13 #482

Add support for eclipse 4.13 #482

Conversation

k-brooks commented Oct 31, 2019

This comment has been minimized.

k-brooks Oct 31, 2019 Author Contributor

This comment has been minimized.

nedtwigg Oct 31, 2019 Member

This comment has been minimized.

fvgh Nov 1, 2019 Contributor

nedtwigg left a comment

This comment has been minimized.

nedtwigg Oct 31, 2019 Member

This comment has been minimized.

k-brooks Nov 1, 2019 Author Contributor

This comment has been minimized.

nedtwigg Oct 31, 2019 Member

nedtwigg commented Oct 31, 2019

nedtwigg commented Nov 1, 2019

fvgh commented Nov 1, 2019

k-brooks commented Nov 1, 2019 • edited

k-brooks commented Nov 1, 2019

This comment has been minimized.

k-brooks Nov 1, 2019 Author Contributor

This comment has been minimized.

k-brooks Nov 1, 2019 Author Contributor

This comment has been minimized.

nedtwigg Nov 1, 2019 Member

nedtwigg commented Nov 1, 2019

k-brooks commented Nov 1, 2019

nedtwigg commented Nov 1, 2019

k-brooks commented Nov 1, 2019

k-brooks commented Nov 4, 2019

fvgh left a comment

nedtwigg commented Nov 6, 2019

k-brooks commented Nov 6, 2019

nedtwigg commented Nov 6, 2019

k-brooks commented Nov 6, 2019

nedtwigg commented Nov 7, 2019

k-brooks commented Nov 7, 2019

nedtwigg commented Nov 12, 2019

k-brooks commented Nov 1, 2019 •

edited