[Click] [PATCH 04/12] Configury: prepend net_device namespace for NETREG_REGISTERED

Eddie Kohler kohler at cs.ucla.edu
Wed Sep 29 19:06:34 EDT 2010


Should the before portion of the s{} have beginning-of-line and end-of-line 
markers?  And can you include in the message an example of an offending line?

Eddie


On 09/25/2010 10:29 PM, Joonwoo Park wrote:
> eliminate syntax error below:
> error: ‘NETREG_REGISTERED’ was not declared in this scope
>
> Signed-off-by: Joonwoo Park<joonwpark81 at gmail.com>
> ---
>   linuxmodule/fixincludes.pl |    3 +++
>   1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/linuxmodule/fixincludes.pl b/linuxmodule/fixincludes.pl
> index 5553d9a..4088643 100755
> --- a/linuxmodule/fixincludes.pl
> +++ b/linuxmodule/fixincludes.pl
> @@ -211,6 +211,9 @@ sub one_includeroot ($$) {
>   	    if ($d eq "kobject.h") {
>   		s{(^\#include \<linux\/sysfs.h\>(.*\n)*)(^enum kobj_ns_type\s\{\n([^\}].*\n)*\}\;)((.*\n)*)}{"$3\n$1\/*\n$3\n*\/$5"}emg;
>   	    }
> +	    if ($d eq "netdevice.h") {
> +		s{(.*)(\(.*\b)(NETREG_.+)(\b\))}{#ifdef __cplusplus\n$1$2net_device::$3$4\n#else\n$1$2$3$4\n#endif}g;
> +	    }
>
>   	    # unquote.
>   	    $_ = sunprotect($_);



More information about the click mailing list