Bugzilla Bug 152856

As per summary, the attached patch makes env-update actually update the
password and capability databases needed on FreeBSD. Although running those
commands is not technically needed every time env-update is called, it's
probably simpler to always run them for safety, so that users still only need
env-update if they changed settings like login.conf.

This is needed to get the stage correctly building with catalyst and latest
baselayout by the way :)

Thanks,
Diego

Created an attachment (id=100513) [details]
portage-mkdb-env.patch

Uhm maybe this is not really needed, Roy what you think, this implementation
works, but might be overkill, as nobody should change master.passwd by hand
anyway...

If you do decide that this is needed, can we make those commands ${ROOT}
friendly?  The relevant variable in that scope is target_root.

Err okay remind me not to send patches without having my coffee first. I did
put %s for adding target_root and forgot about it -_-

Created an attachment (id=100516) [details]
pkg_postinst method

IMO that's overkill.

What is needed is that we only run pwd_mkdb and cap_mkdb when we need to.
For Linux, baselayout installs /etc/passwd and the shadow ebuild runs grpconv
if a file is missing, so we can take the same approach here.

Attached is a pkg_postinst() snippet for the freebsd-usbin ebuild - as that is
the ebuild that installs pwd_mkdb and cap_mkdb.

Created an attachment (id=100519) [details]
pkg_postinst method

Better in respect to ${ROOT}

bsd team will change our ebuilds for this.

Fixored