summaryrefslogtreecommitdiff
path: root/meta-phosphor/scripts
AgeCommit message (Collapse)AuthorFilesLines
2021-11-05meta-ampere: repotest: more removalsPatrick Williams1-4/+0
More patches and scripts have been cleaned up. Remove them from the exemption lists. Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Ieae41805e2787d7f63db2fd776495d79e6faa987
2021-11-05repotest: remove sh extension aliasPatrick Williams1-1/+0
The tooling in openbmc-build-scripts no longer needs the symlink alias. Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Ib214c4f2bd5cc5854626a554adb86c9b2ceb1d2b
2021-11-05meta-ampere: repotest: reduce exempted shellcheck scriptsPatrick Williams1-14/+0
Commit dde1fede1f832f029742a1d27290cfe252ab1bc5 cleaned up a number of shellcheck errors from meta-ampere. Remove these, now fixed, scripts from the repotest exemption list. Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Ia575d3130678d3ca2be69da64639f7354c28aea8
2021-11-04repotest: reduce patches in meta-bytedanceLei YU1-3/+0
With the previous commit and 98f38d4c8, several patches in meta-bytedance are removed. Remove such patches from repotest. Signed-off-by: Lei YU <yulei.sh@bytedance.com> Change-Id: If6226949a8063ca5e2aab56aff2b57e54cc963f8
2021-11-04meta-facebook: fix shellcheck issues in scriptsPatrick Williams1-3/+0
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I62cf17d414795e91d673d468a196d4f0a6c0a71b
2021-11-04repotest: add json lintingPatrick Williams1-1/+10
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I99dadfac3c2d324bb5ef6a5ab399ad35825b34d5
2021-11-04repotest: refactor shellcheck slightlyPatrick Williams1-11/+43
Refactor the shellcheck implementation so that: 1. Files are checked by extension if their `file` type doesn't match a known list. Some shell scripts do not use the proper shebang and end up not being detected by `file`; attempt to revert to extension in that case. 2. Facilitate easier addition of multiple linting tools for other file types. A follow up commit will add 'json' linting with only a few changes. Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I5ce60432c4bd123d9d9c14466505c2d4b2477959
2021-11-04setup: fix shellcheck errorsPatrick Williams1-1/+0
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I4c09e61294506659500f9760993dd4b8b6f22f23
2021-11-04repotest: fix shellcheck errorsPatrick Williams1-7/+6
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I4f4e7957218642dc252880a0a0ea575ebde10512
2021-11-04repotest: enable shellcheck-based lintingPatrick Williams1-1/+134
Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I48ff0b633697ddd16a7119519399c6579688436d
2021-10-27meta-xilinx: remove subtreePatrick Williams1-1/+0
The meta-xilinx layer was used for a now-deleted EVB. Neither the EVB nor the meta-xilinx layer have been updated for the Yocto override syntax change and the meta-xilinx still doesn't have a hardknott or honister branch (or corresponding support). I've asked the Xilinx maintainer back in May on when a hardknott version would be supported and I was told "about a month from now". I followed up in August and was told "work is in progress". As of today there are still zero commits in meta-xilinx since January 2021. As such, I do not believe this layer is well-maintained and we have no specific use for it anymore. Remove it until someone finds a good reason to include it and the upstream shows signs of life. Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Id14ea55db2ac2779edf42e63cb57ad7d25172ad5
2021-10-27repotest: clean up via trapPatrick Williams1-4/+1
In the case of early exit, the script could leave tempfiles laying around. Use `trap` to ensure they're properly cleaned up. Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I403dcacbd15227e445d50ccb2c2627675376fdcd
2021-10-26repotest: remove extensionPatrick Williams2-83/+84
The fact the script is current written in bash is not useful information to anyone invoking it. Remove the extension to allow future rewrite, if desired, without impacting other tooling or users. Temporarily create a symlink until `openbmc-build-scripts` can be updated for new script name. Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: Ib9cf07b587cae78c09e78184e0d35f2ed95d77ad
2021-10-19repotest: reduce patches in meta-amperePatrick Williams1-3/+0
With 8115d65b8c83099b62f303de7d5dbccacd2761b6, the number of required patches for meta-ampere's u-boot config was reduced. Remove the old patches from the repotest. Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Change-Id: I4ffa2a0134d3f630ae368e79ac34cb6f6e3a960c
2021-09-20Ignore grep failuresEd Tanous1-1/+1
When the last round of reviews added -e to the bash statement, it exposed a slight flaw, in that when there aren't failures, grep will return a code if it's not able to find any matches. This commit changes that such that grep errors are "ignored" in the context of bash. Tested: ran run-repotest.sh on a clean repository, and observed "repo test passed" Signed-off-by: Ed Tanous <edtanous@google.com> Change-Id: I716220652d48b9a8489bf4dc0c3b4734b98a70e4
2021-09-18Implement a test for patch complianceEd Tanous1-0/+82
Per our documentation, patches outside of a couple specific directories aren't allowed. This commit implements a build-time check to see if any new patches have been added. Functionally, because there's several meta layers that have broken the rules already, it has to contain a check against the various patches that already exist in the tree. This list doesn't constitute a grandfather-ing in of those patches, but recognizes that we can't immediately back them out, and having the maintainers take care of them will take time. Of note, this patchset also ignores patches in any of the poky layers, as we can't define their policies, and allows patches in meta-phosphor, as a backdoor in case we need a trapdoor to this policy, and all the maintainers agree to it. This change was originally checked into openbmc-build-scripts, but was moved to support more users, and keeping this list up to date. All comments on the latest review from https://gerrit.openbmc-project.xyz/c/openbmc/openbmc-build-scripts/+/46064/2 have been addressed. Change-Id: I1c3e062278b4856cbfda910bdbf122e4b7539f89 Signed-off-by: Ed Tanous <edtanous@google.com>
2021-09-18Add repo tests to main repositoryEd Tanous1-0/+4
This commit adds the repo test scripts to the main openbmc repository to be used for verification of meta layers. It is intended to go in hand with https://gerrit.openbmc-project.xyz/c/openbmc/openbmc-build-scripts/+/46063 to allow testing meta layers at compile time for compliance. For the moment it is just a no-op test; In the future, more capabilities will be added. Signed-off-by: Ed Tanous <edtanous@google.com> Change-Id: I034c9cd60590907db73d759eb9d943e3e62e62ec