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

Lensfunpy #10199

Merged
merged 18 commits into from
Nov 20, 2019
14 changes: 14 additions & 0 deletions recipes/lensfunpy/0001_change_function_to_prevent_rebuild.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
diff --git a/setup.py b/setup.py
index e60c7f2..b4fa13c 100644
--- a/setup.py
+++ b/setup.py
@@ -209,7 +209,8 @@ package_data = {'lensfunpy': []}
# evil hack, check cmd line for relevant commands
# custom cmdclasses didn't work out in this case
cmdline = ''.join(sys.argv[1:])
-needsCompile = any(s in cmdline for s in ['install', 'bdist', 'build_ext', 'wheel', 'nosetests'])
+#needsCompile = any(s in cmdline for s in ['install', 'bdist', 'build_ext', 'wheel', 'nosetests'])
+needsCompile = False
if isWindows and needsCompile:
windows_lensfun_compile()
package_data['lensfunpy'].append('*.dll')
21 changes: 15 additions & 6 deletions recipes/lensfunpy/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,9 @@ package:

source:
url: "https://pypi.io/packages/source/{{ name[0] }}/{{ name }}/{{ name }}-{{ version }}.tar.gz"
sha256: d173e6fdbbeb83c10401ad7af276606e812960307c0d5dc58f03c3a39efd7746 #1.1.6 pypi

sha256: d173e6fdbbeb83c10401ad7af276606e812960307c0d5dc58f03c3a39efd7746
patches:
- 0001_change_function_to_prevent_rebuild.patch # [osx]

build:
number: 0
Expand All @@ -18,17 +19,25 @@ build:
requirements:
build:
- {{ compiler('cxx') }}
- lensfun

- lensfun <0.3.95
Copy link
Member

Choose a reason for hiding this comment

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

This should only be in host

host:
- pip
- python
- lensfun
- lensfun <0.3.95
- numpy
Copy link
Member

Choose a reason for hiding this comment

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


run:
- python

- cython
- nose
Copy link
Member

Choose a reason for hiding this comment

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

This is needed only for testing. No need at runtime.

- wheel >=0.25
Copy link
Member

Choose a reason for hiding this comment

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

Why do you need wheel at runtime?

- sphinx
- sphinx_rtd_theme
Copy link
Member

Choose a reason for hiding this comment

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

sphinx is for docs which is not needed at runtime

- scipy
- lensfun ==0.3.2
beckermr marked this conversation as resolved.
Show resolved Hide resolved


test:
imports:
- lensfunpy
Expand Down