qemu/scripts/clean-includes
<<
>>
Prefs
   1#!/bin/sh -e
   2#
   3# Clean up QEMU #include lines by ensuring that qemu/osdep.h
   4# is the first include listed in .c files, and no headers provided
   5# by osdep.h itself are redundantly included in either .c or .h files.
   6#
   7# Copyright (c) 2015 Linaro Limited
   8#
   9# Authors:
  10#  Peter Maydell <peter.maydell@linaro.org>
  11#
  12# This work is licensed under the terms of the GNU GPL, version 2
  13# or (at your option) any later version. See the COPYING file in
  14# the top-level directory.
  15
  16# Usage:
  17#   clean-includes [--git subjectprefix] file ...
  18# or
  19#   clean-includes [--git subjectprefix] --all
  20#
  21# If the --git subjectprefix option is given, then after making
  22# the changes to the files this script will create a git commit
  23# with the subject line "subjectprefix: Clean up includes"
  24# and a boilerplate commit message.
  25#
  26# Using --all will cause clean-includes to run on the whole source
  27# tree (excluding certain directories which are known not to need
  28# handling).
  29
  30# This script requires Coccinelle to be installed.
  31
  32# .c files will have the osdep.h included added, and redundant
  33# includes removed.
  34# .h files will have redundant includes (including includes of osdep.h)
  35# removed.
  36# Other files (including C++ and ObjectiveC) can't be handled by this script.
  37
  38# The following one-liner may be handy for finding files to run this on.
  39# However some caution is required regarding files that might be part
  40# of the guest agent or standalone tests.
  41
  42# for i in `git ls-tree --name-only HEAD`  ; do test -f $i && \
  43#   grep -E '^# *include' $i | head -1 | grep 'osdep.h' ; test $? != 0 && \
  44#   echo $i ; done
  45
  46
  47GIT=no
  48
  49# Extended regular expression defining files to ignore when using --all
  50XDIRREGEX='^(tests/tcg|tests/multiboot|pc-bios|disas/libvixl)'
  51
  52if [ $# -ne 0 ] && [ "$1" = "--git" ]; then
  53    if [ $# -eq 1 ]; then
  54        echo "--git option requires an argument"
  55        exit 1
  56    fi
  57    GITSUBJ="$2"
  58    GIT=yes
  59    shift
  60    shift
  61fi
  62
  63if [ $# -eq 0 ]; then
  64    echo "Usage: clean-includes [--git subjectprefix] [--all | foo.c ...]"
  65    echo "(modifies the files in place)"
  66    exit 1
  67fi
  68
  69if [ "$1" = "--all" ]; then
  70    # We assume there are no files in the tree with spaces in their name
  71    set -- $(git ls-files '*.[ch]' | grep -E -v "$XDIRREGEX")
  72fi
  73
  74# Annoyingly coccinelle won't read a scriptfile unless its
  75# name ends '.cocci', so write it out to a tempfile with the
  76# right kind of name.
  77COCCIFILE="$(mktemp --suffix=.cocci)"
  78
  79trap 'rm -f -- "$COCCIFILE"' INT TERM HUP EXIT
  80
  81cat >"$COCCIFILE" <<EOT
  82@@
  83@@
  84
  85(
  86+ #include "qemu/osdep.h"
  87 #include "..."
  88|
  89+ #include "qemu/osdep.h"
  90 #include <...>
  91)
  92EOT
  93
  94
  95for f in "$@"; do
  96  case "$f" in
  97    *.inc.c)
  98      # These aren't standalone C source files
  99      echo "SKIPPING $f (not a standalone source file)"
 100      continue
 101      ;;
 102    *.c)
 103      MODE=c
 104      ;;
 105    *include/qemu/osdep.h | \
 106    *include/qemu/compiler.h | \
 107    *include/standard-headers/ )
 108      # Removing include lines from osdep.h itself would be counterproductive.
 109      echo "SKIPPING $f (special case header)"
 110      continue
 111      ;;
 112    *include/standard-headers/*)
 113      echo "SKIPPING $f (autogenerated header)"
 114      continue
 115      ;;
 116    *.h)
 117      MODE=h
 118      ;;
 119    *)
 120      echo "WARNING: ignoring $f (cannot handle non-C files)"
 121      continue
 122      ;;
 123  esac
 124
 125  if [ "$MODE" = "c" ]; then
 126    # First, use Coccinelle to add qemu/osdep.h before the first existing include
 127    # (this will add two lines if the file uses both "..." and <...> #includes,
 128    # but we will remove the extras in the next step)
 129    spatch  --in-place --no-show-diff --cocci-file "$COCCIFILE" "$f"
 130
 131    # Now remove any duplicate osdep.h includes
 132    perl -n -i -e 'print if !/#include "qemu\/osdep.h"/ || !$n++;' "$f"
 133  else
 134    # Remove includes of osdep.h itself
 135    perl -n -i -e 'print if !/\s*#\s*include\s*(["<][^>"]*[">])/ ||
 136                            ! (grep { $_ eq $1 } qw ("qemu/osdep.h"))' "$f"
 137  fi
 138
 139  # Remove includes that osdep.h already provides
 140  perl -n -i -e 'print if !/\s*#\s*include\s*(["<][^>"]*[">])/ ||
 141                          ! (grep { $_ eq $1 } qw (
 142           "config-host.h" "config-target.h" "qemu/compiler.h"
 143           <setjmp.h> <stdarg.h> <stddef.h> <stdbool.h> <stdint.h> <sys/types.h>
 144           <stdlib.h> <stdio.h> <string.h> <strings.h> <inttypes.h>
 145           <limits.h> <unistd.h> <time.h> <ctype.h> <errno.h> <fcntl.h>
 146           <sys/stat.h> <sys/time.h> <assert.h> <signal.h>
 147           "sysemu/os-posix.h, sysemu/os-win32.h "glib-compat.h"
 148           "qemu/typedefs.h"
 149            ))' "$f"
 150
 151done
 152
 153if [ "$GIT" = "yes" ]; then
 154    git add -- "$@"
 155    git commit --signoff -F - <<EOF
 156$GITSUBJ: Clean up includes
 157
 158Clean up includes so that osdep.h is included first and headers
 159which it implies are not included manually.
 160
 161This commit was created with scripts/clean-includes.
 162
 163EOF
 164
 165fi
 166