On 10/15/19 6:36 PM, Bhaskar Chowdhury wrote: > On 16:03 Tue 15 Oct 2019, Randy Dunlap wrote: >> Subject: s/prunning/pruning/ >> Subject: s/:scripts:/scripts:/ >> >> On 10/15/19 7:55 AM, Bhaskar Chowdhury wrote: >>> This patch will remove old kernels from system selective way. >> >> in a selective way. >> >>> >>> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@xxxxxxxxx> >>> --- >>> scripts/prune-kernel | 16 ++++++++-------- >>> 1 file changed, 8 insertions(+), 8 deletions(-) >>> >>> diff --git a/scripts/prune-kernel b/scripts/prune-kernel >>> index 74143f229f84..28fae6c91218 100755 >>> --- a/scripts/prune-kernel >>> +++ b/scripts/prune-kernel >>> @@ -33,7 +33,7 @@ printf "\t\n Enlist the installed kernels \n\n\n" >>> cd $boot_dir && pwd >>> >>> #Bash experts not recomend to run ls inside script,but you can by running that too >>> -# ls -1 vmlinuz-* >>> +# ls -1 vmlinuz-* >>> >>> #This is the recommended way , little complex but that's what people want to see! >>> >>> @@ -42,10 +42,10 @@ find $boot_dir -name "vmlinuz-*" -type f -print0 -exec ls -1 {} \; >>> printf "\n\n\n Well, we need to purge some kernel to gain some space.\n\n\n" >>> >>> >>> -printf "Please give the kernel version to remove: %s" >>> +printf "Please give the kernel version to remove: %s" >>> read kernel_version >>> >>> -remove_old_kernel >>> +remove_old_kernel >>> >>> printf "\n\n Remove associated modules too ... \n\n" >>> >>> @@ -54,7 +54,7 @@ cd $modules_dir && pwd >>> printf "\n\n\n Enlist the installed modules \n\n\n" >>> >>> #This is (-1) minus one not l(el) >>> -# ls -1 >>> +# ls -1 >>> >>> find $modules_dir -name "$kernel_version-*" -type f -print0 -exec ls -1 {} \; >>> >>> @@ -66,14 +66,14 @@ rm -rf $modules_version >>> printf "\n\n Done \n\n" >>> >>> printf "\n\n Want to remove another?[Yn] : %s" >>> -read response >>> +read response >>> >>> >>> if [[ $response == "Y" ]]; then >>> - printf "Please give another version to remove : %s" >>> + printf "Please give another version to remove : %s" >>> read kernel_version >>> >>> -remove_old_kernel >>> +remove_old_kernel >>> >>> elif [[ $response == "n" ]]; then >>> >>> @@ -81,4 +81,4 @@ elif [[ $response == "n" ]]; then >>> >>> fi >>> >>> -exit 0 >>> +exit 0 >>> -- >>> 2.21.0 >>> >> >> Trying to apply this patch with 'patch --verbose' says: >> >> checking file scripts/prune-kernel >> Using Plan A... >> Hunk #1 FAILED at 33. >> Hunk #2 FAILED at 42. >> Hunk #3 FAILED at 54. >> Hunk #4 FAILED at 66. >> Hunk #5 FAILED at 81. >> 5 out of 5 hunks FAILED >> Hmm... Ignoring the trailing garbage. >> done >> >> >> Those + and - lines with the same content don't make any sense. >> >> And probably Cc: the author of the script: >> J. Bruce Fields <bfields@xxxxxxxxxxxx> >> >> -- >> ~Randy > > Thank you Randy, silly mistakes creeps in...my bad ...will correct that > and resend. > BTW I haven't found Bruce's name anywhere...where did you get it? Or did I > miss the obvious?? It's in the git repository history for this file. > Ran against get_maintainers and it only throw a open list , no other > names.So kinda, stump by that.> Time to redo it again...huh -- ~Randy