Skip to content

Commit

Permalink
Fix pypi mirror job (#8756)
Browse files Browse the repository at this point in the history
  • Loading branch information
scbedd authored Aug 1, 2024
1 parent 286da43 commit 079cb9f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
5 changes: 2 additions & 3 deletions eng/pipelines/mirror-pypi-to-dev-feed.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,7 @@ jobs:
displayName: Install Requirements
- pwsh: |
$configurations = '${{ convertToJson(parameters.PackageSpecifiers) }}' -replace '\\', '/' -replace '"', '`"'
python $(Build.SourcesDirectory)/eng/scripts/download_targeted_packages.py $configurations --dest-dir='$(Build.ArtifactStagingDirectory)'
python $(Build.SourcesDirectory)/eng/scripts/download_targeted_packages.py ${{ parameters.PackageSpecifiers }} --dest-dir='$(Build.ArtifactStagingDirectory)'
displayName: 'Download Targeted Packages'
- task: TwineAuthenticate@0
Expand All @@ -57,7 +56,7 @@ jobs:
$results = Get-ChildItem -Force -Path $(Build.ArtifactStagingDirectory)
foreach ($file in $results) {
Write-Host "Uploading $file"
twine upload --repository "${{ parameters.DevFeed }}" --config-file $(PYPIRC_PATH) $($file.FullName)
}
displayName: 'Publish Packages to Dev Feed'
Expand Down
5 changes: 2 additions & 3 deletions eng/scripts/download_targeted_packages.py
Original file line number Diff line number Diff line change
Expand Up @@ -140,9 +140,8 @@ def download_dependencies(targeted_packages: Set[str], target_folder: str) -> No
)

args = parser.parse_args()

# powershell does some weird escaping. Leaving behind just the '`' in what would normally look like '`"`.
pkg_list = read_package_list(args.target_packages.replace("`", "\""))

pkg_list = args.target_packages.split(",")

assembled_package_list = set()

Expand Down

0 comments on commit 079cb9f

Please sign in to comment.