Skip to content

Fix build toolchain#145

Closed
tobrie wants to merge 1 commit intoInfiniTimeOrg:developfrom
tobrie:pr-build
Closed

Fix build toolchain#145
tobrie wants to merge 1 commit intoInfiniTimeOrg:developfrom
tobrie:pr-build

Conversation

@tobrie
Copy link

@tobrie tobrie commented Nov 27, 2020

with the help of @ZephyrLabs

@JF002
Copy link
Collaborator

JF002 commented Dec 1, 2020

What's the issue with the current github actions build?

I don't know/use Github Actions, so I'm not sure I know what I'm talking about, but I think the file was written to use the cache functionality of Github action (to avoid downloading the toolchain for each build), and I'm not sure your modification still uses it.

Could you give more info about the modification?

@ZephyrLabs
Copy link
Contributor

@JF002 the current github actions script used a inbuilt action to install arm-none-eabi-gcc compiler, that action is now deprecated due to an update in github actions itself, which does not permit the function from using the "add-path" (which is the actual deprecated feature)

this PR is for replacing it with standard linux commands that can be run instead as a replacement for that function :)

@JF002
Copy link
Collaborator

JF002 commented Dec 5, 2020

Thanks for the info!
Could you confirm that the toolchain is still cached and won't be downloaded/installed for each build?

@JF002
Copy link
Collaborator

JF002 commented Jan 13, 2021

A response to my last question would be appreciated :)

@JF002 JF002 added the needs more info More information is needed label Jan 13, 2021
@pfeerick
Copy link
Contributor

pfeerick commented Jan 14, 2021

I haven't used caching with github actions, but judging by the changes and how the nrf52 SDK is installed, I suspect not - I think this PR will still check if it is cached, and then proceed to download it regardless of if it is in the cache or not, rather than using a similar process to the nrf52 SDK install stage.

@JF002
Copy link
Collaborator

JF002 commented Jan 14, 2021

Wouldn't it be fair to not download the sdk if it is already in the cache, if possible ?

@pfeerick
Copy link
Contributor

Since there is the facility to do so, yes it would... it's a pity @tobrie didn't use the NRF52 installation stage as a reference, and just change the download commands... copy/paste/tweak/done.

@pfeerick
Copy link
Contributor

I believe this is no longer relevant due to #294.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs more info More information is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants