[PATCH] makedumpfile: -v shows if lzo or snappy support enabled or not

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2014/01/08 15:32:51, kexec <kexec-bounces at lists.infradead.org> wrote:
> On 01/08/14 at 01:57pm, Zhang Yanfei wrote:
> > Hello chao
> >
> > On 01/08/2014 01:40 PM, WANG Chao wrote:
> > > It would be great to have makedumpfile -v to show if lzo or snappy
> > > support is enabled or not, since --help prints too much and we have to
> > > scroll back three screens to check it out.
> > >
> > > Using "+lzo" to indicate lzo is enabled and vice versa "-lzo" for lzo is
> > > disabled. Same for snappy.
> > >
> > > For exmaple, If lzo and snappy support are enabled,
> > >
> > >     $ ./makedumpfile -v
> > >     makedumpfile: version 1.5.5 (released on 18 Dec 2013)
> > >     +lzo +snappy
> > >
> > > If both of them are disabled,
> > >
> > >     $ ./makedumpfile -v
> > >     makedumpfile: version 1.5.5 (released on 18 Dec 2013)
> > >     -lzo -snappy
> >
> > This is weird... we may even think we have lzo/snappy enabled
> > since an option is always specified with '-'.
>
> I took this idea from `vim --version`. But it seems like vim has too
> many features that there's always +xxx out there to distinguish them
> from -xxx. I think you make a good point.
>
> >
> > My idea is to have "with lzo/snappy enabled" directly appended
> > when lzo/snappy enabled.
>
> That works for me either as long as -v has the information about
> lzo/snappy is enabled or not. But to simplify the code, I prefer
> something like this:
> " lzo    enabled
>   snappy enabled"
> and also display "lzo disabled" explicitly.

I agree with this idea.

> >
> > And is it ok to add such info to the version ? I don't know.
> > kumagai may give the answer.
>
> Yeah, let's wait.

I think it's good, I'm tired of that scroll back :-(
Could you post a v2 patch ?


Thanks
Atsushi Kumagai

>
> Thanks
> WANG Chao
>
> >
> > Thanks.
> >
> > >
> > > Signed-off-by: WANG Chao <chaowang at redhat.com>
> > > ---
> > >  print_info.c | 12 +++++++++++-
> > >  1 file changed, 11 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/print_info.c b/print_info.c
> > > index 90b6cee..242b42c 100644
> > > --- a/print_info.c
> > > +++ b/print_info.c
> > > @@ -26,7 +26,17 @@ void
> > >  show_version(void)
> > >  {
> > >   MSG("makedumpfile: version " VERSION " (released on " RELEASE_DATE ")\n");
> > > - MSG("\n");
> > > +#ifdef USELZO
> > > + MSG("+lzo ");
> > > +#else
> > > + MSG("-lzo ");
> > > +#endif
> > > +#ifdef USESNAPPY
> > > + MSG("+snappy ");
> > > +#else
> > > + MSG("-snappy ");
> > > +#endif
> > > + MSG("\n\n");
> > >  }
> > >
> > >  void
> > >
> >
> >
> > --
> > Thanks.
> > Zhang Yanfei
>
> _______________________________________________
> kexec mailing list
> kexec at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/kexec



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux