Frogging-Family / linux-tkg

linux-tkg custom kernels
GNU General Public License v2.0
1.32k stars 163 forks source link

CacULE/v5.16/0001-cacULE-5.16-full.patch 404 error #416

Closed iamtrazy closed 2 years ago

iamtrazy commented 2 years ago

https://raw.githubusercontent.com/CachyOS/cacule-cpu-scheduler/master/patches/CacULE/v5.16/0001-cacULE-5.16-full.patch SSL_INIT Loaded CA certificate '/etc/ssl/certs/ca-certificates.crt' Resolving raw.githubusercontent.com (raw.githubusercontent.com)... 185.199.109.133, 185.199.108.133, 185.199.110.133, ... Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|185.199.109.133|:443... connected. HTTP request sent, awaiting response... 404 Not Found 2022-01-31 09:51:24 ERROR 404: Not Found.

there is no patch named 0001-cacULE-5.16-full.patch in cachyOS repo https://github.com/CachyOS/cacule-cpu-scheduler/tree/master/patches/CacULE/v5.16

iamtrazy commented 2 years ago

is it okay if i use https://github.com/CachyOS/cacule-cpu-scheduler/blob/master/patches/CacULE/v5.16/0001-cacULE-5.16.patch , i think there is a issue with naming scheme

iamtrazy commented 2 years ago

I think CachyOS renamed there patch in this commit https://github.com/CachyOS/cacule-cpu-scheduler/commit/c28a7acf306a9d7ebcb2adb124acce6fb585ce0d

Orangian commented 2 years ago

Can confirm. Commit Frogging-Family/linux-tkg@f190ac0 seems to have caused this. Line 571 of prepare needs to be changed from tkgpatch="$srcdir/0001-cacULE-${_basekernel}-full.patch" && _tkg_patcher to tkgpatch="$srcdir/0001-cacULE-${_basekernel}.patch" && _tkg_patcher

ptr1337 commented 2 years ago

I think CachyOS renamed there patch in this commit CachyOS/cacule-cpu-scheduler@c28a7ac

Yes, cause his commit was the weird naming scheme, so i did change it. I also wrote TKG that i changed it. So its the same name scheming.

Orangian commented 2 years ago

Ah. In that case lines 553-569 in prepare can be restructured to no longer check for kernel versions above 5.15.

RyanHakurei commented 2 years ago

@Orangian Line 568 needs to be changed as well.

AdelKS commented 2 years ago

Oh this issue can be closed ! Good job guys