[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <4223a383-7731-0a4d-d807-1141b6fcbea8@fb.com>
Date: Fri, 3 May 2019 16:56:20 +0000
From: Yonghong Song <yhs@...com>
To: Quentin Monnet <quentin.monnet@...ronome.com>,
netdev <netdev@...r.kernel.org>,
"bpf@...r.kernel.org" <bpf@...r.kernel.org>,
"Alexei Starovoitov" <ast@...com>,
Daniel Borkmann <daniel@...earbox.net>
Subject: Re: bpftool doc man page build failure
On 5/3/19 9:54 AM, Quentin Monnet wrote:
> 2019-05-03 16:21 UTC+0000 ~ Yonghong Song <yhs@...com>
>> Quentin,
>>
>> I hit the following errors with latest bpf-next.
>>
>> -bash-4.4$ make man
>> GEN bpftool-perf.8
>> GEN bpftool-map.8
>> GEN bpftool.8
>> GEN bpftool-net.8
>> GEN bpftool-feature.8
>> GEN bpftool-prog.8
>> GEN bpftool-cgroup.8
>> GEN bpftool-btf.8
>> GEN bpf-helpers.rst
>> Parsed description of 111 helper function(s)
>> Traceback (most recent call last):
>> File "../../../../scripts/bpf_helpers_doc.py", line 421, in <module>
>> printer.print_all()
>> File "../../../../scripts/bpf_helpers_doc.py", line 187, in print_all
>> self.print_one(helper)
>> File "../../../../scripts/bpf_helpers_doc.py", line 378, in print_one
>> self.print_proto(helper)
>> File "../../../../scripts/bpf_helpers_doc.py", line 356, in print_proto
>> proto = helper.proto_break_down()
>> File "../../../../scripts/bpf_helpers_doc.py", line 56, in
>> proto_break_down
>> 'type' : capture.group(1),
>> AttributeError: 'NoneType' object has no attribute 'group'
>> make: *** [bpf-helpers.rst] Error 1
>> -bash-4.4$ pwd
>> /home/yhs/work/net-next/tools/bpf/bpftool/Documentation
>> -bash-4.4$
>>
>> Maybe a format issue in the comments with some recent helpers?
>>
>> Thanks,
>>
>> Yonghong
>>
>
> Hi Yonghong,
>
> Thanks for the notice! Yes, I observed the same thing not long ago. It
> seems that the Python script breaks on the "unsigned long" pointer
> argument for strtoul(): the script only accepts "const" or "struct" for
> types made of several words, not "unsigned".
>
> I'll fix the script so it can take any word and send a patch next week,
> along with some other clean-up fixes for the doc.
Thanks!
>
> Best regards,
> Quentin
>
Powered by blists - more mailing lists