Skip to content

Conversation

@thewtex
Copy link
Member

@thewtex thewtex commented May 14, 2020

In CI, this is unnecessary now because every Python package version is
generated in a clean virtual machine instance. On some packages that use
CMake FetchContent, the cleanup step can fail with:

Adding 'itk_elastix-0.6.0.dist-info/RECORD'
removing _skbuild\win-amd64-3.5\setuptools\bdist.win-amd64\wheel
C:\P\IPP\venv-35-x64\lib\site-packages\wheel\pep425tags.py:82: RuntimeWarning: Config variable 'Py_DEBUG' is unset, Python ABI tag may be incorrect
warn=(impl == 'cp')):
C:\P\IPP\venv-35-x64\lib\site-packages\wheel\pep425tags.py:87: RuntimeWarning: Config variable 'WITH_PYMALLOC' is unset, Python ABI tag may be incorrect
sys.version_info < (3, 8)))
running clean
removing '_skbuild\win-amd64-3.5\cmake-install'
removing '_skbuild\win-amd64-3.5\cmake-build'
error: [WinError 5] Access is denied: '_skbuild\win-amd64-3.5\cmake-build\_deps\elx-src\.git\objects\pack\pack-fe0f18355ee7166ea63b5deefec6fde96d8691fc.idx'
SCRIPT_DIR: C:\P\IPP\scripts
ROOT_DIR: D:\a\im

In CI, this is unnecessary now because every Python package version is
generated in a clean virtual machine instance. On some packages that use
CMake FetchContent, the cleanup step can fail with:

  Adding 'itk_elastix-0.6.0.dist-info/RECORD'
  removing _skbuild\win-amd64-3.5\setuptools\bdist.win-amd64\wheel
  C:\P\IPP\venv-35-x64\lib\site-packages\wheel\pep425tags.py:82: RuntimeWarning: Config variable 'Py_DEBUG' is unset, Python ABI tag may be incorrect
    warn=(impl == 'cp')):
  C:\P\IPP\venv-35-x64\lib\site-packages\wheel\pep425tags.py:87: RuntimeWarning: Config variable 'WITH_PYMALLOC' is unset, Python ABI tag may be incorrect
    sys.version_info < (3, 8))) \
  running clean
  removing '_skbuild\win-amd64-3.5\cmake-install'
  removing '_skbuild\win-amd64-3.5\cmake-build'
  error: [WinError 5] Access is denied: '_skbuild\\win-amd64-3.5\\cmake-build\\_deps\\elx-src\\.git\\objects\\pack\\pack-fe0f18355ee7166ea63b5deefec6fde96d8691fc.idx'
  SCRIPT_DIR: C:\P\IPP\scripts
  ROOT_DIR: D:\a\im
@thewtex thewtex requested review from dzenanz and hjmjohnson May 14, 2020 12:53
@thewtex thewtex merged commit 843c0ec into InsightSoftwareConsortium:master May 14, 2020
@thewtex thewtex deleted the windows-no-cleanup branch May 14, 2020 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants