Skip to content

Compiler warning and CI maintainance

Enrico Bothmann requested to merge tmp-replace-sprintf-with-snprintf into master

Description

  • Fix compiler warnings on recent clang by replacing calls to the deprecated sprintf with snprintf
  • Disable (accidentally) verbose Makefiles on Apple platforms
  • Cleanup CI specification file
  • Re-enable all (available) tests in the CI, note that some tests are only run for certain builds. Currently, only the Fedora builds have all tests enabled (since they configure Sherpa with all bells and whistles)

Test Plan

Any problem with this MR should have been caught by the CI.

Example Result

See the Fedora CI builds which now include all tests (and they all pass).

Additional Checklist Items

  • I have updated documentation as needed
    not applicable
  • I considered if this should be cherry-picked/ported somewhere else (rel-2-x or master), and if yes, opened a corresponding issue or MR
    not applicable
Edited by Enrico Bothmann

Merge request reports