[Buildroot] [PATCH v2] pulseview: new package

[Bartosz Golaszewski]

2015-04-08 21:12 GMT+02:00 Thomas Petazzoni
<thomas.petazzoni at free-electrons.com>:
> I've reproduced the problem in the autobuilder directly, and the
> problem is fairly clear:

Thanks for the investigation!

>
> So, generating the C++ bindings requires Doxygen. And Doxygen is not
> part of the mandatory Buildroot dependencies, and not packaged in
> Buildroot as a host package.

Can we add an error message similar to those emitted when e.g. git or
svn is missing. but for doxygen (only when libsigrokcxx is selected)?

> The reason why it works for you is because you have Doxygen installed
> on your machine. Several of the autobuilders run in a minimal chroot
> that has only the strict requirements of Buildroot installed, nothing
> else, specifically to detect such issues.

Yes, this is the culprit.

> So, in the mean time, I've marked libsigrokcxx as broken, as well as
> pulseview. I believe the best solution is probably to create a
> host-doxygen package (or convince the libsigrok developers to not use
> something such as doxygen to generate C++ bindings). Can you look into
> this?

C++ bindings are generated using xml files created by doxygen. I will
take a look tomorrow, but it may be impossible to drop using doxygen
without having to do a major overhaul of libsigrokcxx. I think that
host-doxygen is a more feasible solution unless we can demand user to
install doxygeb on the host machine.

Best regards,
Bartosz Golaszewski