Repository navigation
Conversation
west init tracked the mcuxsdk-manifests default branch, so nxp-mcuxpresso-test followed an unreleased SDK. That branch pins fw_bin to NXP/wifi_nb_fw@2553ab1d, a commit the remote no longer serves, so west update_board aborts with "upload-pack: not our ref" and the job fails on main and on every pull request. Pin the manifest to v26.09.00, the most recent tag whose fw_bin revision still resolves, overridable through MCUXSDK_MANIFEST_REV. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This PR needs a
|
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
nxp-mcuxpresso-testis red onmainand on every pull request, for a reason that has nothing to do with the code under test.test_build_from_scratch.shranwest initagainstmcuxsdk-manifestswithout a revision, so the job followed that repository's default branch — a moving target tracking the unreleased 26.12.00 SDK. That branch pins thefw_binproject toNXP/wifi_nb_fw@2553ab1d2097f0adb6e0846f054142c86045db3f, a commit the remote no longer serves. Because the project is declared withclone-depth: 1, west fetches that exact SHA and has no fallback:This pins the manifest to
v26.09.00, the most recent tag whosefw_binrevision still resolves (ae66e11f). Thev26.12.00-pvw1tag carries the same dangling pin as the default branch, so it is not an option.MCUXSDK_MANIFEST_REVlets the revision be overridden, matching howARM_TOOLCHAIN_URLis already handled in the same script.Beyond unbreaking the job, pinning removes a standing dependency on an upstream branch that can change under us at any time.
What was checked
NXP/wifi_nb_fw(GitHub returns422 No commit found for SHA).v26.09.00, every SDK path this example'sCMakeLists.txtconsumes exists:eiq_examples/executorch_cifarnet/{main.cpp,RegisterKernels.cpp},eiq_examples/common/timer.c,_boards/mimxrt700evk/eiq_examples/executorch_cifarnet/cm33_core0/{hardware_init.c,pin_mux.c,gcc/MIMXRT798Sxxxx_cm33_core0_flash.ld}and.../npu.v26.09.00and the default branch,mcuxsdk-middleware-executorchadvances by exactly one commit, a documentation URL fix — so the pin gives up nothing on the ExecuTorch side.What was not checked
The full ARM build was not reproduced locally, so that the example still links against the September SDK rests on the file-level checks above rather than on an actual build. CI on this PR is the real verification.