Page MenuHomeFreeBSD

Allow dhclient and ping to build WITHOUT_DYNAMICROOT
ClosedPublic

Authored by emaste on Sep 7 2018, 6:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 12, 8:10 PM
Unknown Object (File)
Oct 24 2024, 9:44 AM
Unknown Object (File)
Oct 4 2024, 7:25 AM
Unknown Object (File)
Oct 3 2024, 8:02 PM
Unknown Object (File)
Oct 1 2024, 4:07 PM
Unknown Object (File)
Oct 1 2024, 3:19 PM
Unknown Object (File)
Oct 1 2024, 12:13 PM
Unknown Object (File)
Sep 27 2024, 8:09 PM
Subscribers

Details

Summary

dhclient and ping normally use libcasper services. These are not available in statically-linked binaries, so when WITHOUT_DYNAMICROOT is set disable libcasper use, as with rescue builds. Also emit a warning as it's undesirable to build this way.

Reported by:    Michael Dexter (D17040)
Sponsored by:   The FreeBSD Foundation

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Sep 7 2018, 6:26 PM

It just hit me and I have to ask, why isnt libcasper avaliable in a static configuration? I certainly see a /usr/lib/libcasper.a on my systems.

I certainly see a /usr/lib/libcasper.a on my systems.

See rS325398 - it's not built any longer.

That looks good to me. Please advise if you want me to test it.

Please go ahead and test if you have time, a confirmation would be good. I will send the request to re after.

Please go ahead and test if you have time, a confirmation would be good. I will send the request to re after.

OK. Will do shortly.

This diff should address ping as well.

This diff should address ping as well.

See that. Sorry. Building.

Ed, this builds and installs for me wth both pristine sources that include only these, and my sources with my other build option changes updated to this for dhcpclient/ping.

The warnings work too:

make[5]: "/usr/srcpristine/sbin/dhclient/Makefile" line 48: warning: dhclient built without libcasper support

Looks good.

This revision was automatically updated to reflect the committed changes.

Seems D17040 did not generate a link above, so mention it again to add the xref.