[Buildroot] [next] package/rpcbind: add systemd dependency when cfg for systemd

[Buildroot] [next] package/rpcbind: add systemd dependency when cfg for systemd

[Matt Weber]

Signed-off-by: Matthew Weber <matthew.weber@rockwellcollins.com>
---
 package/rpcbind/rpcbind.mk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/rpcbind/rpcbind.mk b/package/rpcbind/rpcbind.mk
index 0f5011e522..46e5b27d35 100644
--- a/package/rpcbind/rpcbind.mk
+++ b/package/rpcbind/rpcbind.mk
@@ -17,6 +17,7 @@ RPCBIND_CONF_OPTS += --with-rpcuser=root
 
 ifeq ($(BR2_INIT_SYSTEMD),y)
 RPCBIND_CONF_OPTS += --with-systemdsystemunitdir=/usr/lib/systemd/system
+RPCBIND_DEPENDENCIES += systemd
 else
 RPCBIND_CONF_OPTS += --with-systemdsystemunitdir=no
 endif
-- 
2.17.1

[Buildroot] [next] package/rpcbind: add systemd dependency when cfg for systemd

[Thomas Petazzoni]

On Wed, 20 Nov 2019 09:00:15 -0600
Matt Weber <matthew.weber@rockwellcollins.com> wrote:

> Signed-off-by: Matthew Weber <matthew.weber@rockwellcollins.com>

The commit log is too short. Does it fix a build issue? Why does it
need to have systemd built before? Is it using a library provided by
systemd?

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [next] package/rpcbind: add systemd dependency when cfg for systemd

[Matthew Weber]

Thomas,


On Wed, Nov 20, 2019 at 10:59 AM Thomas Petazzoni
<thomas.petazzoni@bootlin.com> wrote:
>
> On Wed, 20 Nov 2019 09:00:15 -0600
> Matt Weber <matthew.weber@rockwellcollins.com> wrote:
>
> > Signed-off-by: Matthew Weber <matthew.weber@rockwellcollins.com>
>
> The commit log is too short. Does it fix a build issue? Why does it
> need to have systemd built before? Is it using a library provided by
> systemd?

When building using top level parallel build and the init system is
set to systemd, it is possible for the rpcbind package to build before
systemd.  This is an issue as rpcbind sets a library dependency
conditionally on systemd when "RPCBIND_CONF_OPTS +=
--with-systemdsystemunitdir=/usr/lib/systemd/system" is set.

Matt

[Buildroot] [next] package/rpcbind: add systemd dependency when cfg for systemd

[Thomas Petazzoni]

On Wed, 20 Nov 2019 11:13:11 -0600
Matthew Weber <matthew.weber@collins.com> wrote:

> > The commit log is too short. Does it fix a build issue? Why does it
> > need to have systemd built before? Is it using a library provided by
> > systemd?  
> 
> When building using top level parallel build and the init system is
> set to systemd, it is possible for the rpcbind package to build before
> systemd.  This is an issue as rpcbind sets a library dependency
> conditionally on systemd when "RPCBIND_CONF_OPTS +=
> --with-systemdsystemunitdir=/usr/lib/systemd/system" is set.

So, this is really a bug, and should be fixed in master, and not in
next. Do you agree?

Without top-level parallel build, you should be able to reproduce the
same bug by running "make rpcbind" from a completely clean tree. Is it
the case ?

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [External] Re: [next] package/rpcbind: add systemd dependency when cfg for systemd

[Matthew Weber]

Thomas,

On Wed, Nov 20, 2019 at 2:16 PM Thomas Petazzoni
<thomas.petazzoni@bootlin.com> wrote:
>
> On Wed, 20 Nov 2019 11:13:11 -0600
> Matthew Weber <matthew.weber@collins.com> wrote:
>
> > > The commit log is too short. Does it fix a build issue? Why does it
> > > need to have systemd built before? Is it using a library provided by
> > > systemd?
> >
> > When building using top level parallel build and the init system is
> > set to systemd, it is possible for the rpcbind package to build before
> > systemd.  This is an issue as rpcbind sets a library dependency
> > conditionally on systemd when "RPCBIND_CONF_OPTS +=
> > --with-systemdsystemunitdir=/usr/lib/systemd/system" is set.
>
> So, this is really a bug, and should be fixed in master, and not in
> next. Do you agree?

Good point, I setup the following build now to test.

BR2_aarch64=y
BR2_TOOLCHAIN_EXTERNAL=y
BR2_INIT_SYSTEMD=y

$ make rpc-bind

checking pkg-config is at least version 0.9.0... yes
checking for SYSTEMD... no
checking for SYSTEMD... no
configure: error: libsystemd support requested but found
package/pkg-generic.mk:228: recipe for target
'/accts/mlweber1/tmp.BwI6Sa4k2G-buildroot/output/build/rpcbind-1.2.5/.stamp_configured'
failed
make[1]: *** [/accts/mlweber1/tmp.BwI6Sa4k2G-buildroot/output/build/rpcbind-1.2.5/.stamp_configured]
Error 1
Makefile:84: recipe for target '_all' failed
make: *** [_all] Error 2

Looks like it should go on master as  bugfix.

Regards,
Matt

[Buildroot] [next] package/rpcbind: add systemd dependency when cfg for systemd

[Thomas Petazzoni]

On Wed, 20 Nov 2019 09:00:15 -0600
Matt Weber <matthew.weber@rockwellcollins.com> wrote:

> Signed-off-by: Matthew Weber <matthew.weber@rockwellcollins.com>
> ---
>  package/rpcbind/rpcbind.mk | 1 +
>  1 file changed, 1 insertion(+)

As discussed in the thread, I've improved the commit log and applied to
master.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [next] package/rpcbind: add systemd dependency when cfg for systemd

[Peter Korsgaard]

>>>>> "Matt" == Matt Weber <matthew.weber@rockwellcollins.com> writes:

 > Signed-off-by: Matthew Weber <matthew.weber@rockwellcollins.com>

Committed to 2019.02.x and 2019.08.x, thanks.

-- 
Bye, Peter Korsgaard