<div dir="ltr"><div dir="ltr"><div>Hi,</div><div><br></div><div>I wasn't aware that `bash-completion` also implied `bash` installation. It can be easily be made optional[0][1].<br><br></div><div>I have provided a MR that handles this issue, taking also in account the change in `bash-completion` 2.10.</div><div><br></div><div>Thank you for notifying the issue Yegor,<br></div><div><br>[0] <a href="https://github.com/GNOME/dconf/blob/4057f8c84ff8c834dcd1cb82f946dd8cc49712ff/meson.build#L65">https://github.com/GNOME/dconf/blob/4057f8c84ff8c834dcd1cb82f946dd8cc49712ff/meson.build#L65</a><br>[1] <a href="https://github.com/GNOME/tracker/blob/a07580610fe0d91124b2632f9a2ca1d5dfa8a40d/meson.build#L258">https://github.com/GNOME/tracker/blob/a07580610fe0d91124b2632f9a2ca1d5dfa8a40d/meson.build#L258</a><br>[2] <a href="https://gitlab.freedesktop.org/mobile-broadband/libmbim/-/merge_requests/77">https://gitlab.freedesktop.org/mobile-broadband/libmbim/-/merge_requests/77</a></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El sáb, 24 jul 2021 a las 22:31, Yegor Yefremov (<<a href="mailto:yegorslists@googlemail.com">yegorslists@googlemail.com</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hey Aleksander, Iñigo,<br>
<br>
On Thu, Jul 22, 2021 at 9:44 AM Aleksander Morgado<br>
<<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>> wrote:<br>
><br>
> Hey Yegor,<br>
><br>
> Adding Iñigo in CC, who is the one who did all this.<br>
><br>
> > have you already tried to build libmbim via meson in BR?<br>
> ><br>
><br>
> Nope, no.<br>
><br>
> > 1. libmbim needs bash_completion now. This means bash must be<br>
> > installed as well. Does it make sense to make this feature optional?<br>
> ><br>
><br>
> I recall something about this, I think we had to add it in the CI<br>
> image. Iñigo, is this something we can make optional?<br>
<br>
Should I submit a patch?<br>
<br>
> > 2. meson_post_install.py has a problem. In my case<br>
> > "bash_completion_completionsdir" is an absolute path<br>
> > "/usr/share/bash-completion/completions", hence os.path.join() ignores<br>
> > prefix according to its definition. If you bump the minimal meson<br>
> > version to 0.46, you can use install_data command with rename feature.<br>
> ><br>
><br>
> We default to 0.45.1 because that is the meson version released in a<br>
> default Ubuntu 18.04 (LTS in support until April 2023). Unfortunately<br>
> meson is pretty new, and the newest releases are very powerful, but<br>
> until now we've always tried to be in line with what the LTS releases<br>
> provide.<br>
<br>
OK. I see. As long as bash completion is optional, we can do without<br>
this feature in BR.<br>
<br>
> > But I had no problems with gobject-introspection :-)<br>
> ><br>
><br>
> Hah :)<br>
><br>
> Also, there is another blocker issue in the meson port right now,<br>
> something related to how gtk-doc documentation is built. We cannot<br>
> fully switch to meson until that is resolved. That would not be an<br>
> issue for buildroot, though, no gtk-doc should be built there.<br>
<br>
Yup.<br>
<br>
Best regards,<br>
Yegor<br>
</blockquote></div></div>