Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

V0.6.2 #4

Merged
merged 26 commits into from
Mar 20, 2024
Merged
Show file tree
Hide file tree
Changes from 18 commits
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
22 changes: 0 additions & 22 deletions .ci_support/linux_ppc64le_python3.10.____cpython.yaml

This file was deleted.

22 changes: 0 additions & 22 deletions .ci_support/linux_ppc64le_python3.11.____cpython.yaml

This file was deleted.

22 changes: 0 additions & 22 deletions .ci_support/linux_ppc64le_python3.8.____73_pypy.yaml

This file was deleted.

22 changes: 0 additions & 22 deletions .ci_support/linux_ppc64le_python3.8.____cpython.yaml

This file was deleted.

22 changes: 0 additions & 22 deletions .ci_support/linux_ppc64le_python3.9.____73_pypy.yaml

This file was deleted.

22 changes: 0 additions & 22 deletions .ci_support/linux_ppc64le_python3.9.____cpython.yaml

This file was deleted.

3 changes: 3 additions & 0 deletions abs.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
channels:
- https://staging.continuum.io/prefect/fs/rust-feedstock/pr7/1481b8e
- https://staging.continuum.io/prefect/fs/rust-activation-feedstock/pr6/b2372ac
cbouss marked this conversation as resolved.
Show resolved Hide resolved
15 changes: 15 additions & 0 deletions recipe/conda_build_config.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
rust_compiler:
- rust
rust_compiler_version:
- 1.76.0
# use {{ compiler('rust-gnu') }} when requiring a build using the m2w64-toolchain
rust_gnu_compiler: # [win]
- rust-gnu # [win]
rust_gnu_compiler_version: # [win]
- 1.76.0 # [win]
MACOSX_DEPLOYMENT_TARGET: # [osx and x86_64]
- "10.12" # [osx and x86_64]
rust_compiler_version:
- 1.76.0

Choose a reason for hiding this comment

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

Suggested change
rust_compiler_version:
- 1.76.0

This is already defined at line 3 and 4.

CONDA_BUILD_SYSROOT:
- /opt/MacOSX10.12.sdk # [osx and x86_64]
13 changes: 9 additions & 4 deletions recipe/meta.yaml
Original file line number Diff line number Diff line change
@@ -1,15 +1,18 @@
{% set name = "y-py" %}
{% set version = "0.5.9" %}
{% set version = "0.6.2" %}

package:
name: {{ name }}
version: {{ version }}

source:
url: https://pypi.io/packages/source/y/y_py/y_py-{{ version }}.tar.gz
sha256: 50cfa0532bcee27edb8c64743b49570e28bb76a00cd384ead1d84b6f052d9368
sha256: 4757a82a50406a0b3a333aa0122019a331bd6f16e49fed67dca423f928b3fd4d

build:
run_exports: # [osx and x86_64]
strong_constrains: # [osx and x86_64]
- __osx >={{ MACOSX_DEPLOYMENT_TARGET|default("10.12") }} # [osx and x86_64]

Choose a reason for hiding this comment

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

Do we really need this? You'll get these for free if you use your latest builds for rust and rust-activation if I'm not mistaken.

Copy link

@Jrice1317 Jrice1317 Mar 18, 2024

Choose a reason for hiding this comment

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

I was just about to ask if this was required as well

number: 0
skip: True # [s390x]
script:
Expand All @@ -21,18 +24,20 @@ build:
requirements:
build:
- python
- maturin >=0.13,<0.14
- maturin >=1.2.3,<2
- {{ compiler('c') }}
- {{ compiler('rust') }}
- cargo-bundle-licenses
host:
- maturin =0.13.7
- maturin 1.3.1
- pip
- python
- wheel
- setuptools

Choose a reason for hiding this comment

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

Suggested change
- wheel
- setuptools

setuptools and wheel are not needed when using maturin.

run:
# this is needed otherwise import y_py in tests fails with ModuleNotFoundError
- python
- __osx >=10.12 # [osx and x86_64]

Choose a reason for hiding this comment

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

Is this needed if the cbc.yaml has 10.12 listed for osx?


test:
source_files:
Expand Down