Re: [Cocci] [PATCH 3/4] scripts: add glimpse.sh for indexing the kernel
From: SF Markus Elfring
Date: Sat Jun 11 2016 - 13:10:04 EST
> Glimpse is a tool you can use to index the kernel. The tool
> was recently open sourced under the ISC license and can be
> obtained at:
How do you think about to mention the script addition also directly in
the commit message?
> @@ -0,0 +1,12 @@
> +#!/bin/bash
> +
> +DIR=$(dirname $(readlink -f $0))
> +DIR="${DIR}/../"
Would you like to use the following variable assignment (instead of two
before)?
+DIR="$(dirname $(readlink -f $0))/../"
By the way: How are the chances to achieve further software improvements?
https://github.com/gvelez17/glimpse/issues
Regards,
Markus