Skip to content

Switch build to splayed layout - #83

Merged
jakirkham merged 7 commits into
conda-forge:mainfrom
conda-forge-admin:conda_forge_admin_82
Jun 26, 2024
Merged

jakirkham merged 7 commits into
conda-forge:mainfrom
conda-forge-admin:conda_forge_admin_82

Conversation

@conda-forge-admin

@conda-forge-admin conda-forge-admin commented Jun 26, 2024 •

Copy link
Copy Markdown
Contributor

Fixes #82
Closes #74
Closes NVIDIA/cuda-python#46

@conda-forge-webservices

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jakirkham
jakirkham marked this pull request as draft June 26, 2024 18:07
@conda-forge-webservices

This comment was marked as resolved.

@jakirkham
jakirkham force-pushed the conda_forge_admin_82 branch 2 times, most recently from 7983c59 to d469436 Compare June 26, 2024 18:46
@conda-forge-webservices

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jakirkham
jakirkham force-pushed the conda_forge_admin_82 branch from d469436 to 8ee6bfe Compare June 26, 2024 18:49
@jakirkham

jakirkham commented Jun 26, 2024 •

Copy link
Copy Markdown
Member

GitHub autogenerated artifacts fail to checksum as something changed in the autogeneration process again

Here is what we see on CI (attached log for posterity):

Traceback (most recent call last):
  File "/opt/conda/bin/conda-build", line 11, in <module>
    sys.exit(execute())
  File "/opt/conda/lib/python3.10/site-packages/conda_build/cli/main_build.py", line 590, in execute
    api.build(
  File "/opt/conda/lib/python3.10/site-packages/conda_build/api.py", line 250, in build
    return build_tree(
  File "/opt/conda/lib/python3.10/site-packages/conda_build/build.py", line 3638, in build_tree
    packages_from_this = build(
  File "/opt/conda/lib/python3.10/site-packages/conda_build/build.py", line 2419, in build
    try_download(m, no_download_source=False, raise_error=True)
  File "/opt/conda/lib/python3.10/site-packages/conda_build/render.py", line 762, in try_download
    source.provide(metadata)
  File "/opt/conda/lib/python3.10/site-packages/conda_build/source.py", line 1035, in provide
    unpack(
  File "/opt/conda/lib/python3.10/site-packages/conda_build/source.py", line 169, in unpack
    src_path, unhashed_fn = download_to_cache(
  File "/opt/conda/lib/python3.10/site-packages/conda_build/source.py", line 125, in download_to_cache
    raise RuntimeError(
RuntimeError: SHA256 mismatch: 'eb6d1b2cd881de0c14963ddb9cb7f3ec5df1b31370133267678265e31fb743b2' != 'ee7e3df43da7d924207e03efa4d5e5466801510b3e1f0377d425c6e6a8d823bb'

Edit: Raising this upstream ( NVIDIA/cuda-python#71 )

@jakirkham jakirkham changed the title MNT: rerender Switch build to splayed layout Jun 26, 2024
@jakirkham
jakirkham marked this pull request as ready for review June 26, 2024 19:33
@jakirkham

Copy link
Copy Markdown
Member

Think this is working as we can see lines like this on CI:

 Parsing headers in "['/home/conda/feedstock_root/build_artifacts/cuda-python_1719430415930/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold/targets/x86_64-linux/include', '/home/conda/feedstock_root/build_artifacts/cuda-python_1719430415930/_build_env/targets/x86_64-linux/include']" (Caching False)

@vzhurba01 vzhurba01 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jakirkham
jakirkham merged commit 915031a into conda-forge:main Jun 26, 2024
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.

Configure build for splayed layout Drop environment variable workarounds in build Splayed layout support

3 participants