Skip to content

Fix #11140#11141

Open
trxcllnt wants to merge 1 commit intoopenucx:masterfrom
trxcllnt:patch-1
Open

Fix #11140#11141
trxcllnt wants to merge 1 commit intoopenucx:masterfrom
trxcllnt:patch-1

Conversation

@trxcllnt
Copy link

What?

Remove libnvidia-compute dependency from debian package dependencies.

Why?

Fixes #11140

How?

It appears the string sed -i 's/libnvidia-compute | libnvidia-ml1, //g' tmp/DEBIAN/control is replacing has changed:

$ dpkg-deb -R ucx-cuda-1.19.0.deb tmp
$ grep Depends: tmp/DEBIAN/control
> Depends: libc6 (>= 2.34), libnvidia-compute | libnvidia-ml1 (>= 555.42.06), ucx

Updating the pattern with a wildcard match seems to work:

$ sed 's/libnvidia-compute | libnvidia-ml1.*, //g' tmp/DEBIAN/control | grep Depends
> Depends: libc6 (>= 2.34), ucx

# Remove superfluous dependency
dpkg-deb -R "ucx-cuda-${VER}.deb" tmp # Extract
sed -i 's/libnvidia-compute | libnvidia-ml1, //g' tmp/DEBIAN/control
sed -i 's/libnvidia-compute | libnvidia-ml1.*, //g' tmp/DEBIAN/control
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems it does greedy match, other package is eaten, see below:

$ echo 'Depends: libc6 (>= 2.34), libnvidia-compute | libnvidia-ml1 (>= 555.42.06), other, ucx' | \
    sed 's/libnvidia-compute | libnvidia-ml1.*, //g'
Depends: libc6 (>= 2.34), ucx

Maybe something like:

$ echo 'Depends: libc6 (>= 2.34), libnvidia-compute | libnvidia-ml1 (>= 555.42.06), other, ucx' | sed 's/libnvidia-compute | libnvidia-ml1[^,]*, //'
Depends: libc6 (>= 2.34), other, ucx

@tvegas1
Copy link
Contributor

tvegas1 commented Feb 3, 2026

To unblock CI, please rename PR/commit with something like "AZP: Fix debian package dependency cleanup", and provide actual email address in AUTHORS file.

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

Successfully merging this pull request may close these issues.

CUDA 12 debs require libnvidia-compute

2 participants