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

added xonsh recipe #13

Merged
merged 4 commits into from
Nov 5, 2015
Merged

added xonsh recipe #13

merged 4 commits into from
Nov 5, 2015

Conversation

scopatz
Copy link
Member

@scopatz scopatz commented Oct 30, 2015

No description provided.

@pelson
Copy link
Member

pelson commented Oct 31, 2015

Strange that it can't find setup.py on Windows. I wonder if you have to use a fn entry for the source.
Also, I don't think the bld.bat is needed in this case (it would come from script anyway). Last but not least, would you mind adding an import test and a commands test?

@scopatz
Copy link
Member Author

scopatz commented Oct 31, 2015

Updated!

@scopatz
Copy link
Member Author

scopatz commented Oct 31, 2015

Oooh so how do we disable legacy Python for this recipe?

@pelson
Copy link
Member

pelson commented Nov 2, 2015

Oooh so how do we disable legacy Python for this recipe?

There are two ways, either use the skip section, or define a condition on build/run. The latter doesn't work very well though with conda-build at the moment (one to be improved). The skip section would look something like:

build:
   skip: True  # [py2k]

@scopatz
Copy link
Member Author

scopatz commented Nov 2, 2015

Disabled Python 2

- xonsh
commands:
- xonsh -h
nosetests -w tests
Copy link
Member

Choose a reason for hiding this comment

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

I suspect this is a problem.

Traceback (most recent call last): 
  File "C:\conda\scripts\obvci_conda_build_dir.py", line 15, in <module>
    bld_dir.main()
  File "C:\conda\lib\site-packages\obvci\cli\conda_build_dir.py", line 19, in main
    return Builder.handle_args(args).main()
  File "C:\conda\lib\site-packages\obvci\conda_tools\build_directory.py", line 223, in main
    recipe_metas = self.fetch_all_metas()
  File "C:\conda\lib\site-packages\obvci\conda_tools\build_directory.py", line 196, in fetch_all_metas
    recipe_metas = fetch_metas(conda_recipes_root)
  File "C:\conda\lib\site-packages\obvci\conda_tools\build_directory.py", line 65, in fetch_metas
    packages.append(MetaData(package_dir))
  File "C:\conda\lib\site-packages\conda_build\metadata.py", line 349, in __init__
    self.parse_again()
  File "C:\conda\lib\site-packages\conda_build\metadata.py", line 357, in parse_again
    self.meta = parse(get_contents(self.meta_path))
  File "C:\conda\lib\site-packages\conda_build\metadata.py", line 133, in parse
    res = yamlize(data)
  File "C:\conda\lib\site-packages\conda\utils.py", line 119, in __call__
    value = self.func(*args, **kw)
  File "C:\conda\lib\site-packages\conda_build\metadata.py", line 96, in yamlize
    return yaml.load(data, Loader=BaseLoader)
  File "C:\conda\lib\site-packages\yaml\__init__.py", line 71, in load
    return loader.get_single_data()
  File "C:\conda\lib\site-packages\yaml\constructor.py", line 37, in get_single_data
    node = self.get_single_node()
  File "_yaml.pyx", line 707, in _yaml.CParser.get_single_node (ext/_yaml.c:8308)
  File "_yaml.pyx", line 725, in _yaml.CParser._compose_document (ext/_yaml.c:8581)
  File "_yaml.pyx", line 776, in _yaml.CParser._compose_node (ext/_yaml.c:9306)
  File "_yaml.pyx", line 890, in _yaml.CParser._compose_mapping_node (ext/_yaml.c:10838) 
  File "_yaml.pyx", line 776, in _yaml.CParser._compose_node (ext/_yaml.c:9306)
  File "_yaml.pyx", line 890, in _yaml.CParser._compose_mapping_node (ext/_yaml.c:10838) 
  File "_yaml.pyx", line 774, in _yaml.CParser._compose_node (ext/_yaml.c:9284)
  File "_yaml.pyx", line 853, in _yaml.CParser._compose_sequence_node (ext/_yaml.c:10333)
  File "_yaml.pyx", line 905, in _yaml.CParser._parse_next_event (ext/_yaml.c:11045) 
yaml.scanner.ScannerError: while scanning a simple key
  in "<unicode string>", line 29, column 3
could not find expected ':'
  in "<unicode string>", line 31, column 1

@pelson
Copy link
Member

pelson commented Nov 3, 2015

Looks like this isn't building on OSX and Linux. Very strange - I suspect we need to update some of our dependencies... would you mind rebasing this on upstream/master to see your base was out of date?

@scopatz
Copy link
Member Author

scopatz commented Nov 3, 2015

OK. I rebased and nothing changed. Not sure why this is a problem on windows but not elsewhere. I'll just remove it for now.

@scopatz
Copy link
Member Author

scopatz commented Nov 3, 2015

Now seeing this gem which doesn't make sense to me since it is suppossed to be on Python 3.4

conda install -c http://conda.binstar.org/pelson/channel/development --yes --quiet obvious-ci
Fetching package metadata: ...... 
Solving package specifications: ........... 
Error: Unsatisfiable package specifications.
Generating hint: 
[                    ]|                                                  |   0%
[1/3                 ]|################                                  |  33%
[      COMPLETE      ]|##################################################| 100%



Hint: the following packages conflict with each other:
  - obvious-ci
  - python 3.5*

Use 'conda info obvious-ci' etc. to see the dependencies for each package.

Note that the following features are enabled:
  - vc14

Command exited with code 1

@ocefpaf
Copy link
Member

ocefpaf commented Nov 4, 2015

I just got affected by that too in another recipe. Maybe this comment can help.

@scopatz
Copy link
Member Author

scopatz commented Nov 4, 2015

@ocefpaf any ideas on how to actually fix this though?

@ocefpaf
Copy link
Member

ocefpaf commented Nov 4, 2015

Not yet... I am struggling with some hiccups in my day job.

@scopatz
Copy link
Member Author

scopatz commented Nov 4, 2015

Good luck

@pelson
Copy link
Member

pelson commented Nov 5, 2015

I've made some changes in #16 which (combined with https://github.com/conda-forge/obvious-ci-feedstock) I believe will allow this to build successfully. Either way I'm happy to merge this and fix any resulting problems (even if that means disabling a subset of the build matrix for a period). Progress. 😉

pelson added a commit that referenced this pull request Nov 5, 2015
@pelson pelson merged commit 386031a into conda-forge:master Nov 5, 2015
@dfroger dfroger mentioned this pull request Jan 14, 2017
traversaro added a commit to traversaro/staged-recipes that referenced this pull request Sep 16, 2023
# This is the 1st commit message:

Add recipe for the trlib C++ library
# The commit message #2 will be skipped:

# Update meta.yaml

# The commit message conda-forge#3 will be skipped:

# Update meta.yaml

# The commit message conda-forge#4 will be skipped:

# Update meta.yaml

# The commit message conda-forge#5 will be skipped:

# Create disable_valgrind.patch

# The commit message conda-forge#6 will be skipped:

# Update meta.yaml

# The commit message conda-forge#7 will be skipped:

# Remove dependency on C++ compiler

# The commit message conda-forge#8 will be skipped:

# Update 23.patch

# The commit message conda-forge#9 will be skipped:

# Update 23.patch

# The commit message conda-forge#10 will be skipped:

# Delete recipes/trlib/23.patch

# The commit message conda-forge#11 will be skipped:

# Create 23.patch

# The commit message conda-forge#12 will be skipped:

# Update build.sh

# The commit message conda-forge#13 will be skipped:

# Create win_fix.patch
traversaro added a commit to traversaro/staged-recipes that referenced this pull request Sep 16, 2023
# This is the 1st commit message:

Add recipe for the trlib C++ library
# The commit message #2 will be skipped:

# Update meta.yaml

# The commit message conda-forge#3 will be skipped:

# Update meta.yaml

# The commit message conda-forge#4 will be skipped:

# Update meta.yaml

# The commit message conda-forge#5 will be skipped:

# Create disable_valgrind.patch

# The commit message conda-forge#6 will be skipped:

# Update meta.yaml

# The commit message conda-forge#7 will be skipped:

# Remove dependency on C++ compiler

# The commit message conda-forge#8 will be skipped:

# Update 23.patch

# The commit message conda-forge#9 will be skipped:

# Update 23.patch

# The commit message conda-forge#10 will be skipped:

# Delete recipes/trlib/23.patch

# The commit message conda-forge#11 will be skipped:

# Create 23.patch

# The commit message conda-forge#12 will be skipped:

# Update build.sh

# The commit message conda-forge#13 will be skipped:

# Create win_fix.patch

# The commit message conda-forge#14 will be skipped:

# Update meta.yaml

# The commit message conda-forge#15 will be skipped:

# Update win_fix.patch
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

3 participants