Skip to content

Commit

Permalink
Fix incorrect inclusion of visual studio for unix builds
Browse files Browse the repository at this point in the history
Add cmake to build requirements
Fix typo
Add clang c++ library to dependencies for C models linked with C++
  • Loading branch information
langmm committed May 24, 2024
1 parent fe8d3a8 commit 478a855
Show file tree
Hide file tree
Showing 5 changed files with 20 additions and 7 deletions.
3 changes: 2 additions & 1 deletion recipe/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -45,14 +45,15 @@ build:

requirements:
build:
- cmake >= 3.16
- cross-python_{{ target_platform }} # [build_platform != target_platform]
- hatch-fancy-pypi-readme
- numpy >=1.13.0
- python # [build_platform != target_platform]
- python-build
- scikit-build-core
- setuptools_scm
- vs2019_win-64
- vs2019_win-64 # [win]
- {{ compiler('c') }}
- {{ compiler('cxx') }}
host:
Expand Down
10 changes: 10 additions & 0 deletions utils/requirements/requirements.json
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,16 @@
},
{
"vs2019_win-64": {
"os": "win",
"method": "conda_recipe",
"flags": {
"build": true,
"build_only": true
}
}
},
{
"cmake >= 3.16": {
"method": "conda_recipe",
"flags": {
"build": true,
Expand Down
11 changes: 6 additions & 5 deletions utils/requirements/requirements.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,15 +26,16 @@ general:
build: True
build_only: True
- vs2019_win-64:
os: win
method: conda_recipe
flags:
build: True
build_only: True
- cmake >= 3.16:
method: conda_recipe
flags:
build: True
build_only: True
# - cmake >= 3.16:
# method: conda_recipe
# flags:
# build: True
# build_only: True
- deprecation
- flask
- git:
Expand Down
1 change: 1 addition & 0 deletions yggdrasil/drivers/CPPModelDriver.py
Original file line number Diff line number Diff line change
Expand Up @@ -188,6 +188,7 @@ class ClangPPLinker(ClangLinker):
default_executable = ClangPPCompiler.default_executable
toolset = ClangPPCompiler.toolset
version_regex = LDLinker.version_regex + ClangPPCompiler.version_regex
standard_library = ClangPPCompiler.standard_library


class MSVCPPLinker(MSVCLinker):
Expand Down
2 changes: 1 addition & 1 deletion yggdrasil/drivers/CompiledModelDriver.py
Original file line number Diff line number Diff line change
Expand Up @@ -3736,7 +3736,7 @@ def _search_brute(self, fname, libtype=None, verbose=False,
if fname_base.startswith('lib'):
fname_try.append(fname_base[3:])
else:
fname_try.append('lib' + fname_try)
fname_try.append('lib' + fname_base)
search_list = self.tool(libtype).get_search_path(
libtype=libtype, cfg=self.cfg, **kwargs)
for fname_base in fname_try:
Expand Down

0 comments on commit 478a855

Please sign in to comment.