Skip to content

Commit

Permalink
Installer build steps aren't cross builds
Browse files Browse the repository at this point in the history
  • Loading branch information
jkoritzinsky committed Mar 21, 2024
1 parent 471f0eb commit ab65443
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 1 deletion.
3 changes: 2 additions & 1 deletion eng/pipelines/common/templates/global-build-step.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,12 @@ parameters:
useContinueOnErrorDuringBuild: false
shouldContinueOnError: false
archParameter: $(_archParameter)
crossArg: $(crossArg)
displayName: Build product
container: ''

steps:
- script: $(Build.SourcesDirectory)$(dir)build$(scriptExt) -ci ${{ parameters.archParameter }} $(_osParameter) $(crossArg) ${{ parameters.buildArgs }} $(_officialBuildParameter) $(_buildDarwinFrameworksParameter) $(_overrideTestScriptWindowsCmdParameter)
- script: $(Build.SourcesDirectory)$(dir)build$(scriptExt) -ci ${{ parameters.archParameter }} $(_osParameter) ${{ parameters.crossArg }} ${{ parameters.buildArgs }} $(_officialBuildParameter) $(_buildDarwinFrameworksParameter) $(_overrideTestScriptWindowsCmdParameter)
displayName: ${{ parameters.displayName }}
${{ if eq(parameters.useContinueOnErrorDuringBuild, true) }}:
continueOnError: ${{ parameters.shouldContinueOnError }}
Expand Down
2 changes: 2 additions & 0 deletions eng/pipelines/installer/steps/build-linux-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@ steps:
buildArgs: -s packs.installers ${{ parameters.packagingArgs }} /bl:artifacts/log/$(_BuildConfig)/msbuild.${{ parameters.packageType }}.installers.binlog
container: ${{ parameters.target }}
displayName: Package Runtime Deps, Runtime, Framework Packs - ${{ parameters.packageType }} packages
# Even for cross-build targets, our installer build steps are not cross-builds
crossArg: ''
condition: ${{ parameters.condition }}

# Broken symbolic links break the SBOM processing
Expand Down
1 change: 1 addition & 0 deletions eng/pipelines/runtime.yml
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,7 @@ extends:
packageType: rpm
target: rpmpkg
packagingArgs: -c $(_BuildConfig) /p:BuildRpmPackage=true
condition: and(succeeded(), eq(variables.osSubgroup, ''))
condition: >-
or(
eq(dependencies.evaluate_paths.outputs['SetPathVars_non_mono_and_wasm.containsChange'], true),
Expand Down

0 comments on commit ab65443

Please sign in to comment.