From 839572074f231cca1c9e14ee660b05dfe51131d0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Christoph=20L=C3=A4ubrich?= Date: Fri, 19 Jan 2024 16:11:38 +0100 Subject: [PATCH] Remove warning about unspecified environment Currently if not target platform environment is specified Tycho emits a warning "No explicit target runtime environment configuration. Build is platform dependent." beside from that nothing ever seems go wrong and actually Tycho builds *can* be environment agnostic. Also there is not really a way to tell Tycho that it is okay, so either the user has to use a specific one or configure many ones that make resolving slower. This now removes this warning, if with modern Tycho there are any problems that arise from a missing environment configuration one should better give a specific error/hint at this place instead of warn "just in case". --- .../core/resolver/DefaultTargetPlatformConfigurationReader.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/tycho-core/src/main/java/org/eclipse/tycho/core/resolver/DefaultTargetPlatformConfigurationReader.java b/tycho-core/src/main/java/org/eclipse/tycho/core/resolver/DefaultTargetPlatformConfigurationReader.java index 176fac7edd..169ea2941a 100644 --- a/tycho-core/src/main/java/org/eclipse/tycho/core/resolver/DefaultTargetPlatformConfigurationReader.java +++ b/tycho-core/src/main/java/org/eclipse/tycho/core/resolver/DefaultTargetPlatformConfigurationReader.java @@ -167,8 +167,6 @@ public TargetPlatformConfiguration getTargetPlatformConfiguration(MavenSession s } if (result.getEnvironments().isEmpty()) { - // applying defaults - logger.warn("No explicit target runtime environment configuration. Build is platform dependent."); result.addEnvironment(TargetEnvironment.getRunningEnvironment()); result.setImplicitTargetEnvironment(true); } else {