Commit Graph

46 Commits

Author SHA1 Message Date
Vadim Yanitskiy be18bcd4ed coverity: build osmo-trx with MS TRX (--with-mstrx)
Change-Id: I4fbb052eacd8e6b8ddb8130d2e280a248719b250
Related: OS#5599
2022-12-29 18:12:39 +07:00
Vadim Yanitskiy ac0b663951 {jobs,coverity}: add a new project - libosmo-gprs
Change-Id: I83e86084e9f7a7431bf5a70c372da1f9ccd28436
2022-08-10 04:39:36 +07:00
Vadim Yanitskiy 6d81605790 coverity: add missing libosmo-pfcp, dependency of osmo-upf
Since recently [1] osmo-upf depends on libosmo-{gtlv,pfcp}.so, which
are provided by libosmo-pfcp.git.  We need to build libosmo-pfcp
before attempting to build osmo-upf, otherwise it fails:

  configure: error: Package requirements (libosmo-gtlv >= 0.1.0) were not met:

  Package libosmo-gtlv was not found in the pkg-config search path.
  Perhaps you should add the directory containing `libosmo-gtlv.pc'
  to the PKG_CONFIG_PATH environment variable
  Package 'libosmo-gtlv', required by 'world', not found

Change-Id: Ic4fe7b40cf9a57e0462b2c0806f8810fcc04a10c
2022-08-09 15:19:19 +07:00
Vadim Yanitskiy 618f171cb1 Revert "coverity/build_Osmocom.sh: run 'make check' in do_build()"
This reverts commit bf84ba9f69.

Change-Id: I722f8cbb1934a4f5ce2969e243f223019f123777
2022-07-04 14:38:45 +07:00
Vadim Yanitskiy 9baa805a74 Revert "coverity/build_Osmocom.sh: don't do check for all"
This reverts commit ae41942214.

Change-Id: I5018e0da020f32e3a46a2772e79794e5a3a686ca
2022-07-04 14:38:45 +07:00
Oliver Smith ae41942214 coverity/build_Osmocom.sh: don't do check for all
'make check' does not pass for asn1c, it fails with the following. I
looked into it and the cause is not obvious to me, so in order to make
the jenkins job pass again I've changed it back to just run 'make'
instead.

Fix for:
  ...
  fatal: making test-suite.log: failed to create ../tests/98-attribute-class-OK.asn1.log
  fatal: making test-suite.log: failed to create ../tests/99-class-sample-OK.asn1.trs
  fatal: making test-suite.log: failed to create ../tests/99-class-sample-OK.asn1.log
  Makefile:735: recipe for target 'test-suite.log' failed
  make[3]: *** [test-suite.log] Error 1
  make[3]: Leaving directory '/home/osmocom-build/osmo-ci/coverity/source-Osmocom/asn1c/libasn1fix'
  Makefile:841: recipe for target 'check-TESTS' failed

Disable 'make check' for libusrp too as it also didn't pass:

  Making check in usrp2
  make[4]: Entering directory '/tmp/coverity/source-Osmocom/libusrp/firmware/src/usrp2'
  test -f `basename 'eeprom_boot.a51'` || ln -s 'eeprom_boot.a51' .
  test -f ../common/`basename 'eeprom_boot.a51'` -o \
          \! -f `dirname 'eeprom_boot.a51'`/../common/`basename 'eeprom_boot.a51'` \
          || ln -s `dirname 'eeprom_boot.a51'`/../common/`basename 'eeprom_boot.a51'` ../common/`basename 'eeprom_boot.a51'`
  sdas8051 -plosgff `basename 'eeprom_boot.a51'`
  sdcc -mmcs51 --no-xinit-opt -I../../../firmware/include -I../../../firmware/src/usrp2 -I../../../firmware/src/common -I../../../firmware/src/common -DHAVE_USRP2 \
          -c -o eeprom_init.rel `test -f 'eeprom_init.c' || echo './'`eeprom_init.c
  test -f `basename '_startup.a51'` || ln -s '_startup.a51' .
  test -f ../common/`basename '_startup.a51'` -o \
          \! -f `dirname '_startup.a51'`/../common/`basename '_startup.a51'` \
          || ln -s `dirname '_startup.a51'`/../common/`basename '_startup.a51'` ../common/`basename '_startup.a51'`
  sdas8051 -plosgff `basename '_startup.a51'`
  make[4]: *** No rule to make target '../../lib/libfx2.lib', needed by 'eeprom_boot.ihx'.  Stop.

All other 'make check' calls do work as expected, the build script runs
through again with this patch.

Fixes: bf84ba9f ("coverity/build_Osmocom.sh: run 'make check' in do_build()")
Change-Id: I6b9eb8e73c51d925897579fb51bc0a1e86e77270
2022-06-30 12:25:56 +02:00
Vadim Yanitskiy bf84ba9f69 coverity/build_Osmocom.sh: run 'make check' in do_build()
This enables building and running tests (check_PROGRAMS), so that
Coverity will be able to find bugs in their code too.

Change-Id: I3f3149a61df96646798c72d0ad31c41b97e6b81f
2022-06-29 13:40:16 +00:00
Oliver Smith 8637f74ced coverity: add osmo-bsc-nat
Fixes: OS#5596
Change-Id: Id7cf497ea15c3ccdf76c047c6a67f17fac94e0dd
2022-06-28 12:11:15 +02:00
Pau Espin 702fd8ff94 Add new repo osmo-hnbgw (nightly) to build jobs and scripts
Change-Id: I2fdbd651e4b50191e1a67e183590de99d6366ca5
2022-01-05 11:39:41 +01:00
Vadim Yanitskiy 72e42f70d5 coverity: disable Doxygen when building projects
Change-Id: I2fe2f80eb3ec00e9b6c0a3c0ee96e1a0da217b27
2021-10-25 10:00:24 +00:00
Pau Espin 2146640769 Add osmo-hnodeb to ansible/jenkins/coverity setup
Change-Id: Ie4b03aefeda13d216706bd648145689cf2ef93bb
2021-10-25 11:06:48 +02:00
Vadim Yanitskiy 1dcc508ce6 coverity: fix build_simtrace2(): generate Makefile using autotools
Starting from I57e77f927ee9e169cc794c5dc6b128a2d590201b, autotools
are required in order to configure and build the project.

Change-Id: I48696af1ccb5eb884f5de9085bcda37fc41766fb
Fixes: I74da8b523c9d51887d5eceb01ca0f505bb846812
2021-10-24 16:44:29 +03:00
Vadim Yanitskiy c2b01a9b81 fixup: coverity: also build and check SIMTrace host tools
Change-Id: I97b173a4b3fe47a119145e8d6fa0ad116fd38237
2021-10-24 13:22:00 +03:00
Vadim Yanitskiy d304d61327 coverity: also build and check osmo-{cbc,gbproxy}
Change-Id: Ibbff2bdf3163ba33dac11ccaa2e405894a7a0f10
2021-10-21 22:16:04 +03:00
Vadim Yanitskiy 56489bc521 coverity: also build and check SIMTrace host tools
Change-Id: I74da8b523c9d51887d5eceb01ca0f505bb846812
2021-10-21 22:16:04 +03:00
Vadim Yanitskiy 27e5f8ac15 coverity: build osmo-trx with USRP1 support
This allows Coverity to analyze the libusrp specific code.

Change-Id: I36df72149a32cb0df43c74af910d1fbb92b88f4a
2021-10-21 22:16:04 +03:00
Vadim Yanitskiy 0909dade00 coverity: build osmo-trx with UHD+LMS+IPC support
This allows Coverity to analyze the UHD/LMS/IPC specific code.

Change-Id: Ib58bee277414596cb46e523c798d42514c0eb78d
2021-10-21 22:16:04 +03:00
Vadim Yanitskiy baee7c45e0 coverity: build osmo-msc with --enable-smpp
This allows Coverity to analyze the SMPP specific code.

Change-Id: Ie1e4f1610d86fe57d940527ac64d3d1beb7165e0
2021-10-21 22:16:04 +03:00
Vadim Yanitskiy a2e790e763 coverity: build osmo-{msc,sgsn} with --enable-iu
This allows Coverity to analyze the Iu (UMTS) specific code.

Change-Id: I3ef08fcf42950b33e18b3bfcddff890547308364
2021-10-21 22:16:04 +03:00
Vadim Yanitskiy b25ffe1356 coverity: allow passing configure args to build_default()
Change-Id: I79aa2c65cd0d3b06c0bd9ec4f44288b21e40c3b7
2021-10-21 22:15:54 +03:00
Vadim Yanitskiy 7e053f3a23 coverity: also specify --with-sysmobts for osmo-pcu
It's present in build_osmobts(), so let's add it for consistency.

Change-Id: Ie17e2f9f81258e4d60c8cecf0a7af38e8372f9dd
2021-10-21 22:01:39 +03:00
Vadim Yanitskiy acf98b8ec0 coverity: drop --enable-sysmocom-bts for osmo-pcu
There is no such parameter in osmo-pcu's configure script.

Change-Id: I7f06f3d804acd97526a9eb0f93a23375ec555ee8
2021-10-21 22:01:39 +03:00
Pau Espin 8b8051689f coverity/build_Osmocom.sh: Fix trailing whitespace
Change-Id: I999c112c27d243366a010f216749a067d7b0509d
2021-10-21 12:52:37 +02:00
Vadim Yanitskiy f37d4fee7a coverity: enable parallel build for libsmpp34
We have parallel build enabled in libsmpp34.git since 2018.

Change-Id: Ibe57a8c823d9b35e2f51c9bd7a13d6baf61bf0ca
Related: Id41fbcb5a96093eb6c3dc00bcacbd379111ada70
2021-10-20 21:44:14 +03:00
Neels Hofmeyr 2dc35de85b jenkins: add osmo-smlc to coverity and ttcn3 jobs
Change-Id: I690962cd8352e255a2bf0ae16aa8ddda68edf10a
2020-10-21 00:55:59 +02:00
Vadim Yanitskiy f137b5b1a4 coverity: add OsmocomBB (without firmware)
Change-Id: I2a4bcca411361484e872361dfd16d46189f52f35
2019-05-10 10:42:59 +00:00
Max bdd3713beb Add osmo-sysmon to coverity submission
Change-Id: I1e753d2c14198eccb046c259b3eec582b1921431
2018-11-22 14:21:47 +01:00
Alexander Couzens 3e50c20188 coverity: don't install systemd file
Because the systemd install directory does not use --prefix as
base, they are installed by default to /lib/systemd.
As the coverity only scans the compilation, don't install systemd files.

Change-Id: I39b491fc09f0db0efd8d9d09e8f4cba4ba49cdde
2018-09-19 00:36:58 +02:00
Harald Welte af55468721 coverity: Re-order builds to ensure osmo-hlr before osmo-msc
osmo-msc now needs libosmo-gsup-client, which is built+installed
as part of osmo-hlr

Change-Id: I581d86cf9679ef978a018a5ba8deb8b0f034677b
2018-08-08 07:55:28 +02:00
Max f086d63c13 cosmetic: remove unused function
Change-Id: Ie3885237cea2afbe5e8c0f89d2280284bb4d9150
2018-01-22 11:02:14 +01:00
Max e10bf97f8f Coverity: specify sysmoBTS L1 headers explicitly
Fix upload by specifying L1 headers location properly.

Fixes: OS#2845
Change-Id: Iba55922295cb2a258f90c5345f592d7d5f34ac51
2018-01-22 11:01:49 +01:00
Max 66b46cd9b9 Enable osmo-iuh submission
It was disabled back in a days when it was dependent on a separate
branch. Thisis no longer the case so we can enable it back. Also remove
remaining leftovers.

Change-Id: I5fd3bdab217b61f9ee50a5f1413b60d396535395
2018-01-11 23:07:58 +01:00
Max 3e1ca6f866 Remove OpenBSC from Coverity submission
The OpenBSC codebase only receive occasional backports hence none of the
issues uncovered by Coverity would be fixed unless it's also fixed in
corresponding Osmo* project. Let's not clutter output with the
information about issues which won't be fixed anyway.

Change-Id: Ief3dfc641684fa33407957bf7cfcb6ecf35b847a
2017-12-14 15:21:16 +01:00
Harald Welte 43aa00dbd4 coverity: Don't use PARALLEL_MAKE for libsmpp34
... which apparently doesn't support this and every so often breaks
the coverity upload build

Change-Id: Id78f80cf0878a0807cd06a24fa5c9561c7b84b36
2017-11-05 08:55:12 +01:00
Harald Welte a41cee7703 coverity: build with --enable-trx
Change-Id: I0684bc04b82ad57a1513bbd4627144d6c254b965
2017-10-11 08:45:35 +08:00
Harald Welte f30f7ad645 coverity: clone + build osmo-{mgw,bsc,msc,hlr}
Related: OS#2564
Change-Id: I6f046943f045a97c2bae4e99485a474c11d90a90
2017-10-11 08:44:44 +08:00
Harald Welte e314bfe247 coverity: osmo-bts now builds without openbsc
Change-Id: I09565441c0d9eb907edba82b26df38d08a5d868d
2017-10-11 08:44:23 +08:00
Harald Welte 171bfc2dd7 coverity: Add support for $PARALLEL_MAKE environment variable
Change-Id: I95b0e5f104155ac7c5ae993b7f3c0c0721d0157b
2017-10-11 08:44:23 +08:00
Harald Welte 34058b0856 coverity: Reduce codeduplication in build_Osmocom.sh
We don't need a separate build_foo function if it is identical in
its body except for the 'foo' (project name).  Let's clean this up.

Change-Id: I27e9fc94142b42a7b7c2f9eca89056e1f90f1f0e
2017-10-11 08:44:23 +08:00
Max 8e9fe08080 Use new GPRS repositories
* use coverity check on osmo-ggsn instead of openggsn
* move osmo-sgsn and osmo-ggsn from nightly-split into nightly

Change-Id: Ia49969cbfb9ef57b635a3b5759f411f71a54f8e1
2017-10-02 08:37:40 +00:00
Neels Hofmeyr e72f35cfa9 coverity: add osmo-hlr to be scanned 2017-03-16 14:04:03 +01:00
Neels Hofmeyr 7a83d97807 coverity: build master branches, not iu branches
Rebasing the iu branches onto the master branches is currently not happening
regularly, since I'm focusing on the VLR. Rather scan the master branches so
that the coverity issues are caught without further rebase effort required.

Comment out everything Iu related with '#IU' comment markers:
- don't checkout Iu branches
- don't build osmo-iuh (depends on libosmo-sccp iu branch)
- don't pass --enable-iu to openbsc, don't switch branches for osmo-bts

We might consider adding a separate Iu build, but then we might get the same
coverity warnings twice, so not pursuing that actively now.

Change-Id: I0d6640b893b8d65d321af904b80d89da5bf3ea6a
2016-12-11 03:20:55 +01:00
Neels Hofmeyr 6fbe3aa929 coverity: check out clean sources every time
Change-Id: I76782099e801ab89202c0103f97b7142b2c115fd
2016-10-11 14:06:36 +02:00
Neels Hofmeyr 5b4e64c305 coverity: don't pull --rebase, instead fetch and reset --hard
Change-Id: Ida553a0f38c438a02139396512ff5da038359721
2016-10-11 13:58:06 +02:00
Neels Hofmeyr 9c5feb21c4 coverity: add osmo-trx to Osmocom build
Change-Id: I57d4658120444a4f6884c0b35ec61e8e7034c83c
2016-10-01 01:58:47 +02:00
Neels Hofmeyr a7e0987f68 coverity: rename iuh build to Osmocom
This corresponds to the scan.coverity project name this is uploaded to.

Also apply renaming in file content: call renamed build_Osmocom.sh, use dir
source-Osmocom and install-Osmocom, use a token named 'Osmocom'.

The point is that the combined build of all osmocom projects finds problems
more accurately, since coverity can analyse across all libraries at the same
time.

Still build the iuh branches, since they are basically an extension of the
current master branches (given that they are rebased onto master regularly,
which they currently are).

Name the generated tar Osmocom.tgz instead of myproject.tgz.

Change-Id: If316d54327cca8aa1b499bf35b6b22385bd83702
2016-10-01 01:58:19 +02:00