On Fri, 12 Jan 2024 at 15:02, Erik Schilling <erik.schill...@linaro.org> wrote:
>
> So far one could only specify yes/no (optionally for all drivers through
> --enable-all-drivers={yes,no}). However, even when stating a =yes, the
> dependency check would override it to a =no if the dependencies are
> missing.
>
> This is a bit annoying for scripts since a script may specify
> --enable-foo-driver=yes but in fact the build may not contain
> `foo-driver` due to missing build dependencies. This is especially
> problematic for distro's which look like they currently do not build a
> defined driver set - which results in random drivers missing [1].
>
> This fixes that by following the autoconf examples [2] that allow a third
> state `check`. With that `no` means no, `yes` means yes (raise an error
> if deps are unavailable) and `check` gracefully disables drivers with
> unavailable dependencies.
>
> With that --enable-all-drivers can be used to ensure that all drivers are
> built in. Certain drivers can still be explicitly disabled by overriding
> the individual driver setting.
>
> This way distros can specify --enable-all-drivers and new drivers with
> unmet dependencies will cause an error rather than silently getting
> dropped of the build.
>
> [1] https://sigrok.org/bugzilla/show_bug.cgi?id=1699
> [2] 
> https://www.gnu.org/software/autoconf/manual/autoconf-2.67/html_node/External-Software.html#External-Software
>
> To: sigrok-devel@lists.sourceforge.net
> Cc: Daniel Thompson <daniel.thomp...@linaro.org>
> ---
> I realized that Fedora only builds libsigrok with a somewhat random set
> of enabled drivers. It also seems to cause ambigious situations for the
> appimage builds.
>
> This proposes a more explicit and less error-prone method of specifying
> the drivers expected to be built in.
> ---
>  configure.ac | 13 +++++++++----
>  1 file changed, 9 insertions(+), 4 deletions(-)

Posted as https://github.com/sigrokproject/libsigrok/pull/252, as
GitHub seems more active nowadays :)


_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to