Skip to content

Commit 416105a

Browse files
Patch version (#37)
* Add SMF_PLUGIN_PATCH_VERSION to build_plugin workflow. * Update build_plugin.yml * Update create_release.yml * Pass the SMF_PLUGIN_PATCH_VERSION through all the workflows. * Pass SMF_PLUGIN_PATCH_VERSION to windows cmake. * EBP-330: Add type to all required intputs * Added a missing `$` to workflow file that introduces input variable.
1 parent afb444c commit 416105a

File tree

6 files changed

+41
-7
lines changed

6 files changed

+41
-7
lines changed

.github/workflows/build_plugin.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,11 @@ on:
1818
description: "Patch version of the wireshark tag to use when building plugin"
1919
required: false
2020
default: ''
21+
SMF_PLUGIN_PATCH_VERSION:
22+
description: "PATCH for SMF plugin"
23+
type: string
24+
required: true
25+
default: ''
2126
jobs:
2227
# Commenting out intel mac for regular builds as it is currently broken.
2328
#build_macos:
@@ -32,15 +37,20 @@ jobs:
3237
MAJOR_VERSION: ${{ inputs.MAJOR_VERSION || '4' }}
3338
MINOR_VERSION: ${{ inputs.MINOR_VERSION || '6' }}
3439
PATCH_VERSION: ${{ inputs.PATCH_VERSION || '0' }}
40+
SMF_PLUGIN_PATCH_VERSION: ${{ inputs.SMF_PLUGIN_PATCH_VERSION || 'dev' }}
3541
build_windows:
3642
uses: ./.github/workflows/windows.yml
3743
with:
3844
MAJOR_VERSION: ${{ inputs.MAJOR_VERSION || '4' }}
3945
MINOR_VERSION: ${{ inputs.MINOR_VERSION || '6' }}
4046
PATCH_VERSION: ${{ inputs.PATCH_VERSION || '0' }}
47+
# this goes into FILEVERSION which appears when you hover over the DLL, but cannot
48+
# contain a string like "dev"
49+
SMF_PLUGIN_PATCH_VERSION: ${{ inputs.SMF_PLUGIN_PATCH_VERSION || '0' }}
4150
build_linux:
4251
uses: ./.github/workflows/ubuntu.yml
4352
with:
4453
MAJOR_VERSION: ${{ inputs.MAJOR_VERSION || '4' }}
4554
MINOR_VERSION: ${{ inputs.MINOR_VERSION || '6' }}
4655
PATCH_VERSION: ${{ inputs.PATCH_VERSION || '0' }}
56+
SMF_PLUGIN_PATCH_VERSION: ${{ inputs.SMF_PLUGIN_PATCH_VERSION || 'dev' }}

.github/workflows/create_release.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ on:
1616
default: '0'
1717
SMF_PLUGIN_PATCH_VERSION:
1818
description: "PATCH for SMF plugin"
19+
type: string
1920
required: true
2021
default: ''
2122
env:
@@ -30,21 +31,24 @@ jobs:
3031
MAJOR_VERSION: ${{ inputs.MAJOR_VERSION || '4' }}
3132
MINOR_VERSION: ${{ inputs.MINOR_VERSION || '6' }}
3233
PATCH_VERSION: ${{ inputs.PATCH_VERSION || '0' }}
33-
34+
SMF_PLUGIN_PATCH_VERSION: ${{ inputs.SMF_PLUGIN_PATCH_VERSION || '0' }}
35+
3436
build_windows:
3537
uses: SolaceLabs/wireshark-smf-plugin/.github/workflows/windows.yml@main
3638
with:
3739
MAJOR_VERSION: ${{ inputs.MAJOR_VERSION || '4' }}
3840
MINOR_VERSION: ${{ inputs.MINOR_VERSION || '6' }}
3941
PATCH_VERSION: ${{ inputs.PATCH_VERSION || '0' }}
40-
42+
SMF_PLUGIN_PATCH_VERSION: ${{ inputs.SMF_PLUGIN_PATCH_VERSION || '0' }}
43+
4144
build_macos:
4245
uses: SolaceLabs/wireshark-smf-plugin/.github/workflows/macos_arm64.yml@main
4346
with:
4447
MAJOR_VERSION: ${{ inputs.MAJOR_VERSION || '4' }}
4548
MINOR_VERSION: ${{ inputs.MINOR_VERSION || '6' }}
4649
PATCH_VERSION: ${{ inputs.PATCH_VERSION || '0' }}
47-
50+
SMF_PLUGIN_PATCH_VERSION: ${{ inputs.SMF_PLUGIN_PATCH_VERSION || '0' }}
51+
4852
create_release:
4953
runs-on: ubuntu-latest
5054
# Wait for all jobs to finish before continuing

.github/workflows/macos.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,11 @@ on:
2020
type: string
2121
required: true
2222
default: "0"
23+
SMF_PLUGIN_PATCH_VERSION:
24+
description: "PATCH for SMF plugin"
25+
type: string
26+
required: true
27+
default: ""
2328

2429
jobs:
2530
macos:
@@ -76,7 +81,7 @@ jobs:
7681
-DPython_FIND_STRATEGY=LOCATION \
7782
-DSMF_PLUGIN_MAJOR_VERSION=${{ inputs.MAJOR_VERSION }} \
7883
-DSMF_PLUGIN_MINOR_VERSION=${{ inputs.MINOR_VERSION }} \
79-
-DSMF_PLUGIN_PATCH_VERSION=${{ inputs.PATCH_VERSION }} \
84+
-DSMF_PLUGIN_PATCH_VERSION=${{ inputs.SMF_PLUGIN_PATCH_VERSION }} \
8085
-DSMF_PLUGIN_COMMIT_HASH=0 \
8186
-DCMAKE_POLICY_DEFAULT_CMP0094=NEW
8287

.github/workflows/macos_arm64.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,11 @@ on:
3535
type: string
3636
required: true
3737
default: '0'
38+
SMF_PLUGIN_PATCH_VERSION:
39+
description: "PATCH for SMF plugin"
40+
type: string
41+
required: true
42+
default: ''
3843

3944
env:
4045
# This is the name for the file in releases
@@ -102,7 +107,7 @@ jobs:
102107
-DPython_EXECUTABLE=${{ env.pythonLocation }}/bin/python \
103108
-DSMF_PLUGIN_MAJOR_VERSION=${{ inputs.MAJOR_VERSION }} \
104109
-DSMF_PLUGIN_MINOR_VERSION=${{ inputs.MINOR_VERSION }} \
105-
-DSMF_PLUGIN_PATCH_VERSION=${{ inputs.PATCH_VERSION }} \
110+
-DSMF_PLUGIN_PATCH_VERSION=${{ inputs.SMF_PLUGIN_PATCH_VERSION }} \
106111
-DSMF_PLUGIN_COMMIT_HASH=0 \
107112
-DPython_FIND_STRATEGY=LOCATION
108113

.github/workflows/ubuntu.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,11 @@ on:
3535
type: string
3636
required: true
3737
default: '0'
38+
SMF_PLUGIN_PATCH_VERSION:
39+
description: "PATCH for SMF plugin"
40+
type: string
41+
required: true
42+
default: ''
3843

3944
env:
4045
# This is the name for the file in releases
@@ -77,7 +82,7 @@ jobs:
7782
run: cmake -GNinja ${{ github.workspace }}/wireshark \
7883
-DSMF_PLUGIN_MAJOR_VERSION=${{ inputs.MAJOR_VERSION }} \
7984
-DSMF_PLUGIN_MINOR_VERSION=${{ inputs.MINOR_VERSION }} \
80-
-DSMF_PLUGIN_PATCH_VERSION=${{ inputs.PATCH_VERSION }} \
85+
-DSMF_PLUGIN_PATCH_VERSION=${{ inputs.SMF_PLUGIN_PATCH_VERSION }} \
8186
-DSMF_PLUGIN_COMMIT_HASH=0
8287

8388
- name: Build

.github/workflows/windows.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,11 @@ on:
3535
type: string
3636
required: true
3737
default: '0'
38+
SMF_PLUGIN_PATCH_VERSION:
39+
description: "PATCH for SMF plugin"
40+
type: string
41+
required: true
42+
default: ''
3843

3944
env:
4045
# This is the name for the file in releases
@@ -101,7 +106,7 @@ jobs:
101106

102107
- name: Cmake
103108
run: |
104-
cmake -DUSE_qt6=ON -DCMAKE_SYSTEM_VERSION="10.0.20348.0" -A x64 ../wireshark -DCMAKE_POLICY_DEFAULT_CMP0094=NEW -DPython3_ROOT_DIR='${{ env.Python3_ROOT_DIR }}' -DPython3_EXECUTABLE='${{ env.Python3_ROOT_DIR }}\python.exe' -DPython3_LIBRARY='${{ env.Python3_ROOT_DIR }}\libs\python380.lib' -DPython3_INCLUDE_DIR='${{ env.Python3_ROOT_DIR }}\Include'
109+
cmake -DUSE_qt6=ON -DCMAKE_SYSTEM_VERSION="10.0.20348.0" -A x64 ../wireshark -DCMAKE_POLICY_DEFAULT_CMP0094=NEW -DPython3_ROOT_DIR='${{ env.Python3_ROOT_DIR }}' -DPython3_EXECUTABLE='${{ env.Python3_ROOT_DIR }}\python.exe' -DPython3_LIBRARY='${{ env.Python3_ROOT_DIR }}\libs\python380.lib' -DPython3_INCLUDE_DIR='${{ env.Python3_ROOT_DIR }}\Include' -DSMF_PLUGIN_PATCH_VERSION='${{ inputs.SMF_PLUGIN_PATCH_VERSION }}'
105110
env:
106111
PLATFORM: x64
107112
WIRESHARK_BASE_DIR: C:/wireshark-libs

0 commit comments

Comments
 (0)