Skip to content

Add optional SDK path arg to the build.sh#127

Open
janeoa wants to merge 3 commits into
servo:mainfrom
janeoa:support-custom-ohos-sdk-path
Open

Add optional SDK path arg to the build.sh#127
janeoa wants to merge 3 commits into
servo:mainfrom
janeoa:support-custom-ohos-sdk-path

Conversation

@janeoa
Copy link
Copy Markdown
Contributor

@janeoa janeoa commented May 4, 2026

When building the local images using podman, it failes to download the SDK due to 403 error (does not happen on docker for some reason).

So I added an optional arg to the build.sh, so that the build script could use locally pre-downloaded SDK of any version for image building.

It also supports -h/--help just in case

Signed-off-by: Jane <5373400+janeoa@users.noreply.github.com>
@janeoa janeoa requested a review from jschwe as a code owner May 4, 2026 12:12
@jschwe
Copy link
Copy Markdown
Member

jschwe commented May 4, 2026

When building the local images using podman, it failes to download the SDK due to 403 error (does not happen on docker for some reason).

Is this behind a proxy? Also in production, we actually commented this and COPY the image instead, since the mirror doesn't have a recent enough version of the commandline tools. I guess having a flag could be worth considering.

Copy link
Copy Markdown
Contributor

@Narfinger Narfinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you just add a print at the start, something like: "Using sdk version $VERSION from web.

janeoa added 2 commits May 4, 2026 17:29
Signed-off-by: Jane <5373400+janeoa@users.noreply.github.com>
Signed-off-by: Jane <5373400+janeoa@users.noreply.github.com>
@janeoa janeoa changed the title Add optinal SDK path arg to the build.sh Add optional SDK path arg to the build.sh May 13, 2026
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.

3 participants