[Click] [PATCH 03/12] Configury: declare kobj_ns_type before including linux/sysfs.h

[Joonwoo Park]

You are right exactly.
Please find attached revised patch.

Thanks,
Joonwoo

On Wed, Sep 29, 2010 at 4:04 PM, Eddie Kohler <kohler at cs.ucla.edu> wrote:
> I believe 4e73b8b9157a6c62c2d5e855e0c13ffbf353668e means we don't need the
> rcupdate.h portion of this patch.  Confirm & resubmit?
>
> E
>
>
> On 09/25/2010 10:29 PM, Joonwoo Park wrote:
>>
>> eliminate systax error below:
>> use of enum ‘kobj_ns_type’ without previous declaration
>>
>> Signed-off-by: Joonwoo Park<joonwpark81 at gmail.com>
>> ---
>>  linuxmodule/fixincludes.pl |    6 ++++++
>>  1 files changed, 6 insertions(+), 0 deletions(-)
>>
>> diff --git a/linuxmodule/fixincludes.pl b/linuxmodule/fixincludes.pl
>> index a4c9aa7..5553d9a 100755
>> --- a/linuxmodule/fixincludes.pl
>> +++ b/linuxmodule/fixincludes.pl
>> @@ -205,6 +205,12 @@ sub one_includeroot ($$) {
>>                # array dimension.
>>
>>  s&sizeof\s*\(\s*struct\s*\{\s*\w+\s*:\s*-\s*\!\s*\!\s*\(e\)\s*;\s*\}\s*\)&(sizeof(int[-!!(e)])*(size_t)0)&g;
>>            }
>> +           if ($d eq "rcupdate.h") {
>> +               s{(^static inline notrace)(.+)}{"$1$2;\nstatic
>> inline$2"}emg;
>> +           }
>> +           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;
>> +           }
>>
>>            # unquote.
>>            $_ = sunprotect($_);
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-eliminate-systax-error-below.patch
Type: text/x-patch
Size: 1117 bytes
Desc: not available
Url : http://amsterdam.lcs.mit.edu/pipermail/click/attachments/20101003/27ed3357/attachment.bin