Skip to content

Commit b8551a7

Browse files
authoredJun 4, 2024··
Revert "Add Windows MSI to the release (#560)" (#570)
This reverts commit 4e8f050.
1 parent 99faa93 commit b8551a7

13 files changed

+9
-227
lines changed
 

‎.github/workflows/base-ci-goreleaser.yaml

+1-6
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ jobs:
3232
GOARCH: arm
3333
- GOOS: windows
3434
GOARCH: s390x
35-
runs-on: ubuntu-22.04
35+
runs-on: ubuntu-20.04
3636

3737
steps:
3838
- name: Checkout
@@ -45,11 +45,6 @@ jobs:
4545
with:
4646
platforms: arm64,ppc64le,linux/arm/v7,s390x
4747

48-
- name: Setup wixl
49-
run: |
50-
sudo apt-get update
51-
sudo apt-get install -y wixl
52-
5348
- name: Setup Docker Buildx
5449
uses: docker/setup-buildx-action@d70bba72b1f3fd22344832f00baa16ece964efeb # v3.3.0
5550

‎.github/workflows/base-release.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ jobs:
3232
GOARCH: arm
3333
- GOOS: windows
3434
GOARCH: s390x
35-
runs-on: ubuntu-22.04
35+
runs-on: ubuntu-20.04
3636

3737
steps:
3838
- uses: actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29 # v4.1.6
@@ -95,7 +95,7 @@ jobs:
9595

9696
release:
9797
name: ${{ inputs.distribution }} Release
98-
runs-on: ubuntu-22.04
98+
runs-on: ubuntu-20.04
9999
needs: prepare
100100

101101
permissions:

‎.github/workflows/ci.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ on:
99
jobs:
1010
build:
1111
name: Build
12-
runs-on: ubuntu-22.04
12+
runs-on: ubuntu-20.04
1313
steps:
1414
- name: Checkout
1515
uses: actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29 # v4.1.6

‎CONTRIBUTING.md

+1-2
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,7 @@ This repository contains a set of resources that ultimately results in OpenTelem
66

77
### Distribution directory
88

9-
Each distribution has its own directory under the [`distributions`](./distributions/) folder, such as [`otelcol`](./distributions/otelcol/).
10-
Within each one of those, you'll find at least two files:
9+
Each distribution has its own directory at the root of this repository, such as `opentelemetry-collector`. Within each one of those, you'll find at least two files:
1110

1211
- `Dockerfile`, determining how to build the container image for this distribution
1312
- `manifest.yaml`, which is used with [ocb](https://github.com/open-telemetry/opentelemetry-collector/tree/main/cmd/builder) to generate the sources for the distribution.

‎cmd/goreleaser/internal/configure.go

+3-23
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,8 @@ import (
2424
"path"
2525
"strings"
2626

27-
"github.com/goreleaser/goreleaser-pro/pkg/config"
27+
"github.com/goreleaser/goreleaser/pkg/config"
28+
"github.com/goreleaser/nfpm/v2/files"
2829
)
2930

3031
const ArmArch = "arm"
@@ -44,7 +45,6 @@ func Generate(dist string) config.Project {
4445
Env: []string{"COSIGN_YES=true"},
4546
Builds: Builds(dist),
4647
Archives: Archives(dist),
47-
MSI: WinPackages(dist),
4848
NFPMs: Packages(dist),
4949
Dockers: DockerImages(dist),
5050
DockerManifests: DockerManifests(dist),
@@ -102,26 +102,6 @@ func Archive(dist string) config.Archive {
102102
}
103103
}
104104

105-
func WinPackages(dist string) []config.MSI {
106-
return []config.MSI{
107-
WinPackage(dist),
108-
}
109-
}
110-
111-
// Package configures goreleaser to build a Windows MSI package.
112-
// https://goreleaser.com/customization/msi/
113-
func WinPackage(dist string) config.MSI {
114-
return config.MSI{
115-
ID: dist,
116-
Name: fmt.Sprintf("%s_{{ .Version }}_{{ .Os }}_{{ .MsiArch }}", dist),
117-
WXS: "windows-installer.wxs",
118-
Files: []string{
119-
"config.yaml",
120-
"opentelemetry.ico",
121-
},
122-
}
123-
}
124-
125105
func Packages(dist string) (r []config.NFPM) {
126106
return []config.NFPM{
127107
Package(dist),
@@ -147,7 +127,7 @@ func Package(dist string) config.NFPM {
147127
PostInstall: "postinstall.sh",
148128
PreRemove: "preremove.sh",
149129
},
150-
Contents: config.NFPMContents{
130+
Contents: files.Contents{
151131
{
152132
Source: fmt.Sprintf("%s.service", dist),
153133
Destination: path.Join("/lib", "systemd", "system", fmt.Sprintf("%s.service", dist)),

‎distributions/otelcol-contrib/.goreleaser.yaml

-7
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,6 @@ partial:
33
project_name: opentelemetry-collector-releases
44
env:
55
- COSIGN_YES=true
6-
msi:
7-
- id: otelcol-contrib
8-
name: otelcol-contrib_{{ .Version }}_{{ .Os }}_{{ .MsiArch }}
9-
wxs: windows-installer.wxs
10-
extra_files:
11-
- config.yaml
12-
- opentelemetry.ico
136
builds:
147
- id: otelcol-contrib
158
goos:
-269 KB
Binary file not shown.

‎distributions/otelcol-contrib/windows-installer.wxs

-88
This file was deleted.

‎distributions/otelcol/.goreleaser.yaml

-7
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,6 @@ partial:
33
project_name: opentelemetry-collector-releases
44
env:
55
- COSIGN_YES=true
6-
msi:
7-
- id: otelcol
8-
name: otelcol_{{ .Version }}_{{ .Os }}_{{ .MsiArch }}
9-
wxs: windows-installer.wxs
10-
extra_files:
11-
- config.yaml
12-
- opentelemetry.ico
136
builds:
147
- id: otelcol
158
goos:
-269 KB
Binary file not shown.

‎distributions/otelcol/windows-installer.wxs

-88
This file was deleted.

‎go.mod

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ go 1.21
55
toolchain go1.21.7
66

77
require (
8-
github.com/goreleaser/goreleaser-pro v1.25.1-pro
8+
github.com/goreleaser/goreleaser v1.25.1
99
github.com/goreleaser/nfpm/v2 v2.37.1
1010
gopkg.in/yaml.v3 v3.0.1
1111
)

‎go.sum

-2
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,6 @@ github.com/goreleaser/fileglob v1.3.0 h1:/X6J7U8lbDpQtBvGcwwPS6OpzkNVlVEsFUVRx9+
2121
github.com/goreleaser/fileglob v1.3.0/go.mod h1:Jx6BoXv3mbYkEzwm9THo7xbr5egkAraxkGorbJb4RxU=
2222
github.com/goreleaser/goreleaser v1.25.1 h1:a9skjeROotTN5GPPJDHDfhmOK4n13cBgJ34sTdXRDN0=
2323
github.com/goreleaser/goreleaser v1.25.1/go.mod h1:nsbhCYp9eImbE2fyd9/3Tgv5hjuGuDIQRoBozEUEYbc=
24-
github.com/goreleaser/goreleaser-pro v1.25.1-pro h1:NOoSx96oAK0zNA1+hiL0p6pY1DWL101kwPmpmkiExXk=
25-
github.com/goreleaser/goreleaser-pro v1.25.1-pro/go.mod h1:7q9HURJC4ZYBT9VyX3XlqjK0kwe5QbG/VIUAJSP3CKc=
2624
github.com/goreleaser/nfpm/v2 v2.37.1 h1:RUmeEt8OlEVeSzKRrO5Vl5qVWCtUwx4j9uivGuRo5fw=
2725
github.com/goreleaser/nfpm/v2 v2.37.1/go.mod h1:q8+sZXFqn106/eGw+9V+I8+izFxZ/sJjrhwmEUxXhUg=
2826
github.com/invopop/jsonschema v0.12.0 h1:6ovsNSuvn9wEQVOyc72aycBMVQFKz7cPdMJn10CvzRI=

0 commit comments

Comments
 (0)
Please sign in to comment.