Skip to content
This repository has been archived by the owner on Jul 27, 2022. It is now read-only.

Updates jastadd gradle version #2

Merged
merged 1 commit into from Nov 19, 2015
Merged

Updates jastadd gradle version #2

merged 1 commit into from Nov 19, 2015

Conversation

emsod
Copy link
Contributor

@emsod emsod commented Nov 19, 2015

No description provided.

emsod added a commit that referenced this pull request Nov 19, 2015
Updates jastadd gradle version
@emsod emsod merged commit 591e909 into master Nov 19, 2015
@emsod emsod deleted the fix-build branch November 19, 2015 17:56
@@ -49,7 +49,7 @@ jastadd {
parser.name = 'JavaParser'
}

generateJava << {
generateJava {
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the right thing to do here is to use

generateJava {
    doFirst { ... }

Using doFirst makes it execute before the other steps in this build task.

@llbit
Copy link
Contributor

llbit commented Nov 19, 2015

This works for me. Looks good for now, though I want to tidy up this build script a bit.

@emsod emsod mentioned this pull request Nov 19, 2015
@emsod
Copy link
Contributor Author

emsod commented Nov 19, 2015

Thanks for the comments. I added an issue about the script clean up.

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.

None yet

2 participants