Remove Xenial patches and scripts #13876
Conversation
| @@ -97,31 +97,27 @@ k_tick=$(echo ${kernel_version[2]} | awk -F'-' '{print $2}') | |||
| # do not skip md patch - new d421. | |||
| #[ $k_maj_min -ge 605 ] && skip_md_patch=1 | |||
|
|
|||
| # Construct branch name from distribution codename {xenial,bionic,..} and kernel version | |||
| # Construct branch name from distribution codename {bionic,focal...} and kernel version | |||
There was a problem hiding this comment.
bionic can also be removed.
We support min focal.
Same for all places
| @@ -43,14 +43,6 @@ PLATFORM=$(uname -n) | |||
| #fi | |||
| kernel_branch="master" | |||
There was a problem hiding this comment.
Please remove this file at all
| @@ -39,12 +39,12 @@ else | |||
| kernel_branch=$(choose_kernel_branch $LINUX_BRANCH) | |||
There was a problem hiding this comment.
We refer to it in our doc, installation_odroid.md, remove that file too?
scripts/patch-ubuntu-kernel-4.16.sh
Outdated
| @@ -36,12 +36,12 @@ retpoline_retrofit=0 | |||
|
|
|||
| LINUX_BRANCH=$(uname -r) | |||
|
|
|||
| # Construct branch name from distribution codename {xenial,bionic,..} and kernel version | |||
| # Construct branch name from distribution codename {bionic, focal...} and kernel version | |||
There was a problem hiding this comment.
I am not sure bionic has kernel 4.16.
I think we can drop this file.
Just make sure that on the installation doc no more references to files we delete
There was a problem hiding this comment.
We don't mention this file in our doc, the only references we have to files we remove are in installation_odroid.md and DSO-1369.md, both seem fairly old
| @@ -97,15 +96,9 @@ k_tick=$(echo ${kernel_version[2]} | awk -F'-' '{print $2}') | |||
| # do not skip md patch - new d421. | |||
| #[ $k_maj_min -ge 605 ] && skip_md_patch=1 | |||
|
|
|||
| # Construct branch name from distribution codename {bionic,focal...} and kernel version | |||
| # Construct branch name from distribution codename {focal, jammy...} and kernel version | |||
There was a problem hiding this comment.
Please try running this script so we can make sure it works before we merge.
This is our main patches script
There was a problem hiding this comment.
Followed the documentation with the modified script on a VM, the camera was detected fine
Tracked on: [LRS-1145]