Hi, Please don't break non-root builds. I may get stabby.
:) -adrian On 26 October 2015 at 19:44, Bryan Drewery <bdrew...@freebsd.org> wrote: > On 10/13/2015 3:55 PM, Xin LI wrote: >> Author: delphij >> Date: Tue Oct 13 22:55:17 2015 >> New Revision: 289269 >> URL: https://svnweb.freebsd.org/changeset/base/289269 >> >> Log: >> Use chroot(2) instead of using prefixes for files. >> >> Previously, the code prefixes the chroot path to actual file paths to >> simulate the effect. This, however, will not work for tzset(3) which >> expects the current system have a working set of timezone data files, >> and that is not always the case. >> >> This changeset simplifies the handling of paths and use an actual >> chroot(2) call to implement the effect. >> >> PR: bin/197313 >> MFC after: 2 weeks >> >> Modified: >> head/usr.sbin/tzsetup/tzsetup.c >> >> Modified: head/usr.sbin/tzsetup/tzsetup.c >> ============================================================================== >> --- head/usr.sbin/tzsetup/tzsetup.c Tue Oct 13 21:34:54 2015 >> (r289268) >> +++ head/usr.sbin/tzsetup/tzsetup.c Tue Oct 13 22:55:17 2015 >> (r289269) >> @@ -40,6 +40,7 @@ __FBSDID("$FreeBSD$"); >> #include <stdio.h> >> #include <stdlib.h> >> #include <string.h> >> +#include <sysexits.h> >> #include <time.h> >> #include <unistd.h> >> >> @@ -944,23 +945,18 @@ main(int argc, char **argv) >> if (argc - optind > 1) >> usage(); >> >> - if (chrootenv == NULL) { > > I would think there would be a reason for not really using chroot(2) > here, such as preparing images as non-root. > > -- > Regards, > Bryan Drewery > _______________________________________________ svn-src-all@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"