On Tue, 20 Feb 2018, Geert Stappers <stapp...@stappers.nl> wrote: > On Tue, Feb 20, 2018 at 11:27:10AM +0900, Hideki Yamane wrote: >> --- a/scripts/sid >> +++ b/scripts/sid >> @@ -94,7 +95,9 @@ Status: install ok installed" >> >> "$TARGET/var/lib/dpkg/status" >> } >> >> if doing_variant fakechroot; then >> - setup_proc_fakechroot >> + setup_proc_symlink >> + elif work_on docker; then >> + setup_proc_symlink >> else >> setup_proc >> in_target /sbin/ldconfig > > It is > | if doing_variant fakechroot; then > | - setup_proc_fakechroot > | + setup_proc_symlink > that looks strange. > > Please elaborate that change. > Mostly because other modifications were _adding_ code.
As I understand it the way that both fakechroot and docker are being handled is by invoking what used to be called 'setup_proc_fakechroot'. Since that function is no longer _just_ for fakechroot it deserves a new name, so it's been renamed to 'setup_proc_symlink' (as one can see earlier in the patch) and then of course it needs to also be replaced here. Cheers, Phil. -- |)| Philip Hands [+44 (0)20 8530 9560] HANDS.COM Ltd. |-| http://www.hands.com/ http://ftp.uk.debian.org/ |(| Hugo-Klemm-Strasse 34, 21075 Hamburg, GERMANY
signature.asc
Description: PGP signature
