Wireshark-commits: [Wireshark-commits] master-3.2 226939b: gitlab-ci: upgrade to Clang 10, drop spe
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 25 Mar 2020 14:42:12 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=226939b6cc64bc22e45554e61a7a05447db63457
Submitter: "Peter Wu <peter@xxxxxxxxxxxxx>"
Changed: branch: master-3.2
Repository: wireshark

Commits:

226939b by Peter Wu (peter@xxxxxxxxxxxxx):

    gitlab-ci: upgrade to Clang 10, drop special -Wframe-larger-than case
    
    Requires an updated wireshark/wireshark-ubuntu-dev image:
    https://github.com/wireshark/wireshark-ubuntu-dev-docker/pull/2
    
    Remove -Wframe-larger-than while at it. The default size in our CMake
    config is 32k. Clang should not significantly go over it. If so, then it
    has to be solved there, and not in the Gitlab config.
    
    Change-Id: I3891fcbd9dec8e5a4597404aa8131f28a1755a02
    Reviewed-on: https://code.wireshark.org/review/36369
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    (cherry picked from commit 10b88d0d4474ab7b0006939d12fa7ea199d47ff5)
    Reviewed-on: https://code.wireshark.org/review/36569
    

Actions performed:

    from  8885cd2   NAS-5GS: shift supi_fmt field before testing
     add  226939b   gitlab-ci: upgrade to Clang 10, drop special -Wframe-larger-than case


Summary of changes:
 .gitlab-ci.yml | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)