blob: 61f5f94205b215d18254024d98af6b415dc9ae84 [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 "\
Ed Tanous7f8954e2021-09-09 11:46:43 -070037meta-aspeed/recipes-bsp/u-boot/files/default-gcc.patch
Ed Tanous7f8954e2021-09-09 11:46:43 -070038meta-bytedance/meta-g220a/recipes-kernel/linux/linux-aspeed/0005-ARM-dts-aspeed-Enable-g220a-uart-route.patch
Logananth Sundararaj971257a2022-03-23 17:34:12 +053039meta-facebook/meta-yosemitev2/recipes-bsp/u-boot/u-boot-aspeed-sdk/0001-board-aspeed-Add-Mux-for-yosemitev2.patch
40meta-facebook/meta-yosemitev2/recipes-bsp/u-boot/u-boot-aspeed-sdk/0002-spl-host-console-handle.patch
Ed Tanous7f8954e2021-09-09 11:46:43 -070041meta-google/dynamic-layers/nuvoton-layer/recipes-bsp/images/npcm7xx-igps/0001-Set-FIU0_DRD_CFG-and-FIU_Clk_divider-for-gbmc-hoth.patch
42meta-google/recipes-extended/libconfig/files/0001-conf2struct-Use-the-right-perl.patch
43meta-google/recipes-extended/libconfig/files/0001-makefile-Add-missing-LDFLAGS.patch
44meta-google/recipes-phosphor/initrdscripts/obmc-phosphor-initfs/rwfs-clean-dev.patch
45meta-ingrasys/meta-zaius/recipes-bsp/u-boot/u-boot-aspeed/0001-board-aspeed-Add-reset_phy-for-Zaius.patch
46meta-nuvoton/recipes-bsp/images/npcm7xx-igps/0001-Adjust-paths-for-use-with-Bitbake.patch
47meta-yadro/meta-nicole/recipes-bsp/u-boot/files/0001-Add-system-reset-status-support.patch
48meta-yadro/meta-nicole/recipes-bsp/u-boot/files/0002-config-ast-common-set-fieldmode-to-true.patch
49meta-yadro/meta-nicole/recipes-bsp/u-boot/files/0003-aspeed-add-gpio-support.patch
50meta-yadro/meta-nicole/recipes-bsp/u-boot/files/0004-aspeed-add-bmc-position-support.patch
51meta-yadro/meta-nicole/recipes-kernel/linux/linux-aspeed/0001-Add-NCSI-channel-selector.patch
52meta-yadro/meta-nicole/recipes-phosphor/host/op-proc-control/0001-Stop-and-send-SRESET-for-one-thread-only.patch
53meta-yadro/recipes-phosphor/dbus/phosphor-dbus-interfaces/0001-Add-boot-initiator-mailbox-interface.patch
54meta-yadro/recipes-phosphor/ipmi/phosphor-ipmi-host/0001-Add-support-for-persistent-only-settings.patch
55meta-yadro/recipes-phosphor/ipmi/phosphor-ipmi-host/0002-Add-support-for-boot-initiator-mailbox.patch
56meta-yadro/recipes-phosphor/ipmi/phosphor-ipmi-host/0003-Fix-version-parsing-update-AUX-revision-info.patch
Patrick Williams693d3db2021-10-26 16:10:20 -050057" | sort > "$allowed_patches_tmp"
Ed Tanous7f8954e2021-09-09 11:46:43 -070058
Patrick Williams693d3db2021-10-26 16:10:20 -050059files_diff=$(comm -23 "$patch_files_tmp" "$allowed_patches_tmp")
Ed Tanous7f8954e2021-09-09 11:46:43 -070060
Ed Tanous4734c222021-09-20 11:11:34 -070061files_count=$(echo -n "$files_diff" | grep -c '^' || true)
Ed Tanous7f8954e2021-09-09 11:46:43 -070062if [[ $files_count -ne 0 ]]; then
63 echo "Patch files found not in allow list"
64 echo "$files_diff"
65 echo "Patches are not allowed on OpenBMC in these layers. Please upstream your changes and see \
66 https://github.com/openbmc/docs/blob/master/meta-layer-guidelines.md"
67 exit 1
68fi
69
Andrew Geisslerfc328172022-01-21 09:37:06 -060070# Now verify there are no kernel dts files being added in a patch
71# All dts files should be sent upstream and can be carried in the openbmc/linux
72# tree until they are accepted upstream
73dts_files_tmp=$(mktemp)
74allowed_dts_tmp=$(mktemp)
75trap 'rm $dts_files_tmp $allowed_dts_tmp' exit
76
77git ls-files -- \
78 '*.dts' \
79 ':!:poky/**' \
Brad Bishopbec4ebc2022-08-03 09:55:16 -040080 ':!:meta-arm/**' \
Andrew Geisslerfc328172022-01-21 09:37:06 -060081 ':!:meta-security/**' \
82 ':!:meta-raspberrypi/**' \
83 ':!:meta-openembedded/**' \
84 | sort > "$dts_files_tmp"
85
86# There is a single dts currently in the tree that we will exempt for now
87echo "\
88meta-hpe/meta-dl360poc/recipes-kernel/linux/linux-obmc/gxp.dts
89" | sort > "$allowed_dts_tmp"
90
91files_diff=$(comm -23 "$dts_files_tmp" "$allowed_dts_tmp")
92
93files_count=$(echo -n "$files_diff" | grep -c '^' || true)
94if [[ $files_count -ne 0 ]]; then
95echo "Dts files found not in allow list"
96echo "$files_diff"
97echo "Dts files are not allowed on OpenBMC in these layers. Please upstream your changes and see \
98 https://github.com/openbmc/docs/blob/master/kernel-development.md"
99exit 1
100fi
101
Patrick Williams12a27b22021-10-26 16:07:31 -0500102lint_exempt="\
Patrick Williams12a27b22021-10-26 16:07:31 -0500103meta-phosphor/recipes-support/nss-pam-ldapd/files/nslcd.init
Patrick Williams12a27b22021-10-26 16:07:31 -0500104"
105
Patrick Williams2ed00d82021-10-27 09:33:53 -0500106types=(json shell)
107# shellcheck disable=SC2034
108check_json="eslint --resolve-plugins-relative-to /usr/local/lib/node_modules"
Patrick Williams521d8f62021-10-27 09:30:41 -0500109# shellcheck disable=SC2034
110check_shell="shellcheck -x"
111
112for t in "${types[@]}"; do
113 check_cmd="check_${t}"
114 if ! which "${!check_cmd%% *}" > /dev/null 2>&1; then
115 eval "${check_cmd}=\"echo WARNING: Skipping $t due to missing command:\""
116 echo "${!check_cmd}"
117 fi
118done
119
Patrick Williams12a27b22021-10-26 16:07:31 -0500120non_bbfiles=$(git ls-files -- \
121 ':!:poky/**' \
Brad Bishopbec4ebc2022-08-03 09:55:16 -0400122 ':!:meta-arm/**' \
Patrick Williams12a27b22021-10-26 16:07:31 -0500123 ':!:meta-security/**' \
124 ':!:meta-raspberrypi/**' \
125 ':!:meta-openembedded/**' \
126 | grep -v -e "\.patch$" -e "\.bb$" -e "\.bbappend$")
127
128for f in $non_bbfiles; do
Patrick Williams521d8f62021-10-27 09:30:41 -0500129 unset file_type
130 file_info=$(file "$f")
131 case $file_info in
Patrick Williams12a27b22021-10-26 16:07:31 -0500132 *shell\ script*)
Patrick Williams521d8f62021-10-27 09:30:41 -0500133 file_type="shell"
Patrick Williams12a27b22021-10-26 16:07:31 -0500134 ;;
135
Patrick Williams2ed00d82021-10-27 09:33:53 -0500136 *JSON\ data*)
137 file_type="json"
138 ;;
139
Patrick Williams12a27b22021-10-26 16:07:31 -0500140 *)
Patrick Williams521d8f62021-10-27 09:30:41 -0500141 case $f in
142 *.sh)
143 file_type="shell"
144 ;;
145
Patrick Williams2ed00d82021-10-27 09:33:53 -0500146 *.json)
147 file_type="json"
148 ;;
Patrick Williams521d8f62021-10-27 09:30:41 -0500149 esac
Patrick Williams12a27b22021-10-26 16:07:31 -0500150 esac
Patrick Williams521d8f62021-10-27 09:30:41 -0500151
152 if [ -n "$file_type" ]; then
153 check_cmd="check_${file_type}"
154 if ! eval "${!check_cmd} $f"; then
155 if [[ $lint_exempt == *$f* ]]; then
156 echo "EXEMPT: $f"
157 else
158 echo "FAILED: $f"
159 false
160 fi
161 fi
162 fi
163
Patrick Williams12a27b22021-10-26 16:07:31 -0500164done
165
Ed Tanousd5cea142021-09-09 11:42:05 -0700166echo "Repo test passed"