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 Build customization #1585

Merged
merged 93 commits into from
Feb 24, 2017
Merged

WIP Build customization #1585

merged 93 commits into from
Feb 24, 2017

Conversation

msarahan
Copy link
Contributor

@msarahan msarahan commented Dec 18, 2016

Implements #1142

Supersedes #966

Copying design goals from #1142:

Design goals:

  • support config files at 3 levels, with lower levels overriding the upper ones:
    • included with conda-build
    • user's install-wide
    • per-package
    • document how older options, such as CONDA_PY and --python interact with/override config files
  • Identify key confiugration types:
    • pinning versions (versions that must match between build and run)
    • variants of a particular feature (BLAS: MKL/OpenBLAS/ATLAS/etc.)
    • Specifying compiler / toolchain options
  • Specify how to distinguish builds: how do different variations end up with different filenames?
    • Probably can't just have all variations contribute to build string directly.
    • Hash instead? Installation would need to access metadata, not rely on build string

@msarahan
Copy link
Contributor Author

Merging. Outstanding issues:

  • retry logic for compensating for conda's handling of the package cache fails builds on Travis. Working with conda team to come up with a proper fix for the package cache problems.
  • Windows builds failing noarch tests due to conda packaging error. Being tracked at conda 4.3 missing resources folder - noarch packages broken conda#4715 - should be fixed in conda 4.3.14.

@jasongrout
Copy link

Wow, this is huge. Thanks! Is there a high-level summary added to the docs that I can read?

CC @aburgm, @SylvainCorlay

@jakirkham
Copy link
Member

FYI apparently this added six as a requirement, but it doesn't appear in run currently.

@msarahan
Copy link
Contributor Author

@jasongrout docs are at conda/conda-docs#414

@jakirkham thanks - fixed in #1766

@github-actions
Copy link

github-actions bot commented May 5, 2022

Hi there, thank you for your contribution!

This pull request has been automatically locked because it has not had recent activity after being closed.

Please open a new issue or pull request if needed.

Thanks!

@github-actions github-actions bot added the locked [bot] locked due to inactivity label May 5, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked [bot] locked due to inactivity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants