Date: Fri, 18 Jan 2019 10:32:36 -0800 From: Cy Schubert <Cy.Schubert@cschubert.com> To: Eugene Grosbein <eugen@grosbein.net> Cc: Maxim Sobolev <sobomax@freebsd.org>, "Conrad E. Meyer" <cem@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r343118 - in head/usr.sbin: . trim Message-ID: <C2574E91-3C15-4A2A-9817-C60B5B90BDA3@cschubert.com> In-Reply-To: <5C41EC89.9000900@grosbein.net> References: <201901181254.x0ICsbBb076360@slippy.cwsent.com> <5C41EC89.9000900@grosbein.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On January 18, 2019 7:11:05 AM PST, Eugene Grosbein <eugen@grosbein=2Enet> = wrote: >On 18=2E01=2E2019 19:54, Cy Schubert wrote: > >> In message <81bb343b-ad8a-371d-d94f-f72c8dbce754@grosbein=2Enet>, >Eugene=20 >> Grosbein >> writes: >>> 18=2E01=2E2019 5:25, Maxim Sobolev wrote: >>> >>>> Using usr=2Esbin/trim as an example=2E=20 >>> >>> We still lack a command usable at installation stage (escape to >shell) to tri >>> m a device or partition >>> dedicated to ZFS Cache/ZFS Intention Log/zvol any other use when >ports/packag >>> es are not yet available=2E >>=20 >> OK, I will put a review together for dd=2E >>=20 >> Do you realize that what you just said? Your statement is >inconsistent=20 >> with where trim is installed? Then it should have been installed in=20 >> /sbin not where you put it=2E I don't want to start yet another >bikeshed=20 >> arguing the merits of this or that -- I'm intentionally being vague=20 >> here because I really don't want to stat a different bikeshed thread=2E > >> But suffice it to say, dd is the place for it because it lives in >/bin=20 >> and dd is already included in /rescue=2E > >Modern FreeBSD installation media have usr/bin and usr/sbin ready for >shell session >as they contain live system=2E I'm not against dd conv=3Dtrim but it has >different usage pattern >and it is not easy replacement for trim(8)=2E What do you not understand? We do not want it in base without a proper rev= iew=2E This second attempt at commit without a proper review is IMO disresp= ectful=2E Remove it now and submit a proper review=2E --=20 Pardon the typos and autocorrect, small keyboard in use=2E Cheers, Cy Schubert <Cy=2ESchubert@cschubert=2Ecom> FreeBSD UNIX: <cy@FreeBSD=2Eorg> Web: http://www=2EFreeBSD=2Eorg The need of the many outweighs the greed of the few=2E
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?C2574E91-3C15-4A2A-9817-C60B5B90BDA3>