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

A few loose bits in JUnit 4 migration #32

Merged
merged 1 commit into from
May 26, 2017
Merged

Conversation

twogee
Copy link
Contributor

@twogee twogee commented May 26, 2017

This adds annotations that are missing, migrates example code and fixes whitespace

@twogee
Copy link
Contributor Author

twogee commented May 26, 2017

Why is Jenkins lacking org.apache.tools.ant.util.ProcessUtil? Something wrong with Ant?

@hibnico
Copy link
Contributor

hibnico commented May 26, 2017

The job was configured to use the default Ant, I don't know which version exactly. So I have changed the conf to use Ant 1.9.9 (the same version that the one declared in the ivy.xml of Ivy).
A new build is coming.

@twogee
Copy link
Contributor Author

twogee commented May 26, 2017

Unless I'm mistaken, BuildFileRule was available first in 1.9.5; ProcessUtil needs 1.9.4

@twogee
Copy link
Contributor Author

twogee commented May 26, 2017

Jenkins build was complete an hour ago, but status on Github is still pending...

@asfgit asfgit merged commit fbbf092 into apache:master May 26, 2017
asfgit pushed a commit that referenced this pull request May 26, 2017
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.

3 participants