On 07/15/2010 11:43 AM, Brandon Philips wrote:
Hello Michal, Greg, Jeff, et. all-
This is sort of ugly and more of an RFC to see if it makes sense.
What do you all think?
Cheers,
Brandon
We have started tracking config changes in master in doc/config-options.changes. Require that any config changes update that file.
I thought that Jeff's idea was only to annotate differences from the Kconfig defaults. (Personally, I would even leave out all those "Enabled as module; defaults to n" and only document differences against allmodconfig, but I don't care much.)
Signed-off-by: Brandon Philips
--- scripts/log | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-)
--- a/scripts/log +++ b/scripts/log @@ -37,6 +37,22 @@ trap 'rm -rf "$tmpdir"' EXIT tmpdir=$(mktemp -d /tmp/${0##*/}.XXXXXX) message=$tmpdir/message
+config_log_entry() { + git diff doc/config-options.changes \ + | grep "^+[^+][^-]" \ + | tail +2 \ + | sed 's/^+//' \ + >> $tmpdir/configlog + + if [ ! -s $tmpdir/configlog ]; then + echo "ERROR: Config changes,"\ + "update doc/config-options.changes" >&2 + exit 1 + fi + + cat $tmpdir/configlog >> $message +} + log_entry() { local entry=$1
@@ -99,7 +115,7 @@ for file in "${files[@]}"; do case "$file" in config/*) if [ -z "$configs_updated" ]; then - log_entry "Update config files." + config_log_entry configs_updated=1 fi ;;
-- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-kernel+help@opensuse.org