On GitHub, this check is tagged as "Required."
However, if a PR only affects the following folders:.github, docs,
scripts, or cmake, the check is not executed (to avoid false positives),
resulting in the status "execution pending" + "required" on GitHub.
This commit forces the execution of the check but bypasses the
compliance.xml test when the PR only affects the listed folders.
If at least one update affects other folders, the compliance check
runs as usual.
In addition, the cmake folder is removed from the list, as there is
no reason to skip compliance tests on it.
Signed-off-by: Arnaud Pouliquen arnaud.pouliquen@foss.st.com
- bump to Zephyr 4.3
- use west packages pip --install
- use "west sdk install" command to install the expected sdk version.
- only install arm-zephyr-eabi compiler as build tested only on
arm targets
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
The name of the board has been updated since the Zephyr commit:
d8cfa6fb2916 ("boards: nxp: convert lpcxpresso54114 to hwmv2")
Fix the build command according to the new name.
The update is compatible with the Zephyr v3.7.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Allow Zephyr testing to either use locked known good values
or the very latest versions.
This brings open-amp to parity with libmetal. Open-amp did not have the
SDK URL discovery code so that was added here as part of this.
The known good versions are best for PR checking as if the build fails
it is almost always the PR itself that broke it.
The latest version is good for periodically checking compatibility with
the very latest Zephyr changes.
For now we run both on pushes and PRs.
We also run main against the latest zephyr check weekly as a look ahead.
Signed-off-by: Bill Mills <bill.mills@linaro.org>
Reduce the console output for the Zephyr sdk install.
This now matches the intent of libmetal but uses the documented wget
option.
Signed-off-by: Bill Mills <bill.mills@linaro.org>
To have a coherent directory hierarchy, move the open-amp git from root
to the "./open-amp" directory.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Update CI to work with 24.04.
* Lock to 24.04 so we control when to change
* Update locale and timezone to work for 24.04 or 22.04
* 24.04 creates these files if not already there but 22.04 does not
* Keep existing work around for 22.04 but make it work if already there
* use venv
* Breate a virtual environment isolated from the packages in the base
environment
* Inspired from zephyr build environment
* Use pip3 everywhere removing global and --user installation
* Add section to README about desktop testing
* Also fix usage formatting in README
Co-developed-by: Bill Mills <bill.mills@linaro.org>
Signed-off-by: Bill Mills <bill.mills@linaro.org>
Co-developed-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
By default the action closes the PR or issue 7 days after adding the
stale label.
The expected behavior is that the pull requests and issues will never be
closed automatically by the bot.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Update zephyr SDK to 0.16.1 to fix CI build issue:
arm-zephyr-eabi-gcc: fatal error: cannot read spec file
'picolibc.specs': No such file or directory
compilation terminated.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
The CI does not consider "check" reported by checkpatch.
This commit fixes it by:
- rebasing check_compliance.py compliance.yml to integrate Zephyr updates,
- add detection of the "check" in the report.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Add CMAKE_C_FLAGS warnings to highlight potential issue:
"-Wall -Wextra -Wshadow -Wunused-but-set-variable"
And force warning as error with "-Werror"
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
"Sysbuild is a higher-level build system that can be used to combine
multiple other build systems together"
Add the option by default for all platform as it is compatible with
a single image.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Enable the use of codespell tool in checkpatch.pl
For this the checkpatch.pl as been partially rebased on the Linux one.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
- Rebase check_compliance.py and compliance.yml from zephyr project
- Fix checkpatch subprocess command.
- rebase
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
CMake 3.20.0 or higher is required by Zephyr.
Use pip3 as recommended by zephyr project to get the last version.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Update zephyr SDK to 0.13 to fix CI compatibility issue.
The name of the steup file has also be updated.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
When only the libmetal/lib is copied, some files such as
VERSION or CmakeList.txt are not copied. This can lead to
a test error or false positives.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Add the new VERSION file support. Without the copy of the file the
cmake fails as no VERSION file is found in
zephyrproject/modules/lib/open-amp/open-amp/
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Separate the workflow as done in the libmetal library.
One for the patch compliance cheks, another one
for the build tests.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@st.com>
This patch add build tests for Linux, Arm platform and Zephyr OS.
Notice that for Zephyr, openamp and libmetal have been forked.
As consequence the associated openamp and libmetal
modules are updated and some Zephyr samples are compiled directly in
the zephyr environment.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@st.com>
The webispy/checkpatch-action does not seems to work properly.
Even if it detects coding style issue, the return state is successful.
This patch replaces the action by the compliance python script obtained
from the Zephyr project.
This script also runs gitlint to check commit messages.
Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@st.com>
Arnaud Pouliquen
Bill Mills
Ed Mooring
Arnaud Pouliquen