GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Fix nrf sdk download links by dariusarnold · Pull Request #2270 · InfiniTimeOrg/InfiniTime

Conversation

@dariusarnold

@dariusarnold dariusarnold commented Mar 13, 2025

Copy link
Copy Markdown
Contributor

It appears that NRF has moved the download to a new location. The old URL is now a 404.
Update the build script and README to use the same URL the download website currently provides.

Fixes #2267

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
dariusarnold added 2 commits March 13, 2025 21:35
@dariusarnold
Fix docker build error due to new NRF SDK download location
ce43287
It seems like NRF has moved the download to a new location- Update the build script to fetch from the same URL the download website currently provides.
@dariusarnold
Fix another broken download link to the SDK
0a40178
@github-actions

github-actions Bot commented Mar 13, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Build size and comparison to main:

Section Size Difference
text 373088B 0B
data 948B 0B
bss 22536B 0B

Run in InfiniEmu

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@mark9064 mark9064 added documentation Improvements or additions to documentation maintenance Background work labels Mar 13, 2025
mark9064
mark9064 approved these changes Mar 13, 2025
View reviewed changes
@mark9064 mark9064 added this to the 1.16.0 milestone May 16, 2025
@JF002

JF002 commented May 21, 2025

Copy link
Copy Markdown
Collaborator

@dariusarnold Do you know if those URLs are still hosted by Nordic and if they are "official"? The domain name does not seem to be directly linked to Nordic Semiconductor.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@mark9064

mark9064 commented May 21, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

It's official, you can find them on the nordic website

See here https://www.nordicsemi.com/Products/Development-software/nRF5-SDK/Download#infotabs

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@JF002

JF002 commented May 21, 2025

Copy link
Copy Markdown
Collaborator

Oh yes, right! I previously clicked on the big "Download files (.zip)" button and did not notice that I could simply click on the name of the package!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

JF002
JF002 approved these changes May 21, 2025
View reviewed changes
@mark9064
mark9064 merged commit 248a6ae into InfiniTimeOrg:main May 21, 2025
@mark9064 mark9064 mentioned this pull request May 23, 2025
fix nrf sdk path #2296
Closed
NeroBurner added a commit that referenced this pull request May 24, 2025
@NeroBurner
docker: fix NRF_SDK download and subsequent build.sh
59f5b12
The upstream NRF-SDK download url and zip archive filename changed,
which was fixed with #2270

But the archive contents stayed the same, with the "old" folder name.

After #2270 we have basically the same docker-container as before the PR,
but the `GetNrfSdk` function of the `build.sh` script is called again during
firmware build time as the expected foldername for the SDK isn't the same as
the zip filename:

```sh
[ ! -d "$TOOLS_DIR/$NRF_SDK_VER" ] && GetNrfSdk
```

Then during the build the `buils.sh` script tries to execute `GetNrfSdk` again,
which fails as the files already exist resulting in the following error:

```
replace /opt/nRF5_SDK_15.3.0_59ac345/components/802_15_4/api/HAL/hal_atomic.h? [y]es, [n]o, [A]ll, [N]one, [r]ename:  NULL
```

Fix this by reverting the `NRF_SDK_VER` to the folder name in the zip
archive and by some character replacement generate the download URL from
the above (the download is in lower-case without the `_` and `.`
characters).

Furthermore add safeguards to check after the `GetNrfSdk` call if the
expected folder is really created. Then we have an error early during
container image creation if the contents of the zip-archive are
unexpected.
@NeroBurner NeroBurner mentioned this pull request May 24, 2025
docker: fix NRF_SDK download and subsequent build.sh #2299
Merged
NeroBurner added a commit that referenced this pull request May 27, 2025
@NeroBurner
docker: fix NRF_SDK download and subsequent build.sh (#2299)
6f2a661
The upstream NRF-SDK download url and zip archive filename changed,
which was fixed with #2270

But the archive contents stayed the same, with the "old" folder name.

After #2270 we have basically the same docker-container as before the PR,
but the `GetNrfSdk` function of the `build.sh` script is called again during
firmware build time as the expected foldername for the SDK isn't the same as
the zip filename:

```sh
[ ! -d "$TOOLS_DIR/$NRF_SDK_VER" ] && GetNrfSdk
```

Then during the build the `buils.sh` script tries to execute `GetNrfSdk` again,
which fails as the files already exist resulting in the following error:

```
replace /opt/nRF5_SDK_15.3.0_59ac345/components/802_15_4/api/HAL/hal_atomic.h? [y]es, [n]o, [A]ll, [N]one, [r]ename:  NULL
```

Fix this by reverting the `NRF_SDK_VER` to the folder name in the zip
archive and by some character replacement generate the download URL from
the above (the download is in lower-case without the `_` and `.`
characters).

Furthermore add safeguards to check after the `GetNrfSdk` call if the
expected folder is really created. Then we have an error early during
container image creation if the contents of the zip-archive are
unexpected.
@mark9064 mark9064 mentioned this pull request Jun 8, 2025
CMake Presets json file #2308
Closed
JustScott pushed a commit to JustScott/InfiniTime that referenced this pull request Jul 29, 2025
@dariusarnold @JustScott
Fix nRF SDK download links (InfiniTimeOrg#2270)
62dc63e
nRF has moved the download to a new location- Update the build script to fetch from the same URL the download website currently provides.
JustScott pushed a commit to JustScott/InfiniTime that referenced this pull request Jul 29, 2025
@NeroBurner @JustScott
docker: fix NRF_SDK download and subsequent build.sh (InfiniTimeOrg#2299
8d64a22
)

The upstream NRF-SDK download url and zip archive filename changed,
which was fixed with InfiniTimeOrg#2270

But the archive contents stayed the same, with the "old" folder name.

After InfiniTimeOrg#2270 we have basically the same docker-container as before the PR,
but the `GetNrfSdk` function of the `build.sh` script is called again during
firmware build time as the expected foldername for the SDK isn't the same as
the zip filename:

```sh
[ ! -d "$TOOLS_DIR/$NRF_SDK_VER" ] && GetNrfSdk
```

Then during the build the `buils.sh` script tries to execute `GetNrfSdk` again,
which fails as the files already exist resulting in the following error:

```
replace /opt/nRF5_SDK_15.3.0_59ac345/components/802_15_4/api/HAL/hal_atomic.h? [y]es, [n]o, [A]ll, [N]one, [r]ename:  NULL
```

Fix this by reverting the `NRF_SDK_VER` to the folder name in the zip
archive and by some character replacement generate the download URL from
the above (the download is in lower-case without the `_` and `.`
characters).

Furthermore add safeguards to check after the `GetNrfSdk` call if the
expected folder is really created. Then we have an error early during
container image creation if the contents of the zip-archive are
unexpected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JF002 JF002 JF002 approved these changes
@mark9064 mark9064 mark9064 approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation maintenance Background work

Projects

None yet

Milestone

1.16.0

Development

Successfully merging this pull request may close these issues.

Docker build broken because NRF SDK changed download location

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@dariusarnold @JF002 @mark9064
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.