Skip to content
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

doc: Fix nrfutil trace version for v2.9.0 nRF54H20 RC1 #19790

Conversation

RICCIARDI-Adrien
Copy link

The documentation tells to install nrfutil trace v3.10.0, but this version does not seem to exist :

:~/ncs$ nrfutil install trace=3.10.0 --force
nrfutil-trace already installed
Uninstalled nrfutil-trace
Error: Failed to install nrfutil-trace

Caused by:
    0: Package index file error
    1: Unable to find version '3.10.0' in the package index file

However, the latest version seems to be v3.1.0 :

:~/ncs$ nrfutil trace --version
nrfutil-trace 3.1.0 (2da0997 2024-12-11)
commit-hash: 2da0997efbe37ef990d0a0384eb49d13264bab45
commit-date: 2024-12-11
host: x86_64-unknown-linux-gnu
build-timestamp: 2024-12-12T08:08:37.332171808Z
classification: nrf-external

The documentation tells to install nrfutil trace v3.10.0, but this version
does not seem to exist :
```
:~/ncs$ nrfutil install trace=3.10.0 --force
nrfutil-trace already installed
Uninstalled nrfutil-trace
Error: Failed to install nrfutil-trace

Caused by:
    0: Package index file error
    1: Unable to find version '3.10.0' in the package index file
```

However, the latest version seems to be v3.1.0 :
```
:~/ncs$ nrfutil trace --version
nrfutil-trace 3.1.0 (2da0997 2024-12-11)
commit-hash: 2da0997efbe37ef990d0a0384eb49d13264bab45
commit-date: 2024-12-11
host: x86_64-unknown-linux-gnu
build-timestamp: 2024-12-12T08:08:37.332171808Z
classification: nrf-external
```

Signed-off-by: Adrien Ricciardi <[email protected]>
@RICCIARDI-Adrien RICCIARDI-Adrien requested review from a team and FrancescoSer as code owners January 8, 2025 10:44
@CLAassistant
Copy link

CLAassistant commented Jan 8, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Jan 8, 2025
@NordicBuilder
Copy link
Contributor

Thank you for your contribution!
It seems you are not a member of the nrfconnect GitHub organization. External contributions are handled as follows:
Large contributions, affecting multiple subsystems for example, may be rejected if they are complex, may introduce regressions due to lack of test coverage, or if they are not consistent with the architecture of nRF Connect SDK.
PRs will be run in our continuous integration (CI) test system.
If CI passes, PRs will be tagged for review and merged on successful completion of review. You may be asked to make some modifications to your contribution during review.
If CI fails, PRs may be rejected or may be tagged for review and rework.
PRs that become outdated due to other changes in the repository may be rejected or rework requested.
External contributions will be prioritized for review based on the relevance to current development efforts in nRF Connect SDK. Bug fix PRs will be prioritized.
You may raise issues or ask for help from our Technical Support team by visiting https://devzone.nordicsemi.com/.

Note: This comment is automatically posted and updated by the Contribs GitHub Action.

@NordicBuilder NordicBuilder added the external External contribution label Jan 8, 2025
@FrancescoSer
Copy link
Contributor

Hi @RICCIARDI-Adrien, thank you for the contribution. This is actually being updated in PR #19917

I'll close this one. However, please feel free to leave additional comments in #19917

@RICCIARDI-Adrien RICCIARDI-Adrien deleted the 2.9.0-nrf54h20-rc1_update_nrfutil_trace_version branch January 15, 2025 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. external External contribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants