blob: ff6a4f84cac1f242388db3bd540f925cbc28f04a [file] [log] [blame]
Ed Tanousd5cea142021-09-09 11:42:05 -07001#!/bin/bash -e
2#
3
Ed Tanous7f8954e2021-09-09 11:46:43 -07004script_dir="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
5obmc_dir=${script_dir}/../../
6
Patrick Williams12a27b22021-10-26 16:07:31 -05007cd "$obmc_dir"
8
Ed Tanous7f8954e2021-09-09 11:46:43 -07009# openbmc doesn't control what upstream poky, or any of the other layers do,
10# which do use patches as part of their upstreaming process.
11# meta-phosphor is also included such that patches that the community agrees to
12# hold onto will be allowed in that layer.
13
14patch_files_tmp=$(mktemp)
15allowed_patches_tmp=$(mktemp)
Patrick Williams693d3db2021-10-26 16:10:20 -050016trap 'rm $patch_files_tmp $allowed_patches_tmp' exit
Ed Tanous7f8954e2021-09-09 11:46:43 -070017
Patrick Williams12a27b22021-10-26 16:07:31 -050018git ls-files -- \
Ed Tanous7f8954e2021-09-09 11:46:43 -070019 '*.patch' \
20 ':!:poky/**' \
Brad Bishopbec4ebc2022-08-03 09:55:16 -040021 ':!:meta-arm/**' \
Ed Tanous7f8954e2021-09-09 11:46:43 -070022 ':!:meta-security/**' \
Ed Tanous7f8954e2021-09-09 11:46:43 -070023 ':!:meta-raspberrypi/**' \
24 ':!:meta-openembedded/**' \
25 ':!:meta-phosphor/**' \
Patrick Williams693d3db2021-10-26 16:10:20 -050026 | sort > "$patch_files_tmp"
Ed Tanous7f8954e2021-09-09 11:46:43 -070027
28
29# The following patches were present on master at the time this test was
30# written. Their presence in this list should not be acknowlegement that they
31# are now allowed, but ignoring them is required in the intermediate time
32# between when this test was created, and when the maintainers of these repos
33# clean them up.
34#
35# https://github.com/openbmc/docs/blob/master/meta-layer-guidelines.md
36echo "\
Andrew Jeffery7c928572023-08-14 13:00:06 +093037meta-aspeed/recipes-aspeed/python/socsec/0001-otptool-Define-value_start-in-rev_id-path.patch
Ed Tanous7f8954e2021-09-09 11:46:43 -070038meta-aspeed/recipes-bsp/u-boot/files/default-gcc.patch
Ed Tanous7f8954e2021-09-09 11:46:43 -070039meta-bytedance/meta-g220a/recipes-kernel/linux/linux-aspeed/0005-ARM-dts-aspeed-Enable-g220a-uart-route.patch
Logananth Sundararaj971257a2022-03-23 17:34:12 +053040meta-facebook/meta-yosemitev2/recipes-bsp/u-boot/u-boot-aspeed-sdk/0001-board-aspeed-Add-Mux-for-yosemitev2.patch
41meta-facebook/meta-yosemitev2/recipes-bsp/u-boot/u-boot-aspeed-sdk/0002-spl-host-console-handle.patch
Ed Tanous7f8954e2021-09-09 11:46:43 -070042meta-google/dynamic-layers/nuvoton-layer/recipes-bsp/images/npcm7xx-igps/0001-Set-FIU0_DRD_CFG-and-FIU_Clk_divider-for-gbmc-hoth.patch
43meta-google/recipes-extended/libconfig/files/0001-conf2struct-Use-the-right-perl.patch
44meta-google/recipes-extended/libconfig/files/0001-makefile-Add-missing-LDFLAGS.patch
45meta-google/recipes-phosphor/initrdscripts/obmc-phosphor-initfs/rwfs-clean-dev.patch
46meta-ingrasys/meta-zaius/recipes-bsp/u-boot/u-boot-aspeed/0001-board-aspeed-Add-reset_phy-for-Zaius.patch
47meta-nuvoton/recipes-bsp/images/npcm7xx-igps/0001-Adjust-paths-for-use-with-Bitbake.patch
48meta-yadro/meta-nicole/recipes-bsp/u-boot/files/0001-Add-system-reset-status-support.patch
49meta-yadro/meta-nicole/recipes-bsp/u-boot/files/0002-config-ast-common-set-fieldmode-to-true.patch
50meta-yadro/meta-nicole/recipes-bsp/u-boot/files/0003-aspeed-add-gpio-support.patch
51meta-yadro/meta-nicole/recipes-bsp/u-boot/files/0004-aspeed-add-bmc-position-support.patch
52meta-yadro/meta-nicole/recipes-kernel/linux/linux-aspeed/0001-Add-NCSI-channel-selector.patch
53meta-yadro/meta-nicole/recipes-phosphor/host/op-proc-control/0001-Stop-and-send-SRESET-for-one-thread-only.patch
54meta-yadro/recipes-phosphor/dbus/phosphor-dbus-interfaces/0001-Add-boot-initiator-mailbox-interface.patch
55meta-yadro/recipes-phosphor/ipmi/phosphor-ipmi-host/0001-Add-support-for-persistent-only-settings.patch
56meta-yadro/recipes-phosphor/ipmi/phosphor-ipmi-host/0002-Add-support-for-boot-initiator-mailbox.patch
57meta-yadro/recipes-phosphor/ipmi/phosphor-ipmi-host/0003-Fix-version-parsing-update-AUX-revision-info.patch
Patrick Williams693d3db2021-10-26 16:10:20 -050058" | sort > "$allowed_patches_tmp"
Ed Tanous7f8954e2021-09-09 11:46:43 -070059
Patrick Williams693d3db2021-10-26 16:10:20 -050060files_diff=$(comm -23 "$patch_files_tmp" "$allowed_patches_tmp")
Ed Tanous7f8954e2021-09-09 11:46:43 -070061
Ed Tanous4734c222021-09-20 11:11:34 -070062files_count=$(echo -n "$files_diff" | grep -c '^' || true)
Ed Tanous7f8954e2021-09-09 11:46:43 -070063if [[ $files_count -ne 0 ]]; then
64 echo "Patch files found not in allow list"
65 echo "$files_diff"
66 echo "Patches are not allowed on OpenBMC in these layers. Please upstream your changes and see \
67 https://github.com/openbmc/docs/blob/master/meta-layer-guidelines.md"
68 exit 1
69fi
70
Andrew Geisslerfc328172022-01-21 09:37:06 -060071# Now verify there are no kernel dts files being added in a patch
72# All dts files should be sent upstream and can be carried in the openbmc/linux
73# tree until they are accepted upstream
74dts_files_tmp=$(mktemp)
75allowed_dts_tmp=$(mktemp)
76trap 'rm $dts_files_tmp $allowed_dts_tmp' exit
77
78git ls-files -- \
79 '*.dts' \
80 ':!:poky/**' \
Brad Bishopbec4ebc2022-08-03 09:55:16 -040081 ':!:meta-arm/**' \
Andrew Geisslerfc328172022-01-21 09:37:06 -060082 ':!:meta-security/**' \
83 ':!:meta-raspberrypi/**' \
84 ':!:meta-openembedded/**' \
85 | sort > "$dts_files_tmp"
86
87# There is a single dts currently in the tree that we will exempt for now
88echo "\
89meta-hpe/meta-dl360poc/recipes-kernel/linux/linux-obmc/gxp.dts
90" | sort > "$allowed_dts_tmp"
91
92files_diff=$(comm -23 "$dts_files_tmp" "$allowed_dts_tmp")
93
94files_count=$(echo -n "$files_diff" | grep -c '^' || true)
95if [[ $files_count -ne 0 ]]; then
96echo "Dts files found not in allow list"
97echo "$files_diff"
98echo "Dts files are not allowed on OpenBMC in these layers. Please upstream your changes and see \
99 https://github.com/openbmc/docs/blob/master/kernel-development.md"
100exit 1
101fi
102
Patrick Williams12a27b22021-10-26 16:07:31 -0500103lint_exempt="\
Patrick Williams12a27b22021-10-26 16:07:31 -0500104meta-phosphor/recipes-support/nss-pam-ldapd/files/nslcd.init
Patrick Williams12a27b22021-10-26 16:07:31 -0500105"
106
Patrick Williams2ed00d82021-10-27 09:33:53 -0500107types=(json shell)
108# shellcheck disable=SC2034
109check_json="eslint --resolve-plugins-relative-to /usr/local/lib/node_modules"
Patrick Williams521d8f62021-10-27 09:30:41 -0500110# shellcheck disable=SC2034
111check_shell="shellcheck -x"
112
113for t in "${types[@]}"; do
114 check_cmd="check_${t}"
115 if ! which "${!check_cmd%% *}" > /dev/null 2>&1; then
116 eval "${check_cmd}=\"echo WARNING: Skipping $t due to missing command:\""
117 echo "${!check_cmd}"
118 fi
119done
120
Patrick Williams12a27b22021-10-26 16:07:31 -0500121non_bbfiles=$(git ls-files -- \
122 ':!:poky/**' \
Brad Bishopbec4ebc2022-08-03 09:55:16 -0400123 ':!:meta-arm/**' \
Patrick Williams12a27b22021-10-26 16:07:31 -0500124 ':!:meta-security/**' \
125 ':!:meta-raspberrypi/**' \
126 ':!:meta-openembedded/**' \
127 | grep -v -e "\.patch$" -e "\.bb$" -e "\.bbappend$")
128
129for f in $non_bbfiles; do
Patrick Williams521d8f62021-10-27 09:30:41 -0500130 unset file_type
131 file_info=$(file "$f")
132 case $file_info in
Patrick Williams12a27b22021-10-26 16:07:31 -0500133 *shell\ script*)
Patrick Williams521d8f62021-10-27 09:30:41 -0500134 file_type="shell"
Patrick Williams12a27b22021-10-26 16:07:31 -0500135 ;;
136
Patrick Williams2ed00d82021-10-27 09:33:53 -0500137 *JSON\ data*)
138 file_type="json"
139 ;;
140
Patrick Williams12a27b22021-10-26 16:07:31 -0500141 *)
Patrick Williams521d8f62021-10-27 09:30:41 -0500142 case $f in
143 *.sh)
144 file_type="shell"
145 ;;
146
Patrick Williams2ed00d82021-10-27 09:33:53 -0500147 *.json)
148 file_type="json"
149 ;;
Patrick Williams521d8f62021-10-27 09:30:41 -0500150 esac
Patrick Williams12a27b22021-10-26 16:07:31 -0500151 esac
Patrick Williams521d8f62021-10-27 09:30:41 -0500152
153 if [ -n "$file_type" ]; then
154 check_cmd="check_${file_type}"
155 if ! eval "${!check_cmd} $f"; then
156 if [[ $lint_exempt == *$f* ]]; then
157 echo "EXEMPT: $f"
158 else
159 echo "FAILED: $f"
160 false
161 fi
162 fi
163 fi
164
Patrick Williams12a27b22021-10-26 16:07:31 -0500165done
166
Ed Tanousd5cea142021-09-09 11:42:05 -0700167echo "Repo test passed"