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

JENKINS-21377 adapt to tycho-compiler-jdt 0.19.0 #35

Merged
merged 2 commits into from Jan 17, 2014

Conversation

codesimplicity
Copy link
Contributor

This shall fix https://issues.jenkins-ci.org/browse/JENKINS-21377.
Adapted the parser regexp to work with both "[line, column]" and "[line]" as location specs in the Java warning. The latter will be generated by org.eclipse.tycho:tycho-compiler-jdt:0.19.0.

org.eclipse.tycho:tycho-compiler-jdt:0.19.0 generates locations as "[line]" not "[line, column]" as before;
see https://issues.jenkins-ci.org/browse/JENKINS-21377
@cloudbees-pull-request-builder

plugins » warnings-plugin #104 FAILURE
Looks like there's a problem with this pull request

@codesimplicity
Copy link
Contributor Author

Looks like the build is broken since https://jenkins.ci.cloudbees.com/job/plugins/job/warnings-plugin/102/, hope this will be fixed soon.

@uhafner
Copy link
Member

uhafner commented Jan 15, 2014

Can you please add a new testcase? (Or add a new line to the existing eclipse log so that it will be parsed now).

@cloudbees-pull-request-builder

plugins » warnings-plugin #105 FAILURE
Looks like there's a problem with this pull request

@codesimplicity
Copy link
Contributor Author

Good point, test case added.
Still missing the snapshot version of analysis-core though :-(

@uhafner
Copy link
Member

uhafner commented Jan 17, 2014

Our CI build is not set up to build snapshot dependencies...

Thanks for the pull request!

uhafner added a commit that referenced this pull request Jan 17, 2014
[FIXED JENKINS-21377] Update of regexp for tycho-compiler-jdt 0.19.0
@uhafner uhafner merged commit 5e1fc87 into jenkinsci:master Jan 17, 2014
uhafner added a commit that referenced this pull request Aug 3, 2018
@codesimplicity codesimplicity deleted the JENKINS-21377 branch August 29, 2022 12:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants