Skip to content

Adds BSP layer download mirrors #1703

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.

Sign up for GitHub

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

Open
quaresmajose wants to merge 4 commits into
base: master
Choose a base branch
from
+47 −0
Open

Adds BSP layer download mirrors #1703

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
7afc71d
conf/layer: Add QCOM_RELEASE
quaresmajose Mar 6, 2026
2af28cb
qcom-mirrors: add download mirror bbclass
quaresmajose Mar 6, 2026
4a10f13
conf/layer: enable qcom-mirrors
quaresmajose Mar 6, 2026
799eab5
ci: add mirror-download kas fragments
quaresmajose Mar 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 25 additions & 0 deletions classes/qcom-mirrors.bbclass
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
# Layer download mirror to minimize reproducibility issues
#
# Copyright (c) 2026 Qualcomm Innovation Center, Inc.
#
# SPDX-License-Identifier: MIT
#

# Add qcom mirror so we can fallback to it
MIRRORS += "${QCOM_MIRRORS}"
Copy link
Contributor

@ricardosalveti ricardosalveti Mar 6, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

Can you move this assignment after QCOM_MIRRORS is defined? Easier to parse when reading this.


QCOM_MIRRORS_URI ?= "https://artifacts.codelinaro.org/aritfactory/qli-ci/downloads/${QCOM_RELEASE}"
Copy link
Contributor

@lumag lumag Mar 6, 2026

Choose a reason for hiding this comment

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

Why do we need a release-specific dir?


QCOM_MIRRORS ?= " \
svn://.*/.* ${QCOM_MIRRORS_URI}/ \
git://.*/.* ${QCOM_MIRRORS_URI}/ \
gitsm://.*/.* ${QCOM_MIRRORS_URI}/ \
hg://.*/.* ${QCOM_MIRRORS_URI}/ \
p4://.*/.* ${QCOM_MIRRORS_URI}/ \
https?://.*/.* ${QCOM_MIRRORS_URI}/ \
ftp://.*/.* ${QCOM_MIRRORS_URI}/ \
npm://.*/?.* ${QCOM_MIRRORS_URI}/ \
s3://.*/.* ${QCOM_MIRRORS_URI}/ \
crate://.*/.* ${QCOM_MIRRORS_URI}/ \
gs://.*/.* ${QCOM_MIRRORS_URI}/ \
"
3 changes: 3 additions & 0 deletions conf/layer.conf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,9 @@ BBFILES_DYNAMIC += " \
selinux:${LAYERDIR}/dynamic-layers/selinux/*/*/*.bbappend \
"

# Release
QCOM_RELEASE = "main"
Copy link
Contributor

@ricardosalveti ricardosalveti Mar 6, 2026

Choose a reason for hiding this comment

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

QLI_RELEASE or QLI_BASELINE would work better from my perspective.

QCOM is too generic to be used here.


# Set default provider for virtual-diag-router to avoid warnings
# when both diag and diag-router are available
PREFERRED_RPROVIDER_virtual-diag-router ?= "diag"
Expand Down