#201 - Reduce log level for unknown dependencies to debug to reduce l…

[ercpe]

…og spam when using a shared missing file

#201

[ppalaga]

The proposed solution eliminates the warnings also for users that want to see them and I cannot accept it like this. I propose to solve the issue by adding a parameter much like we did in AbstractDownloadLicensesMojo

license-maven-plugin/src/main/java/org/codehaus/mojo/license/AbstractDownloadLicensesMojo.java

Lines 319 to 333 in 1a890f1

	/**
	* What to do on any license download related error. The possible values are:
	* <li>
	* <ul>{@link ErrorRemedy#ignore}: all errors are ignored</ul>
	* <ul>{@link ErrorRemedy#warn}: all errors are output to the log as warnings</ul>
	* <ul>{@link ErrorRemedy#failFast}: a {@link MojoFailureException} is thrown on the first download related
	* error</ul>
	* <ul>{@link ErrorRemedy#xmlOutput}: error messages are added as {@code <downloaderMessages>} to
	* {@link AbstractDownloadLicensesMojo#licensesErrorsFile}; in case there are error messages, the build will
	* fail after processing all dependencies</ul>
	* </li>
	* @since 1.18
	*/
	@Parameter( property = "license.errorRemedy", defaultValue = "warn" )
	protected ErrorRemedy errorRemedy;

I'd recommend to add your own ErrorRemedy-like enum that would initially contain just warn (default) and ignore so that you not need to implement the other options we have in AbstractDownloadLicensesMojo.

Do not forget to add an integration test.

[guidograzioli]

I'm interested in this change, is anybody currently working on it? Happy to help otherwise

[ercpe]

Typo throughout the change Unkown -> Unknown

[ercpe]

Probably cleaner to hand id down to handleUnkownDependency instead of passing the log message around.

[wuwu2000]

postbuild integration tests seem to run not 100% stable -,.-

[wuwu2000]

can someone tell me why some builds fail randomly?

[wuwu2000]

@ppalaga Could you see why this jdk7 build fails please?