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

Upgrade to latest sbt version and refactor Java version validation #42

Merged
merged 4 commits into from
Feb 1, 2015

Conversation

gkossakowski
Copy link

Check individual commit messages for details.

The changes I'm submitting will make it possible to add scalariform to community build.

This makes it easier to add scalariform to the community build.
Previously, validation of required version was done during loading of
build definition. This was needlessly restricting because not everything
in scalariform requires particular Java version. After closer inspection,
it turns out that only the `misc` subproject depends on Swing and cares
about the specific Java version.

We refactor validation logic to run right before `compile` task in `misc`
subproject. That makes it possible to ignore this subproject and build the
rest of scalariform on any Java version.
gkossakowski added a commit to gkossakowski/community-builds that referenced this pull request Jan 23, 2015
The git reference points at pull request that fixes some issues in
scalariform's build definition that prevented it from building by dbuild.
Check the PR for details:
daniel-trinh/scalariform#42
When resolving Scala-version specific dependencies, match any Scala 2.11
release and not just 2.11.0.
Match in the build definition Scala version that includes qualifier.
The qualifier is in the form of hyphen and word characters following
minor version. E.g. this change enables matching Scala version like:

2.11.6-dbuildx7765235bd36ec60fc0c730b29c692e1bc2864526

that are being used by dbuild.
@daniel-trinh daniel-trinh merged commit e68c882 into daniel-trinh:master Feb 1, 2015
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