-
Notifications
You must be signed in to change notification settings - Fork 23
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
Adding host_exports
(like run_exports
except between build
& host
only)
#77
Comments
cc @xhochy (for awareness) |
I'm generally supportive of the idea provided that it's useful for a wide range of cases. A quick look made me see it as a "couple of edge cases". For example, it's not clear to me what "Add dependencies to host if providing package is added to host" means; if the package is already in host, why do we need to export anything? Is it to constrain it further? I guess we can use |
Thanks Jaime! 🙏 Am happy with limiting it to the case where a package is added to So that would mean there is no need for Does that same more reasonable? |
|
run_exports
has been immensely helpful. Just to summarize itrun_exports
:run
weak
only happens when the dependency is added tohost
(notbuild
)strong
happens when a dependency is added tobuild
orhost
However there are still some cases that are not covered. In particular there are cases where a dependency added to
build
needs to add itself tohost
, but there is no need to add it torun
(in fact this would often be better to avoid). IOWrun_exports/strong
does not meet this need. Some examples include:build
&host
of compiler dependencies (like OpenMP)llvm-openmp
only in build leads to arun_exports
against the latestllvm-openmp
conda-forge/openmp-feedstock#126cudart
)Solving these cases would involve a
host_exports
that allows adding other dependencies tohost
. Could imagine to use cases for a package providinghost_exports
host_exports/weak
(default)host
if providing package is added tohost
host_exports/strong
(opt-in)host
if providing package is inbuild
orhost
cudart
)The text was updated successfully, but these errors were encountered: