Skip to content

extract java, google java format and jmh to separate script plugins#401

Closed
qweek wants to merge 1 commit into
rsocket:1.0.xfrom
qweek:feature/extract-script-plugins-from-build
Closed

extract java, google java format and jmh to separate script plugins#401
qweek wants to merge 1 commit into
rsocket:1.0.xfrom
qweek:feature/extract-script-plugins-from-build

Conversation

@qweek

@qweek qweek commented Oct 2, 2017 •
edited
Loading

Copy link
Copy Markdown
Contributor

configuration:

  • explicitly set gradle version and distribution type (upgrade later)

versions:

  • move dependency versions to separate file

java:

  • set encoding to UTF-8
  • disable specific compiler warnings (fix later)

codestyle:

  • set google java format version for plugin (upgrade later)
  • run formatting before compilation

benchmark:

  • exclude test dependencies from jmh jar
  • fix duplicate jmh generated classes (jmh-generated-classes, jmh-generated-sources)
  • fix broken jmh metadata files (BenchmarkList, CompilerHints)
  • fix jmh command line jar (inline dependencies with shadow plugin)
  • fix jmh task for root scope (stop generate dummy jar)
  • cleanup redundant out directory generated by IDEA
  • remove slf4j and log4j dependencies (replace with slf4j-nop logger)
  • add visual jmh report

@qweek
extract java, google java format and jmh to separate script plugins
f8f8657 
configuration:
  explicitly set gradle version and distribution type (upgrade later)
versions:
  move dependency versions to separate file
java:
  set encoding to UTF-8
  disable specific compiler warnings (fix later)
codestyle:
  set google java format version for plugin (upgrade later)
  run formatting before compilation
benchmark:
  exclude test dependencies from jmh jar
  fix duplicate jmh generated classes (jmh-generated-classes, jmh-generated-sources)
  fix broken jmh metadata files (BenchmarkList, CompilerHints)
  fix jmh command line jar (inline dependencies with shadow plugin)
  fix jmh task for root scope (stop generate dummy jar)
  cleanup redundant out directory generated by IDEA
  remove slf4j and log4j dependencies (replace with slf4j-nop logger)
  add visual jmh report
@qweek

qweek commented Oct 2, 2017 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Feel free to suggest a better name for script plugins

cli example:
java -jar build/libs/rsocket-core-0.9-SNAPSHOT-jmh.jar io.rsocket.RSocketPerf.fireAndForgetHello -t 2

known issues:

  • Exception while trying to acquire the JMH lock on Windows (C:\Windows/jmh.lock)

workaround:

  • Run IDE as administrator

@qweek

qweek commented Oct 2, 2017 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

I see your pull request #397 (inline versions)
Do you have any concerns with the previous solution?
Let me know if you need help solving this problem

@yschimke

yschimke commented Oct 3, 2017

Copy link
Copy Markdown
Member

cc @robertroeser

I'll defer to Robert here. I got the gradle scripts working for snapshots and releases, but apart from that, I'm not very fussed about the structure if it's working.

@qweek qweek mentioned this pull request Oct 3, 2017
Closed
@robertroeser

robertroeser commented Oct 4, 2017

Copy link
Copy Markdown
Member

I'd prefer to leave it in one file. It works now, and it's easy to find what we're doing. I don't want to dig through a bunch of different files to figure out what the version of something is, or what configuration we're using.

@qweek

qweek commented Oct 5, 2017

Copy link
Copy Markdown
Contributor Author

If you wish I can merge this changes to main build.gradle file

@qweek qweek mentioned this pull request Oct 5, 2017
Merged
@qweek qweek deleted the feature/extract-script-plugins-from-build branch October 5, 2017 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@qweek @yschimke @robertroeser