search

devonfw / IDEasy

Tool to automate the setup and updates of a development environment for any project (Successor of devonfw-ide).
Apache License 2.0
8 stars 23 forks source link

Again messages break processable command output #754

Open hohwille opened 6 days ago

hohwille commented 6 days ago 
$ ide -v
Duplicate variable definition MVN_VERSION with old value '0.1' and new value '3.9.6' in D:\projects\salog\settings\devon.properties
Both legacy variable MAVEN_VERSION and official variable MVN_VERSION are configured in D:\projects\salog\settings\devon.properties - ignoring legacy variable declaration!
2024.11.001-beta-SNAPSHOT
bash: $'\E[93mDuplicate': command not found
bash: $'\E[93mBoth': command not found

See #627 and #531. This error was introduce by me with PR #746

Adding a log warning to some reasonable place should not lead to this error.

It is not acceptable that before adding a log warning you always have to understand if that may be triggered during context initialization since this way this error will come back again and again.

We need to find a way to finally fix this problem once and forever!

hohwille commented 4 days ago

was auto-closed by PR. But still a bullet-proof solution is needed. Hence reopening...

hohwille commented 23 hours ago

Still buggy after merging PR #755 

$ ide
bash: Skipping: command not found
IDE environment variables have been set for /d/projects/IDEasy in workspace main

Problem was this message: https://github.com/devonfw/IDEasy/blob/2966a61ee492d58798b7bc82bb790e0c3b311114/cli/src/main/java/com/devonfw/tools/ide/commandlet/CreateCommandlet.java#L63

This one is interesting since this message is also in info level. IMHO we need to invent our own log-level for this.