[Buildroot] [PATCH 1/1] package/postgresql: needs wchar
Yann E. MORIN
yann.morin.1998 at free.fr
Thu Nov 15 16:59:03 UTC 2018
Thomas, All,
On 2018-11-01 22:30 +0100, Thomas Petazzoni spake thusly:
> On Tue, 23 Oct 2018 19:04:26 +0100, Arnout Vandecappelle wrote:
> > It is not strictly needed, but it is still useful to have it because:
I agree with Arnout here.
However, from a purely pragmatic point of view, I see that it is totally
useless in practice, so I would not mind we drop them.
And in retrospect, I think it *is* better that we do drop them. If the
top-level option loses that dependency, then it is 'easy' to detect it
has become mandatory for a sub-option, because the autobuilders will
fail.
However, if we repeat the dependency and the top-level option lses it
and we forget to remove it, we will never realise that the sub-option
shouldalso lose it.
> True. In this case:
>
> - Bernd did not propagate the dependency to pĥp, qt and qt5base, which
> all three are also directly selecting BR2_PACKAGE_POSTGRESQL
>
> - A number of places where BR2_PACKAGE_POSTGRESQL is selected do not
> have the !BR2_STATIC_LIBS dependency.
>
> I am really wondering what to do with those "useless" dependencies in
> general. On one hand, I agree with you that semantically, it is better
> to have them. On the other hand:
>
> - It's an additional maintenance burden.
>
> - It's never tested by the autobuilders, because such "useless"
> dependencies are well, useless, because they are hidden by another
> higher-level dependency. Because they are not tested, they are often
> wrong and not maintained.
>
> Due to this, whether we propagate them or not is completely random
> currently through the Buildroot tree, and I'm not sure it's a very nice
> situation.
I agree with you. Whatever we choose, we should do it consistently.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list