make rpm for fedora or other flavors of linux#304
make rpm for fedora or other flavors of linux#304harrisonvanderbyl wants to merge 1 commit intoamd:mainfrom
Conversation
|
Can one of the admins verify this patch? |
| elif [ -x "$(command -v dnf)" ]; then | ||
| dnf install -y jq | ||
| fi |
There was a problem hiding this comment.
Seems the other deps script of xrt uses yum instead of dnf (probably for compatibility?)
So, maybe here also use yum?
There was a problem hiding this comment.
DNF has been the default package manager since EL 8. How about let's not use it's legacy alias?
|
I think it's better to add option for building the rpm in addition of deb files instead of replacing the deb-packaging with rpm packaging. This is slightly modified version of original patch that does that. I have tested it on fedora 40. I am not sure whether we need to change the /lib/firmware/amdnpu to /usr/lib/firmware/amdnpu, so I dropped that part. |
|
Finally find some time to incorporate this change as #394. Closing this one since it is obsolete now. |

Idk, it just needs some if statements in there tbh