Mellanox / nv_peer_memory

305 stars 61 forks source link

Dependency linux-headers | linux-headers-generic is not present on Debian #23

Closed byronyi closed 7 years ago

byronyi commented 7 years ago

Would you consider adding linux-headers-amd64 as an alternative dependency? It might need a couples of other packages for other arches, but I am not aware of a meta package like linux-headers-generic in Debian as in Ubuntu.

ferasd commented 7 years ago
alaahl commented 7 years ago

Actually, I also had some issues with the linux sources dependency in some OFED packages in the past. Eventually, I moved it to "Recommends:" tag instead of "Depends:", and now it looks like this:

Recommends: linux-headers-arm64 | linux-headers-powerpc | linux-headers-ppc64 | linux-headers-ppc64le | linux-headers-amd64 | linux-headers-generic | linux-headers

byronyi commented 7 years ago

Would this change be applied to the project? Certainly I could change this by myself, but it would be better handled else where.

alaahl commented 7 years ago

submitted https://github.com/Mellanox/nv_peer_memory/pull/25

byronyi commented 7 years ago

Sorry just saw this. Thanks @alaahl!