Skip to content

#1264: Fix ide.bat to detect errors properly#1274

Merged
hohwille merged 15 commits intodevonfw:mainfrom
alfeilex:#1264-fix-bat-err-handling
May 15, 2025
Merged

#1264: Fix ide.bat to detect errors properly#1274
hohwille merged 15 commits intodevonfw:mainfrom
alfeilex:#1264-fix-bat-err-handling

Conversation

@alfeilex
Copy link
Member

@alfeilex alfeilex commented Apr 28, 2025

Fixes: #1264

This PR fixes the ide.bat to detect errors properly. The %ERRORLEVEL% is moved to outside the brackets to be set with the correct exit code that is given by the previous ideasy command. If ideasy is run without arguments, then the error handling is skipped to prevent that previous %ERRORLEVEL% values that have been set by other commands, are not used in the error handing.

I tested the behavior by comparing the output of various ide commands in git-bash and cmd side by side and they behave equal.

You can test the behavior by replacing the code of ide.bat in projects/_ide/installation/bin/ide.bat

@github-project-automation github-project-automation bot moved this to 🆕 New in IDEasy board Apr 28, 2025
@alfeilex alfeilex moved this from 🆕 New to Team Review in IDEasy board Apr 28, 2025
@alfeilex alfeilex self-assigned this Apr 28, 2025
@coveralls
Copy link
Collaborator

coveralls commented Apr 28, 2025

Pull Request Test Coverage Report for Build 14997240991

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 67.679%

Totals Coverage Status
Change from base Build 14993147256: 0.0%
Covered Lines: 7974
Relevant Lines: 11374

💛 - Coveralls

@alfeilex alfeilex marked this pull request as ready for review May 6, 2025 08:28
@alfeilex alfeilex marked this pull request as draft May 6, 2025 08:29
@alfeilex alfeilex marked this pull request as ready for review May 6, 2025 10:47
@alfeilex alfeilex changed the title #1264: Fix ide.bat detects errors properly #1264: Fix ide.bat to detect errors properly May 6, 2025
@hohwille hohwille added this to the release:2025.05.001 milestone May 13, 2025
@hohwille hohwille merged commit c7a8df0 into devonfw:main May 15, 2025
4 checks passed
@github-project-automation github-project-automation bot moved this from Team Review to ✅ Done in IDEasy board May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

ide.bat not detecting errors properly

3 participants