Wireshark-commits: [Wireshark-commits] master-3.0 054553e: 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:25 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=054553eb3a0bb8256bff05f1dc2a3e017012b3dc
Submitter: "Peter Wu <peter@xxxxxxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark

Commits:

054553e 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 7579d9844839d34ac5fcfcc6e3037a6ddc381583)
    Reviewed-on: https://code.wireshark.org/review/36570
    

Actions performed:

    from  b8f2f81   RTCP: contain BYE packet dissection to its stated length
     add  054553e   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(-)