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

[Issue]: Tensile should not be vendored #1396

Open
bgamari opened this issue Nov 27, 2024 · 0 comments
Open

[Issue]: Tensile should not be vendored #1396

bgamari opened this issue Nov 27, 2024 · 0 comments

Comments

@bgamari
Copy link

bgamari commented Nov 27, 2024

Problem Description

It appears that hipBLASLt depends upon Tensile. However, instead of recognizing this fact, the dependency is ignored by the documentation and is instead vendored.

Vendoring of dependencies creates untold headaches for distributors as the packager needs to:

  • identify the implicit dependency
  • sort out what version of the dependency the package expects
  • figure out how to furnish the dependency in a way that works with whatever bespoke mechanism the package has developed
  • continually maintain all of the above as the package and its dependency evolve

For this reason, vendoring of dependency is highly discouraged. If hipBLASLt depends upon Tensile then there is a proper way to do so: advertise the dependency in the readme, and discover it via the usual dependency discovery mechanisms.

Operating System

Any

CPU

All

GPU

Other

Other

this is a packaging issue and not hardware specific

ROCm Version

ROCm 6.2.3, ROCm 6.2.2, ROCm 6.2.1, ROCm 6.2.0, ROCm 6.1.2, ROCm 6.1.1, ROCm 6.1.0, ROCm 6.0.2

ROCm Component

hipBLASLt

Steps to Reproduce

No response

(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support

No response

Additional Information

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant