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

find used variables doesn't consider short-circuiting #3255

Open
isuruf opened this issue Nov 10, 2018 · 3 comments
Open

find used variables doesn't consider short-circuiting #3255

isuruf opened this issue Nov 10, 2018 · 3 comments
Labels
stale::recovered [bot] recovered after being marked as stale

Comments

@isuruf
Copy link
Contributor

isuruf commented Nov 10, 2018

Actual Behavior

If I have # [osx and c_compiler == 'toolchain_c'], then conda-build thinks c_compiler is used in windows as well.

Expected Behavior

c_compiler not in used variables in windows.

Steps to Reproduce

example recipe: conda-forge/r-base-feedstock#61

Using conda-build 3.16.2

isuruf added a commit to mingwandroid/r-base-feedstock that referenced this issue Nov 10, 2018
@msarahan
Copy link
Contributor

msarahan commented Nov 10, 2018

Known issue. Conda build would have to parse, not just pattern match. It would be a large increase in complexity. PRs definitely welcome here if you have good ideas on how to do this without too much work. It would be a big improvement, but it's not one that we have time to do ourselves right now.

@github-actions
Copy link

Hi there, thank you for your contribution!

This issue has been automatically marked as stale because it has not had recent activity. It will be closed automatically if no further activity occurs.

If you would like this issue to remain open please:

  1. Verify that you can still reproduce the issue at hand
  2. Comment that the issue is still reproducible and include:
    - What OS and version you reproduced the issue on
    - What steps you followed to reproduce the issue

NOTE: If this issue was closed prematurely, please leave a comment.

Thanks!

@github-actions github-actions bot added the stale [bot] marked as stale due to inactivity label Dec 20, 2022
@jakirkham
Copy link
Member

Let's keep this open

@github-actions github-actions bot added stale::recovered [bot] recovered after being marked as stale and removed stale [bot] marked as stale due to inactivity labels Dec 21, 2022
@github-project-automation github-project-automation bot moved this to 🆕 New in 🧭 Planning Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale::recovered [bot] recovered after being marked as stale
Projects
Status: 🆕 New
Development

No branches or pull requests

3 participants