talos-hostboot/src/build/tools/pre-commit, branch 07-25-2019

Fixed portabiliy issue in pre-commit hook

2019-10-02T13:29:33+00:00

A change was previously applied to pre-commit via 1d4530dac35 that included a bashism that is not portable with other shells. Change-Id: I0ab536005973627953e1a3b4fa41f51963a953ea Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84413 Tested-by: Jenkins Server Reviewed-by: Luis P Fernandez Tested-by: Jenkins OP Build CI Tested-by: FSP CI Jenkins Tested-by: Jenkins OP HW Reviewed-by: Daniel M Crowell

Implement Static Analysis with cppcheck

2019-09-11T15:53:06+00:00

Cppcheck static analysis tool can now be run when compiling by running: make cppcheck And it will always run when committing without stopping the commit if an error is found. This commit suppresses some false positives found in the code. A false positive can be suppressed inline by placing “// cppcheck-suppress syntaxError” in a line before the false positive., e.g.: char arr[5]; // cppcheck-suppress arrayIndexOutOfBounds arr[10] = 0; Change-Id: I9e833a153d6b04df2f8a72cd994e43922c61c2fe RTC: 201451 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79629 Reviewed-by: Zachary Clark Reviewed-by: Nicholas E Bofferding Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Daniel M Crowell

pre-commit: Improve error message if not in a workon

2017-07-07T20:29:50+00:00

Change-Id: I8c8597f8a415053094d0886e972c9ca1e93cc09d Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42028 Reviewed-by: Matt K. Light Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: FSP CI Jenkins Reviewed-by: Thi N. Tran Reviewed-by: Nicholas E. Bofferding

Enable the import tree to have hostboot prologs

2016-08-05T14:44:47+00:00

Also removed suffix from addCopyright script Change-Id: I83f633ee8a9e91eccf817756e8f4f19a6c89da27 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/22460 Tested-by: Jenkins Server Tested-by: FSP CI Jenkins Reviewed-by: Matthew A. Ploetz Reviewed-by: Daniel M. Crowell Reviewed-by: William G. Hoffa

Fix for legacy support of pre-commit hook script

2015-10-09T20:30:48+00:00

Change-Id: I859007e056890497651ed29aada7d8e01a6f7a44 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20950 Reviewed-by: STEPHEN M. CPREK Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: A. Patrick Williams III

Provide legacy support for pre-commit hook script

2015-10-05T18:18:10+00:00

Change-Id: Idad68f690c14959c0978653f343cb2824acc5028 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20598 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: PRACHI GUPTA Reviewed-by: A. Patrick Williams III

Disable git hooks when running mirror tool from another repo

2015-09-29T18:22:47+00:00

Change-Id: Ic2953f6f375996002c6d7a464c2af9697bfe0179 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20764 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: PRACHI GUPTA Reviewed-by: A. Patrick Williams III

Generalize env vars, env setup, and copyright scripts

2015-09-01T21:18:32+00:00

Methods ported over from code in EKB. Change-Id: I26b6f2326314936b0316f64272230730ecd3f9ee RTC: 126634 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/17465 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: A. Patrick Williams III

Change copyright prolog for all files to Apache.

2014-05-22T03:16:32+00:00

Change-Id: I5664587b4f889099290ef50d50fa9ce5e580e1eb Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/11167 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III

Fixed some issues with pre/post commit scripts

2014-04-10T20:57:20+00:00

Change-Id: I9da6959c32fcc967ef86348b7d4a93b1c2bf86f3 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/9877 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III