Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion recipe/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,11 @@ package:
source:
url: https://github.com/conda/{{ name }}/archive/refs/tags/{{ version }}.tar.gz
sha256: 1658015711ce4ae5c4a1888ee3f5c20e71f87be646c704927a19b4260c6ecfc8
patches:
- patches/gh4867.patch

build:
number: 0
number: 1
script: {{ PYTHON }} -m pip install . --no-deps --no-build-isolation -vv
entry_points:
- conda-build = conda_build.cli.main_build:main
Expand Down
218 changes: 218 additions & 0 deletions recipe/patches/gh4867.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,218 @@
From 5c99adf66d761205849b77e7fc010b85a7a60377 Mon Sep 17 00:00:00 2001
From: Finn Womack <[email protected]>
Date: Thu, 20 Apr 2023 13:58:00 -0700
Subject: [PATCH 01/10] add warning and return empty string

---
conda_build/windows.py | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/conda_build/windows.py b/conda_build/windows.py
index ffaad7f5ca..42ad8af748 100644
--- a/conda_build/windows.py
+++ b/conda_build/windows.py
@@ -108,6 +108,11 @@ def msvc_env_cmd(bits, config, override=None):
"If this recipe does not use a compiler, this message is safe to ignore. "
"Otherwise, use {{compiler('<language>')}} jinja2 in requirements/build."
)
+ if bit == "arm64":
+ log.warn(
+ "The legacy MSVC compiler setup does not support arm64 builds. "
+ )
+ return ""
if override:
log.warn(
"msvc_compiler key in meta.yaml is deprecated. Use the new"

From 9c70251111290812cadb321b38dee5b8a95f783e Mon Sep 17 00:00:00 2001
From: Finn Womack <[email protected]>
Date: Tue, 25 Apr 2023 12:59:35 -0700
Subject: [PATCH 02/10] Run tests in branch

---
.github/workflows/tests.yml | 1 +
1 file changed, 1 insertion(+)

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index da7637be2a..c681fdc56e 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -10,6 +10,7 @@ on:
branches:
- main
- feature/**
+ - msvc-env-cmd

# NOTE: github.event context is pull_request payload:
# https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#pull_request

From 9fdb425c56d28ecbbcbbc1a1a32843b64eaa4a5d Mon Sep 17 00:00:00 2001
From: Finn Womack <[email protected]>
Date: Tue, 25 Apr 2023 15:25:47 -0700
Subject: [PATCH 03/10] Update windows.py

bit -> bits
---
conda_build/windows.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/conda_build/windows.py b/conda_build/windows.py
index 42ad8af748..700f5c28ff 100644
--- a/conda_build/windows.py
+++ b/conda_build/windows.py
@@ -108,7 +108,7 @@ def msvc_env_cmd(bits, config, override=None):
"If this recipe does not use a compiler, this message is safe to ignore. "
"Otherwise, use {{compiler('<language>')}} jinja2 in requirements/build."
)
- if bit == "arm64":
+ if bits == "arm64":
log.warn(
"The legacy MSVC compiler setup does not support arm64 builds. "
)

From a68f0ac5a748e2d0a587f06bbacaf048786f1d67 Mon Sep 17 00:00:00 2001
From: Finn Womack <[email protected]>
Date: Wed, 3 May 2023 11:00:08 -0700
Subject: [PATCH 04/10] remove added branch

---
.github/workflows/tests.yml | 1 -
1 file changed, 1 deletion(-)

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 7fe6ffc981..19062be01a 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -11,7 +11,6 @@ on:
- main
- feature/**
- '[0-9].*.x' # e.g., 3.24.x
- - msvc-env-cmd

# NOTE: github.event context is pull_request payload:
# https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#pull_request

From 176f793f7149ef69ba5f7aa91ebdef8f681442d0 Mon Sep 17 00:00:00 2001
From: Finn Womack <[email protected]>
Date: Thu, 4 May 2023 12:32:20 -0700
Subject: [PATCH 05/10] Create news file

---
news/made-msvc_env_cmd-no-op-for-arm64 | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
create mode 100644 news/made-msvc_env_cmd-no-op-for-arm64

diff --git a/news/made-msvc_env_cmd-no-op-for-arm64 b/news/made-msvc_env_cmd-no-op-for-arm64
new file mode 100644
index 0000000000..a20af0a2ff
--- /dev/null
+++ b/news/made-msvc_env_cmd-no-op-for-arm64
@@ -0,0 +1,19 @@
+### Enhancements
+
+* <news item>
+
+### Bug fixes
+
+* <news item>
+
+### Deprecations
+
+* <news item>
+
+### Docs
+
+* <news item>
+
+### Other
+
+* Added a check to print an additional warning and return an empty string when bits is "arm64" in msvc_env_cmd

From 3a9f4d5890f2f937e1a3513b2f7b97760439e6e3 Mon Sep 17 00:00:00 2001
From: Finn Womack <[email protected]>
Date: Thu, 4 May 2023 12:33:22 -0700
Subject: [PATCH 06/10] Changed news file name

---
..._env_cmd-no-op-for-arm64 => made-msvc-env-cmd-no-op-for-arm64} | 0
1 file changed, 0 insertions(+), 0 deletions(-)
rename news/{made-msvc_env_cmd-no-op-for-arm64 => made-msvc-env-cmd-no-op-for-arm64} (100%)

diff --git a/news/made-msvc_env_cmd-no-op-for-arm64 b/news/made-msvc-env-cmd-no-op-for-arm64
similarity index 100%
rename from news/made-msvc_env_cmd-no-op-for-arm64
rename to news/made-msvc-env-cmd-no-op-for-arm64

From 69b231c09ab962d9be1aa4219e0bfdefb84eefce Mon Sep 17 00:00:00 2001
From: Finn Womack <[email protected]>
Date: Thu, 4 May 2023 12:34:16 -0700
Subject: [PATCH 07/10] Rename made-msvc-env-cmd-no-op-for-arm64 to
arm64-msvc-env-cmd

---
news/{made-msvc-env-cmd-no-op-for-arm64 => arm64-msvc-env-cmd} | 0
1 file changed, 0 insertions(+), 0 deletions(-)
rename news/{made-msvc-env-cmd-no-op-for-arm64 => arm64-msvc-env-cmd} (100%)

diff --git a/news/made-msvc-env-cmd-no-op-for-arm64 b/news/arm64-msvc-env-cmd
similarity index 100%
rename from news/made-msvc-env-cmd-no-op-for-arm64
rename to news/arm64-msvc-env-cmd

From 1b46fa62272bc67d53454e78f25bef90f33fc9db Mon Sep 17 00:00:00 2001
From: Finn Womack <[email protected]>
Date: Thu, 4 May 2023 12:34:54 -0700
Subject: [PATCH 08/10] Rename arm64-msvc-env-cmd to arm64-msvc-env-cmd-no-op

---
news/{arm64-msvc-env-cmd => arm64-msvc-env-cmd-no-op} | 0
1 file changed, 0 insertions(+), 0 deletions(-)
rename news/{arm64-msvc-env-cmd => arm64-msvc-env-cmd-no-op} (100%)

diff --git a/news/arm64-msvc-env-cmd b/news/arm64-msvc-env-cmd-no-op
similarity index 100%
rename from news/arm64-msvc-env-cmd
rename to news/arm64-msvc-env-cmd-no-op

From 37f68ea8367e16e4768520d71ab9ed7dc6989b2d Mon Sep 17 00:00:00 2001
From: Finn Womack <[email protected]>
Date: Thu, 4 May 2023 13:13:38 -0700
Subject: [PATCH 09/10] Rename arm64-msvc-env-cmd-no-op to
4867-arm64-msvc-env-cmd-no-op

---
news/{arm64-msvc-env-cmd-no-op => 4867-arm64-msvc-env-cmd-no-op} | 0
1 file changed, 0 insertions(+), 0 deletions(-)
rename news/{arm64-msvc-env-cmd-no-op => 4867-arm64-msvc-env-cmd-no-op} (100%)

diff --git a/news/arm64-msvc-env-cmd-no-op b/news/4867-arm64-msvc-env-cmd-no-op
similarity index 100%
rename from news/arm64-msvc-env-cmd-no-op
rename to news/4867-arm64-msvc-env-cmd-no-op

From b2c65e4bd24630a7a79664f42203782904f0813b Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
<66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Tue, 9 May 2023 11:00:52 +0000
Subject: [PATCH 10/10] [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
---
conda_build/windows.py | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/conda_build/windows.py b/conda_build/windows.py
index 700f5c28ff..c2497580f0 100644
--- a/conda_build/windows.py
+++ b/conda_build/windows.py
@@ -109,9 +109,7 @@ def msvc_env_cmd(bits, config, override=None):
"Otherwise, use {{compiler('<language>')}} jinja2 in requirements/build."
)
if bits == "arm64":
- log.warn(
- "The legacy MSVC compiler setup does not support arm64 builds. "
- )
+ log.warn("The legacy MSVC compiler setup does not support arm64 builds. ")
return ""
if override:
log.warn(