Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Creating Strongback gradle project structure #108

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

jmcshane
Copy link

@jmcshane jmcshane commented Mar 9, 2017

Resolves #92

NB: The ant structure will be maintained and this is still in progress

Please feel free to comment on the PR and address issues as I continue to work on this. I'm hoping to be able to help the teams that have an existing maven/gradle structure in their projects to be able to plugin strongback and strongback-testing right into their code.

Description

This conversion will allow the ant structure to be maintained and all existing ant tools to be preserved while allowing others to reference this gradle project and get the transitive dependencies that come with the project.

PR Status

Complete

  • Refactoring to a directory structure that will fit more easily into a gradle build
  • write build.gradle file for the base project
  • regression test all documented ant functionality after refactor

To Do

  • instructions for downloading third party dependencies for compilation when referencing strongback in a gradle/maven project (ctre, nvax, wpilib) in progress
  • define publishing strategy for strongback.jar and strongback-testing.jar will leave this to repository owners
  • provide example project using the gradle structure in progress

@jmcshane
Copy link
Author

I'm working on resolving this failure now, check out the gradle-download-externals branch in my fork of this repository. I have wpilib resolving, now I just need to add the other ones. The code is rough for now, but I'm going to create a gradle plugin for downloading the latest version of each from the vendor.

jmcshane and others added 3 commits March 11, 2017 12:30
…rence any currently deployed version of wpilib to compile against
…on existing ant tasks, while using the gradle plugin for wpilib to specify the exact version rather than downloading from releases. This will force compilation of strongback to specific wpilib version and can publish multiple artifacts against these versions
Gradle - resolve external dependencies
@jmcshane jmcshane changed the title (WIP: Review only) Converting to a gradle project Creating Strongback gradle project structure Mar 11, 2017
Copy link

@andrewda andrewda left a comment

Choose a reason for hiding this comment

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

Other than these small styling inconsistencies, looks good! 👍

build-common.xml Outdated
<!-- Check if there are any JUnit Tests -->
<target name="if-tests">
<condition property="tests-exists">
<available file="${project.test.source.dir}" type="dir"/>
</condition>
</target>
</target>

Choose a reason for hiding this comment

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

Remove extra spaces.

build.gradle Outdated

dependencies {
testCompile('junit:junit:4.12')
testCompile('org.easytesting:fest-assert:1.4')

Choose a reason for hiding this comment

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

Fix spacing.

build.gradle Outdated
gradleVersion = '2.10'
}

ant.importBuild 'third-party.xml'

Choose a reason for hiding this comment

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

Almost all other files use spaces. Is there a reason this one uses tabs?

Copy link
Author

Choose a reason for hiding this comment

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

good call, I'll update this

@jmcshane
Copy link
Author

jmcshane commented May 1, 2017

Thanks for the review. I'll try to get these changes pushed out this evening

@jmcshane
Copy link
Author

jmcshane commented May 2, 2017

All comments above have been resolved by 4af573a.

Copy link

@andrewda andrewda left a comment

Choose a reason for hiding this comment

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

Looks good 👍

CarlinWilliamson added a commit to Team3132/strongback-java that referenced this pull request Jun 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants