summaryrefslogtreecommitdiffstats
path: root/src/build/tools/pre-commit
Commit message (Collapse)AuthorAgeFilesLines
* Fixed portabiliy issue in pre-commit hookZane Shelley2019-10-021-1/+1
| | | | | | | | | | | | | | 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 <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Luis P Fernandez <luis.fernandez@ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Implement Static Analysis with cppcheckMegan2019-09-111-1/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 <zach@ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* pre-commit: Improve error message if not in a workonJoachim Fenkes2017-07-071-2/+4
| | | | | | | | | | | Change-Id: I8c8597f8a415053094d0886e972c9ca1e93cc09d Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42028 Reviewed-by: Matt K. Light <mklight@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Thi N. Tran <thi@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
* Enable the import tree to have hostboot prologsStephen Cprek2016-08-051-11/+10
| | | | | | | | | | | | Also removed suffix from addCopyright script Change-Id: I83f633ee8a9e91eccf817756e8f4f19a6c89da27 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/22460 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Fix for legacy support of pre-commit hook scriptStephen Cprek2015-10-091-1/+0
| | | | | | | | | | | Change-Id: I859007e056890497651ed29aada7d8e01a6f7a44 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20950 Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com> 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 <iawillia@us.ibm.com>
* Provide legacy support for pre-commit hook scriptStephen Cprek2015-10-051-1/+10
| | | | | | | | | | | 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 <pragupta@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Disable git hooks when running mirror tool from another repoStephen Cprek2015-09-291-2/+3
| | | | | | | | | | | 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 <pragupta@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Generalize env vars, env setup, and copyright scriptsStephen Cprek2015-09-011-6/+4
| | | | | | | | | | | | | 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 <iawillia@us.ibm.com>
* Change copyright prolog for all files to Apache.Patrick Williams2014-05-211-10/+10
| | | | | | | Change-Id: I5664587b4f889099290ef50d50fa9ce5e580e1eb Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/11167 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Fixed some issues with pre/post commit scriptsZane Shelley2014-04-101-2/+4
| | | | | | | Change-Id: I9da6959c32fcc967ef86348b7d4a93b1c2bf86f3 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/9877 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Hook pre-applypatch fails due to missing git-sh-setup.Patrick Williams2013-03-011-14/+15
| | | | | | | | | Change-Id: I2477136d9ce56c590c5f189fbac19337b85e812e Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/3382 Tested-by: Jenkins Server Reviewed-by: Mark W. Wenning <wenning@us.ibm.com> Reviewed-by: Bradley W. Bishop <bradleyb@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Scripts to add copyright blocks on all files during a commitMark Wenning2011-08-241-0/+26
- created - run src/build/tools/setupgithooks.sh to config git to use these scripts - use git log to get the date of the first checkin of the file - if different, put "copyright 20xx - 2011" string. - modify copyright block to say "HostBoot" instead of FSP - add copyright-check.sh script to run during gerrit trigger - new requirements: use templates from "FSP Clearcase Architecture" - use C++ style comments for C, H files - add xml section - set up copyright-check.sh and cxxtest-start.sh Change-Id: Id1e5d78dc0498fe4ef43eadbea81d6e6c5fab413 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/199 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com> Reviewed-by: CAMVAN T. NGUYEN <ctnguyen@us.ibm.com>
OpenPOWER on IntegriCloud