Add clangdev recipe #1481

inducer · 2016-09-04T23:39:28Z

No description provided.

conda-forge-linter · 2016-09-04T23:39:35Z

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipes/clangdev) and found some lint.

Here's what I've got...

For recipes/clangdev:

The recipe could do with some maintainers listed in the extra/recipe-maintainers section.
Selectors are suggested to take a <two spaces>#<one space>[<expression>] form.

conda-forge-linter · 2016-09-04T23:41:13Z

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipes/clangdev) and found some lint.

Here's what I've got...

For recipes/clangdev:

Selectors are suggested to take a <two spaces>#<one space>[<expression>] form.

conda-forge-linter · 2016-09-04T23:42:33Z

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

inducer · 2016-09-05T00:06:50Z

This requires llvmdev, but that's in the numba channel. I could of course copy this:

https://github.com/numba/llvmlite/tree/master/conda-recipes/llvmdev

~~But that seems like the wrong way to go about it.~~ You know what: why not.

inducer · 2016-09-05T00:27:05Z

FWIW, with #1481, #1479, #1482, I'm building towards a recipe for pocl, and based on that pyopencl.

ocefpaf · 2016-09-05T11:16:19Z

recipes/clangdev/build.sh

+  -DBUILD_SHARED_LIBS=ON \
+  -DCMAKE_INSTALL_PREFIX=$PREFIX \
+  ..
+cmake --build . -- -j 8


Don't trust the 8 here as we never know how many cores the CIs will provide. If you are feeling brave you can try the CPU_COUNT env var which should use the last known number of cores for each CI.

Done, although oversubscribing is not usually an issue in my experience--i.e. 8 is fine even for single-core, assuming there's sufficient RAM.

Made the same change in llvmdev, #1482.

conda-forge-linter · 2016-09-05T17:20:56Z

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

inducer · 2016-09-05T17:34:33Z

~~FWIW, this is stalled until #1482 is merged.~~

conda-forge-linter · 2016-10-08T17:39:17Z

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

inducer · 2016-10-09T17:49:00Z

@jakirkham The Windows build here:

https://ci.appveyor.com/project/conda-forge/staged-recipes/build/1.0.7739/job/ciaib6t66ims0kl9#L268

seems to say that llvmdev conflicts with toolchain on account of the vc14 feature. I'm currently working around that by depending on toolchain # [not win], since toolchain appears to be a no-op there. Is that the correct fix?

conda-forge-linter · 2016-10-09T18:18:13Z

Hi! This is the friendly automated conda-forge-linting service.

I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.

Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

inducer · 2016-11-05T16:06:08Z

Hum, no build is created on AppVeyor--why?

yesimon · 2016-11-05T21:13:04Z

Seems like some of the python scripts in llvm don't support python 3. There are also errors in the circleCI build for the same issue while compiling to .pyc. I think these are support scripts like clang-format.py that don't affect the basic functionality of clang.

inducer · 2016-11-05T22:41:05Z

Seems like some of the python scripts in llvm don't support python 3.

Yeah, it's the same for llvmdev. But, as you say, there doesn't seem to be an overall adverse effect on the build.

conda-forge-linter · 2016-12-01T20:20:25Z

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

inducer · 2016-12-05T22:58:36Z

@conda-forge/core Something that I don't understand is happening in my Circle/LInux build. I get what seems like a perfectly successful Python 3.5 build, and then another build starts, perhaps for Python 3.6, which is missing CMake...? I'm not sure what to make of that, and I'd appreciate any hints you can provide.

ocefpaf · 2016-12-05T23:01:56Z

It looks like Python 3.6 leaked here. I fixed the label a not long ago and hopefully that should fix it. I restarted the CI. Fingers crossed.

inducer · 2016-12-05T23:17:04Z

Thanks!

inducer · 2016-12-06T00:27:01Z

Hum, that died again, complaining that it can't find cmake. But cmake is in the dependencies, and it finds it on the first build. No Python 3.6 this time though. Any clues?

ocefpaf · 2016-12-06T19:55:51Z

It seems that it did find cmake but not the expected version on Travis-CI:

 CMake 3.4.3 or higher is required.  You are running version 3.2.2

The weird thing is that cmake 3.6.2 is installed 😕

On CircleCI the same cmake is installed but a different error is issued:

/staged-recipes/build_artefacts/work/cfe-3.9.0.src/conda_build.sh: line 6: cmake: command not found

Did you try that configuration in the docker image?

inducer · 2016-12-07T06:28:46Z

Thanks for helping!

Did you try that configuration in the docker image?

What configuration? How?

ocefpaf · 2016-12-07T09:41:38Z

recipes/clangdev/meta.yaml

+
+    # "compiling .pyc files" fails without this:
+    - python                                # [not win]
+    - python {{ environ['PY_VER'] + '*' }}  # [win]


Let the skip section above that care of this for you. use only python here and remove the pre-selectors. you can also add vc 14 # [win] to ensure the right vc though.

Agreed combining with python should be fine.

ocefpaf · 2016-12-07T09:44:27Z

What configuration?

I meant the failing item of the matrix there. It seems to be CONDA_PY=35.

How?

We do not have docs on the use of the docker image yet but here is the gist of it:

docker pull condaforge/linux-anvil  # download the image
docker run -it --rm -v $(pwd):/home/ condaforge/linux-anvil /bin/sh  # "enter" the image for a staged build
cd home  # the directory we mounted in the image (you need to be in the recipe directory in the command above)
export CONDA_PY=35  # the python version that is failing in your recipe
conda build .

inducer · 2016-12-07T22:08:19Z

cc @mattwala (Just in case being sick this afternoon is not enough to keep you occupied.)

This reverts commit 88b0c70.

jakirkham · 2017-01-18T01:30:45Z

Now that the XCode image has been bumped. Could we please merge/rebase this with master?

…ngdev

inducer · 2017-01-18T10:24:21Z

Done.

inducer · 2017-01-19T06:20:30Z

Looks like all the tests passed.

jakirkham · 2017-01-19T06:55:59Z

Thanks @inducer. Let's get it into a feedstock. 😀

inducer added 3 commits September 4, 2016 18:34

Initial clangdev recipe

7f90a44

Reset build number

26270c5

Peg llvm version

fbd8247

Respect linter feedback

3dcd322

Fix missing space in selector comment

adfe4e1

inducer mentioned this pull request Sep 5, 2016

Add llvmdev #1482

Merged

ocefpaf reviewed Sep 5, 2016
View reviewed changes

inducer added 3 commits September 5, 2016 09:37

make -j 8 -> make -j

1b6471b

Rename as clangdev38, dependon llvmdev38

d32a822

Catch empty CPU_COUNT

c93bb29

inducer changed the title ~~Add clangdev recipe~~ Add clangdev38 recipe Sep 5, 2016

Incorporate wisdom from llvmdev

b51e458

inducer changed the title ~~Add clangdev38 recipe~~ Add clangdev recipe [WIP] Oct 8, 2016

Update skip so that Unix builds actually happen

e0e488a

inducer closed this Oct 8, 2016

inducer reopened this Oct 8, 2016

Don't include 'toolchain' on windows

7d69d5e

Use xcode8 to avoid Apple build crash

499e25e

Merge staged-recipes master

b82ec4d

inducer closed this Nov 5, 2016

inducer reopened this Nov 5, 2016

Do a compile (no link) test on Windows as well

8c99108

Add @mattwala as maintainer

41c66fa

inducer added 2 commits December 4, 2016 19:49

Add versioned cmake dependency

cea29e8

Keep cmake installed for test

b106abb

ocefpaf reviewed Dec 7, 2016

View reviewed changes

Revert "Bump version to 3.9"

dcd16f2

This reverts commit 88b0c70.

isuruf mentioned this pull request Jan 10, 2017

Add windows support conda-forge/doxygen-feedstock#1

Merged

inducer added 3 commits January 18, 2017 04:19

Merge branch 'master' into clangdev

24fa8d4

Remove windows-specific Python version selector

b1d1f43

Merge branch 'clangdev' of github.com:inducer/staged-recipes into cla…

d35d6fe

…ngdev

jakirkham merged commit f27a0a4 into conda-forge:master Jan 19, 2017

jakirkham mentioned this pull request Nov 2, 2017

Migration to conda-build 3 and new compilers from Anaconda conda-forge/conda-forge.github.io#460

Closed

6 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add clangdev recipe #1481

Add clangdev recipe #1481

inducer commented Sep 4, 2016

conda-forge-linter commented Sep 4, 2016

conda-forge-linter commented Sep 4, 2016

conda-forge-linter commented Sep 4, 2016

inducer commented Sep 5, 2016 •

edited

Loading

inducer commented Sep 5, 2016

ocefpaf Sep 5, 2016

inducer Sep 5, 2016

conda-forge-linter commented Sep 5, 2016

inducer commented Sep 5, 2016 •

edited

Loading

conda-forge-linter commented Oct 8, 2016

inducer commented Oct 9, 2016

conda-forge-linter commented Oct 9, 2016

inducer commented Nov 5, 2016

yesimon commented Nov 5, 2016

inducer commented Nov 5, 2016

conda-forge-linter commented Dec 1, 2016

inducer commented Dec 5, 2016

ocefpaf commented Dec 5, 2016

inducer commented Dec 5, 2016

inducer commented Dec 6, 2016

ocefpaf commented Dec 6, 2016

inducer commented Dec 7, 2016

ocefpaf Dec 7, 2016

jakirkham Jan 13, 2017

ocefpaf commented Dec 7, 2016

inducer commented Dec 7, 2016

jakirkham commented Jan 18, 2017

inducer commented Jan 18, 2017

inducer commented Jan 19, 2017

jakirkham commented Jan 19, 2017

Add clangdev recipe #1481

Add clangdev recipe #1481

Conversation

inducer commented Sep 4, 2016

conda-forge-linter commented Sep 4, 2016

conda-forge-linter commented Sep 4, 2016

conda-forge-linter commented Sep 4, 2016

inducer commented Sep 5, 2016 • edited Loading

inducer commented Sep 5, 2016

ocefpaf Sep 5, 2016

Choose a reason for hiding this comment

inducer Sep 5, 2016

Choose a reason for hiding this comment

conda-forge-linter commented Sep 5, 2016

inducer commented Sep 5, 2016 • edited Loading

conda-forge-linter commented Oct 8, 2016

inducer commented Oct 9, 2016

conda-forge-linter commented Oct 9, 2016

inducer commented Nov 5, 2016

yesimon commented Nov 5, 2016

inducer commented Nov 5, 2016

conda-forge-linter commented Dec 1, 2016

inducer commented Dec 5, 2016

ocefpaf commented Dec 5, 2016

inducer commented Dec 5, 2016

inducer commented Dec 6, 2016

ocefpaf commented Dec 6, 2016

inducer commented Dec 7, 2016

ocefpaf Dec 7, 2016

Choose a reason for hiding this comment

jakirkham Jan 13, 2017

Choose a reason for hiding this comment

ocefpaf commented Dec 7, 2016

inducer commented Dec 7, 2016

jakirkham commented Jan 18, 2017

inducer commented Jan 18, 2017

inducer commented Jan 19, 2017

jakirkham commented Jan 19, 2017

inducer commented Sep 5, 2016 •

edited

Loading

inducer commented Sep 5, 2016 •

edited

Loading