On Sun, Jan 15, 2012 at 11:57:34AM +0100, Ingo Schwarze wrote:
> Hi Jason,
> 
> Jason McIntyre wrote on Sun, Jan 15, 2012 at 08:28:29AM +0000:
> 
> > ok, so perhaps the diff below will avoid future confusion.
> 
> I agree with adding that information and don't strongly object
> to your wording, but given that fw_update(1) is just a wrapper
> around pkg_add(1), some might consider it slightly misleading:
> there is nothing wrong with using pkg_add(1) itself for updating
> firmwares, as long as you specify the right PKG_PATH and options.
> 
> See inline for an alternative wording you might consider.
> 
> Regarding afterboot(8), i think your wording is fine.
> 
> Yours,
>   Ingo
> 
> 
> > Index: usr.sbin/pkg_add/pkg_add.1
> > ===================================================================
> > RCS file: /cvs/src/usr.sbin/pkg_add/pkg_add.1,v
> > retrieving revision 1.111
> > diff -u -r1.111 pkg_add.1
> > --- usr.sbin/pkg_add/pkg_add.1      23 Aug 2011 10:32:26 -0000      1.111
> > +++ usr.sbin/pkg_add/pkg_add.1      15 Jan 2012 08:26:37 -0000
> > @@ -68,6 +68,9 @@
> >  option
> >  .Fl u
> >  .Pc .
> > +A separate utility,
> > +.Xr fw_update 1 ,
> > +is used to update non-free firmware packages.
> 
>   Since non-free firmware packages are distributed via separate
>   package repositories, the convenience wrapper
>   .Xr fw_update 1
>   is normally used to update them.
> 

hmm. i would argue that it's more rightly fw_update(8)'s place to go
into that level of detail, not pkg_add(1).

jmc

Reply via email to