-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
Fix package clobbering #52
Comments
To help identify which packages have clobbering issues, tried creating an environment with the Here is the full output:
The following packages were flagged as having clobbering issues. All with the license file: |
So simply doing that consistently across feedstocks would be helpful Maybe this is a good argument for consolidating CUDA Toolkit build scripts into one commonly used script: conda-forge/cf-nvidia-tools-feedstock#1 |
Some packages appear to install content (like license files) in the
$PREFIX
without naming them or placing them in a way to avoid clobbering between packages (for example: conda-forge/cuda-nvvp-feedstock#14 (comment) )We should work to remove any clobbering issues between CUDA Toolkit packages
When it comes to license files specifically, we already make sure to include these in all packages with
about/license_file
(for example). So there is no need to duplicate this effort. So with license files specifically, we should not copy them to the$PREFIX
, which should fix that particular kind of clobberingMore generally we may just want to adapt scripts to only copy directories
If we need to copy over any files, we likely will need to do so in a clearly specified way
Other clobbering issues can be handled on a case by case basis
The text was updated successfully, but these errors were encountered: