-
Notifications
You must be signed in to change notification settings - Fork 3k
Fix various issues in refreshed quarkus update
#46772
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Bold and underline were leaking at times - Fix the wording of a few sentences
This is more consistent with a diff output.
Also improve a bit the formatting of the log file.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't test it, but code looks good! Thanks!
We might need to store the logs in a buffer before writing them in the file (to avoid file locking): |
Status for workflow
|
@gsmet any reason to do |
Ah you already removed the clean 🙏 (ignore my previous messages) |
I wanted to try this on my wife's Windows machine, but I hit #46792 |
...ls-common/src/main/java/io/quarkus/devtools/project/update/rewrite/QuarkusUpdateCommand.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fixes #46639, and the update command is now finished successfully.
After installing git and cloning I could build Quarkus, on that non-dev machine it took 33 minutes with -Dquickly -Dno-test-modules
:/ And the time wasn't massively spent on downloads as I have a fibre channel connection at home :)
Just for the record, I initially built the fork on my dev machine and copied quarkus-cli-999-SNAPSHOT-runner.jar to the Windows machine, but it's not working approach as update is called through quarkus maven plugin. So that's why I needed to build and install the bits on Windows machine ...
I started having a look as bold and underline were leaking (it's very visible when running the tests from https://github.com/quarkusio/quarkus-updates) and things led to others.
One important thing is that the
target/rewrite
directory was dropped by themvn clean process-sources
so you ended up with no logs being available.I did a lot of manual testing on my side.
This needs to go in before we push the first final
3.20
.Fixes #46639 (per Rostislav's comment below)