On 07/15/2010 07:14 PM, Michal Marek wrote:
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.)
I was documenting all the changes. That way it features both as a config option change list and a new features list. Both are things that people have been asking for. -Jeff
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 ;;
-- Jeff Mahoney SuSE Labs -- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-kernel+help@opensuse.org