davsclaus commented on PR #29428:
URL: https://github.com/apache/camel/pull/29428#issuecomment-6735811645
@ammachado thanks for the review, both points are fixed in 936e87faed62:
1. **`--version LTS`**: `camel self-update --version LTS` (or `--version
X.Y`) now installs the newest release of that selector through `fetchAlias` and
pins it, as `install.sh --version LTS` does. A release line refuses an alias
that points outside the line. This path never worked, with or without a nightly
pin. Tests in `SelfUpdateIntegrationTest`:
- `ltsLeavesTheNightliesAndPinsLts`: a `SNAPSHOT` pin, then `--version
LTS`;
- `releaseLineInstallsItsNewestPatchAndPinsTheLine`;
- `releaseLineRefusesAnAliasOutsideTheLine`.
The docs now say `--version LTS`/`X.Y` pins the selector, and only
`--version X.Y.Z` creates no pin.
2. **Prune failure**: pruning is now best effort. A build that can't be
removed prints a warning, and the install (and the `self-update` pin) still
succeeds. `WebsiteInstallTest.aNightlyThatCannotBeRemovedDoesNotFailTheInstall`
locks a directory in an old build.
All six new tests fail without the fixes. The branch is now rebased on main,
since #29427 was merged.
_Claude Code on behalf of davsclaus_
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
