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

treewide: silent download, unpack and others #1630

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Cynerd
Copy link
Contributor

@Cynerd Cynerd commented Mar 6, 2023

This introduces changes from NuttX commit
03b164f59ce40a3f5677b0588af2aee8d9697bf6.

The only left out is 'Register:' as that provides pretty handly list of applications in the firmware.

The download is unified to handle it always the same way.

Follow up on #1588 (comment).

Make.defs Outdated Show resolved Hide resolved
This introduces changes from NuttX commits:
 03b164f59ce40a3f5677b0588af2aee8d9697bf6
 7fef3e48e6092a59e58b541ef62bdd5f2f600b61

The only left out is 'Register:' as that provides pretty handly list of
applications in the firmware.

The download is unified to handle it always the same way.
@Cynerd
Copy link
Contributor Author

Cynerd commented Mar 8, 2023

Please, wait with merge. There seems to be issues with downloading wamr. I have to look into it.

@acassis
Copy link
Contributor

acassis commented Mar 17, 2023

@Cynerd please let me know: does this PR hides the Downloading and Patching lines? I think it is important to users know that the files are downloaded during the building phase.

@slorquet
Copy link
Contributor

I agree. it is critical to know that a build cannot be completed while staying offline.

@hartmannathan
Copy link
Contributor

I agree. it is critical to know that a build cannot be completed while staying offline.

@Cynerd please let me know: does this PR hides the Downloading and Patching lines? I think it is important to users know that the files are downloaded during the building phase.

I agree also. It is definitely critical to know that things are being downloaded.

Wherever we are doing a download, we should also allow a custom URL in Kconfig, so that developers who want to locally host the dependencies will be able to do so.

@hartmannathan
Copy link
Contributor

Please, wait with merge. There seems to be issues with downloading wamr. I have to look into it.

You can mark the PR as Draft until ready for review. That will prevent someone merging by mistake...

@acassis
Copy link
Contributor

acassis commented May 31, 2024

Hi @Cynerd maybe a better approach is just show: Downloading: filename-x.y.z and when it finishes download: done.

What do you think?

@xiaoxiang781216
Copy link
Contributor

@Cynerd could you fix the conflict?

@Cynerd
Copy link
Contributor Author

Cynerd commented Jun 9, 2024

I will look at this next week. This is pretty old and was created only to keep it consistent in my eyes with the rest of the changes introduced at that time. I think that it still makes sense in terms of echoing that the download is happening but not in the way of reporting its progress.

In the meantime, we also modified the build process of our own projects in a way that nothing is being downloaded, and thus, I had less incentive to deal with this.

@acassis
Copy link
Contributor

acassis commented Jun 9, 2024

I will look at this next week. This is pretty old and was created only to keep it consistent in my eyes with the rest of the changes introduced at that time. I think that it still makes sense in terms of echoing that the download is happening but not in the way of reporting its progress.

In the meantime, we also modified the build process of our own projects in a way that nothing is being downloaded, and thus, I had less incentive to deal with this.

Agree! All user friendly improvements are welcome!

@cederom
Copy link

cederom commented Nov 2, 2024

ping :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants