Only the pipeline in merge request context is relevant here.
"ci.skip" only affect branch pipelines and not merge request pipelines.
See https://docs.gitlab.com/ee/gitlab-basics/add-file.html#push-options-for-gitlab-cicd



Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

These tools come from:
https://gitlab.apertis.org/infrastructure/apertis-docker-images/-/tree/apertis/v2025dev3



Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Sometime the main server of snapshot.debian.org is not sync
which block us from downloading new packages from Debian.
Thus to avoid depending on a single server add a mirror
as fallback.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

This tool, inspired by https://tracker.debian.org/pkg/routine-update

,
performs common cleanups in Apertis packages like:
   - remove deprecated debian/apertis/gitlab-ci.yml
   - add debian/apertis/lintian for packages maintained by Apertis

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

pathlib.Path.__enter__() is deprecated and scheduled for removal
in Python 3.13. Some warnings are generated with Python 3.11:

import-debian-package:281: DeprecationWarning: pathlib.Path.__enter__() is deprecated and scheduled for removal in Python 3.13; Path objects as a context manager is a no-op
  with Path(".git/info/attributes") as f:
import-debian-package:225: DeprecationWarning: pathlib.Path.__enter__() is deprecated and scheduled for removal in Python 3.13; Path objects as a context manager is a no-op
  with Path('debian/apertis/component') as f:
import-debian-package:169: DeprecationWarning: pathlib.Path.__enter__() is deprecated and scheduled for removal in Python 3.13; Path objects as a context manager is a no-op
  with Path("debian/changelog") as f:

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Updates from *-proposed-updates repositories are not pulled anymore
by default because it can contain undesirable updates for Apertis.
However, we still want updates from *-updates repositories instead.

"proposed-updates" contains updates not officialy part of Debian
contrary to "updates".

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

The Apertis pipeline imports backport and/or proposed-updates versions
by default, this is not needed and leads to miss some security updates.
Because, importing versions from backports and proposed-updates is not
part of our standard workflow, this should only be optional to avoid
confusing the pipeline.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

`gbp import-dsc` puts all the new changelog entries in the commit
message, generating big walls of text when, for instance, importing a
new package for the first time.
GitLab then puts the whole log message in the `CI_COMMIT_MESSAGE` env
var, which makes the runner to fail during the executor preparation
section with the error:

    exec /usr/bin/dumb-init: argument list too long

See: https://gitlab.com/gitlab-org/gitlab/-/issues/392406



To avoid that, trim the message to only keep the first line.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

`flake8` said:

    tools/ade:1105:13: E741 ambiguous variable name 'l'

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

`flake` lamented:

    tools/ade:511:21: E722 do not use bare 'except'

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

`flake8` was complaining about:

    tools/ade:473:17: F841 local variable 'e' is assigned to but never used

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

`flake8` did report:

   tools/ade:288:26: W605 invalid escape sequence '\s'

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

`flake8` did report a typo:

    tools/ade:217:23: F821 undefined name 'user'

As far as I can tell nothing ever tries to stringify the `Device`
object, so that's probably dead code and that's why we never noticed.

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

Signed-off-by: Emanuele Aina <emanuele.aina@collabora.com>

Walter Lozano authored 1 year ago and Emanuele Aina committed 1 year ago

Since v2024dev2 the file pkg-config.multiarch is not longer shipped, so
in order to detect the sysroot architecture check the arch specific
libraries folder present.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Walter Lozano authored 1 year ago and Emanuele Aina committed 1 year ago

Tool ade needs to find out release and distribution information for SDK and
sysroots. This was previously done reading /etc/image_version but this
custom file was dropped in favor of /etc/os-release.

In order to support both the old and the new approach fallback to
/etc/os-release if /etc/image_version is not found.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Walter Lozano authored 1 year ago and Emanuele Aina committed 1 year ago

Since same code is used in several places move it to new helper functions
to detect sysroots properties.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

If no mount for / (root) at given mount point can be established before mounting a sub partition, a virtual mountpoint
with a small tmpfs (ramdisk) is set up for / (root), so also non hierarchical images without a defined root partition
can be set up. This is usable, if partitions of an image are not mountable at one hierarchical mount tree.
Unmounting the image automatically removes the ram disk.

Signed-off-by: Thomas Mittelstaedt <thomas.mittelstaedt@de.bosch.com>

Partprobe rereads disk structures to the kernel. After returning to caller the process is sometimes not completed.
So information about labels, part labels are sometimes are not available. A small sleep solves this issue so far, but
a better synchronization method would be more appopriate but too complex for such a small tool. And atm a reliable
algorithm is not clear.

Signed-off-by: Thomas Mittelstaedt <thomas.mittelstaedt@de.bosch.com>

Signed-off-by: Thomas Mittelstaedt <thomas.mittelstaedt@de.bosch.com>

Signed-off-by: Thomas Mittelstaedt <thomas.mittelstaedt@de.bosch.com>

Signed-off-by: Thomas Mittelstaedt <thomas.mittelstaedt@de.bosch.com>

When the soname changes, the name of the binary package changes
accordingly. In this case, ABI breakage is expected thus there is
no need to run abi-compliance-checker. apertis-abi-compare detects
this case and report the soname change without failing.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

This flag requires a string explaining the reason of importing
a package. This reason will be written in debian/changelog and
will be useful for a future reconsideration of the value of this
package in Apertis.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

When importing Debian packages it is possible to find cases where packages
already provided an upstream Gitlab CI configuration. In such cases,
when packages are imported Gitlab will try to run CI which has no sense.

For properly handle this scenario, create new repositories with CI
disabled, and let gitlab rulez to update the settings once the package
is fully imported.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Signed-off-by: mtt2hi <thomas.mittelstaedt@de.bosch.com>

Mountimage can be used to mount disks and system images at a given mount point.
One purpose is usage for devroot.

Signed-off-by: mtt2hi <thomas.mittelstaedt@de.bosch.com>

For consistency across all Apertis packages.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Due to a bug in abi-compliance-checker, std headers are occasionally
used for analyzing leading to wrongly detect an ABI breakage.
We globally exclude them to avoid this noise until it is fixed in ACC.

See: https://github.com/lvc/abi-compliance-checker/issues/85



Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

to be consistent with other arguments passed to run_abi_checker()

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

This summary is useful in case of multiple libraries to
easily and quickly identify which ones have issues.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

The call of the abi-compliance-checker might fail due to
errors. Possible reasons are the failed compilation of the
header files. To ease the analysis of the job logs the
additional printout might help.
Correct 'apt update' command.

Signed-off-by: Tino Lippold <Tino.Lippold@de.bosch.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>