Skip to content

Commit

Permalink
Replacing versions in urls should not include 'v' prefix. (#143)
Browse files Browse the repository at this point in the history
Our releases are not prefixed with `v` like the upstream distribution. 

So we need ensure we don't attempt to download our assets from releases
prefixed with 'v'`.
  • Loading branch information
Mpdreamz authored Aug 30, 2024
1 parent 8d8d3ba commit 0f9bb63
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions build/scripts/Packaging.fs
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ let stageInstallationBashScript () =
(File.ReadAllText patchScript.FullName)
.Replace("/open-telemetry/opentelemetry-dotnet-instrumentation/", "/elastic/elastic-otel-dotnet/")
.Replace("opentelemetry-dotnet-instrumentation", "elastic-dotnet-instrumentation")
.Replace("v" + Software.OpenTelemetryAutoInstrumentationVersion.AsString, "v" + Software.Version.NormalizeToShorter())
.Replace("v" + Software.OpenTelemetryAutoInstrumentationVersion.AsString, Software.Version.NormalizeToShorter())

let elasticInstall = distroFile installScript
File.WriteAllText(elasticInstall.FullName, contents)
Expand All @@ -167,7 +167,7 @@ let stageInstallationPsScript () =
]
|> String.concat "\r\n "
)
.Replace("v" + Software.OpenTelemetryAutoInstrumentationVersion.AsString, "v" + Software.Version.NormalizeToShorter())
.Replace("v" + Software.OpenTelemetryAutoInstrumentationVersion.AsString, Software.Version.NormalizeToShorter())
let elasticInstall = distroFile installScript
//ensure we write our new module name
File.WriteAllText(elasticInstall.FullName.Replace("elastic.DotNet.Auto", "Elastic.OpenTelemetry.DotNet"), contents);
Expand Down

0 comments on commit 0f9bb63

Please sign in to comment.