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

WIP: Add paratext #731

Closed
wants to merge 6 commits into from
Closed

WIP: Add paratext #731

wants to merge 6 commits into from

Conversation

danielfrg
Copy link
Member

@danielfrg danielfrg commented Jun 8, 2016

Adds paratext: https://github.com/wiseio/paratext Python 2 and linux for now

Right now there is not an official release. I'll ping upstream to see if we I can get a very initial release going.

@conda-forge-linter
Copy link

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 (recipes/paratext) and found it was in an excellent condition.

@danielfrg danielfrg changed the title Add paratext WIP: Add paratext Jun 8, 2016
@patricksnape
Copy link
Contributor

C++11 rules out Python < 3.5 on Windows (as a note). Would obviously prefer a solid release so thanks for requesting one 😄


test:
imports:
- paratext
Copy link
Contributor

Choose a reason for hiding this comment

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

Given that this may have a bit of a complicated build is there any short test we can run (e.g. run_test.py) to check that it can actually read files?.

Copy link
Member Author

Choose a reason for hiding this comment

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

That would be nice indeed. At the moment the library doesn't seem to have any tests in the repo. I could certainly a very simple write one and add it to the feedstock for now. I did read somewhere that thats probably not something we want in feedstocks since it should really be in the library. I believe the library will add more tests soon that we can call in later versions of the pkg.

@pelson
Copy link
Member

pelson commented Jul 8, 2016

On OSX this is failing with:

+ cd python/
+ /Users/travis/miniconda/envs/_build/bin/python setup.py install --prefix=/Users/travis/miniconda/envs/_build
/Users/travis/miniconda/envs/_build/bin/swig
0.1.1rc1
('running swig: ', ['swig', '-c++', '-python', '-I../src/', '-outdir', './', '../src/paratext_internal.i'])
../src/diagnostic/parse_and_sum.hpp:158: Warning 302: Identifier 'parse_token' redefined (ignored),
../src/diagnostic/parse_and_sum.hpp:129: Warning 302: previous definition of 'parse_token'.
/Users/travis/miniconda/envs/_build/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'include_package_data'
  warnings.warn(msg)
running install
running build
running config_cc
unifing config_cc, config, build_clib, build_ext, build commands --compiler options
running config_fc
unifing config_fc, config, build_clib, build_ext, build commands --fcompiler options
running build_src
build_src
building py_modules sources
building extension "_paratext_internal" sources
build_src: building npy-pkg config files
running build_py
creating build
creating build/lib.macosx-10.6-x86_64-2.7
copying paratext_internal.py -> build/lib.macosx-10.6-x86_64-2.7
creating build/lib.macosx-10.6-x86_64-2.7/paratext
copying paratext/__init__.py -> build/lib.macosx-10.6-x86_64-2.7/paratext
copying paratext/core.py -> build/lib.macosx-10.6-x86_64-2.7/paratext
copying paratext/helpers.py -> build/lib.macosx-10.6-x86_64-2.7/paratext
running build_ext
customize UnixCCompiler
customize UnixCCompiler using build_ext
customize UnixCCompiler
customize UnixCCompiler using build_ext
building '_paratext_internal' extension
compiling C++ sources
C compiler: g++ -DNDEBUG -g -fwrapv -O3 -Wall -arch x86_64
creating build/temp.macosx-10.6-x86_64-2.7
creating build/temp.macosx-10.6-x86_64-2.7/paratext-97a949a648f4f057e2b8764a6d2bb9be35cbfddf
creating build/temp.macosx-10.6-x86_64-2.7/paratext-97a949a648f4f057e2b8764a6d2bb9be35cbfddf/src
compile options: '-I../src/ -I/Users/travis/miniconda/envs/_build/lib/python2.7/site-packages/numpy/core/include -I/Users/travis/miniconda/envs/_build/include/python2.7 -c'
extra options: '-std=c++11 -Wall -Wextra -pthread -m64 -D_REENTRANT'
g++: ../src/paratext_internal.cpp
../src/paratext_internal.cpp:23:10: fatal error: 'type_traits' file not found
#include <type_traits>
         ^
1 error generated.
../src/paratext_internal.cpp:23:10: fatal error: 'type_traits' file not found
#include <type_traits>
         ^
1 error generated.
error: Command "g++ -DNDEBUG -g -fwrapv -O3 -Wall -arch x86_64 -I../src/ -I/Users/travis/miniconda/envs/_build/lib/python2.7/site-packages/numpy/core/include -I/Users/travis/miniconda/envs/_build/include/python2.7 -c ../src/paratext_internal.cpp -o build/temp.macosx-10.6-x86_64-2.7/paratext-97a949a648f4f057e2b8764a6d2bb9be35cbfddf/src/paratext_internal.o -std=c++11 -Wall -Wextra -pthread -m64 -D_REENTRANT" failed with exit status 1
Command failed: /bin/bash -x -e /Users/travis/build/conda-forge/staged-recipes/recipes/paratext/build.sh

@danielfrg
Copy link
Member Author

I am gonna close this one and will reopen if there is action on the source.

@danielfrg danielfrg closed this Sep 5, 2016
@danielfrg danielfrg deleted the paratext branch September 5, 2016 02:28
@arita37
Copy link

arita37 commented Nov 11, 2016

Just wondering, the issues on building the conda forder for paratet ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

6 participants