[Ci-notification] jabberwocky_master_build - Build # 77 - Still Unstable!

[Alexandre Montplaisir]

This job is considered unstable because of the Maven warnings. See [1].

Some of them are real compiler warnings, so that definitely justifies a
UNSTABLE tag. Some others we could fix if we really wanted to. However,
the real problematic one is:

> Source root doesn't exist:
/home/jenkins/workspace/jabberwocky_master_build/jabberwocky-ctf/src/main/java

I honestly don't know how we could avoid those. The reason is that many
artifacts currently contain both Java and Kotlin sources. Normally you
would use a different Maven configuration depending if you're building
Java-only, Kotlin-only, or hybrid sources.
However the Jabberwocky/Scope projects contain a bit of everything, so
the easiest, most maintainable way in this case is to use the hybrid
config[2] for *everything*, defining it once in a parent and having all
sub-modules inherit it.
This then causes Maven warnings when it can't find either Java or Kotlin
sources.

The most "Maven" way of solving this cleanly would be to use Maven
profiles, and active the "Java-only" profile when only the java source
directory (src/main/java) is found, activate the Kotlin-only profile
when there are only Kotlin sources (src/main/kotlin), and activate the
hybrid profile when both are present. However this is currently not
possible to do with Maven, because profile activations can only use one
check... see [3] (opened 10 years ago :/ )


So yeah I'm not sure what's the best approach here. Disable Maven
warnings and only use UNSTABLE for test failures?

Cheers,
Alex

[1] https://ci.lttng.org/job/jabberwocky_master_build/77/warnings38Result/
[2]
https://kotlinlang.org/docs/reference/using-maven.html#compiling-kotlin-and-java-sources
[3] https://issues.apache.org/jira/browse/MNG-3328


On 2017-12-14 16:58, jenkins at lttng.org wrote:
> jabberwocky_master_build - Build # 77 - Still Unstable:
>
> Check console output at https://ci.lttng.org/job/jabberwocky_master_build/77/ to view the results.