lkml.org 
[lkml]   [2007]   [May]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectMakefile question (was [RFC] LZO de/compression support - take 6)
    Date
    On 28 May 2007, at 15:34, Nitin Gupta wrote:

    > diff --git a/lib/Kconfig b/lib/Kconfig
    > index 2e7ae6b..eb95eaa 100644
    > --- a/lib/Kconfig
    > +++ b/lib/Kconfig
    > @@ -64,6 +64,12 @@ config ZLIB_INFLATE
    > config ZLIB_DEFLATE
    > tristate
    >
    > +config LZO1X_COMPRESS
    > + tristate
    > +
    > +config LZO1X_DECOMPRESS
    > + tristate
    > +
    > #
    > # Generic allocator support is selected if needed
    > #
    > diff --git a/lib/Makefile b/lib/Makefile
    > index c8c8e20..448ae37 100644
    > --- a/lib/Makefile
    > +++ b/lib/Makefile
    > @@ -49,6 +49,8 @@ obj-$(CONFIG_GENERIC_ALLOCATOR) += genalloc.o
    > obj-$(CONFIG_ZLIB_INFLATE) += zlib_inflate/
    > obj-$(CONFIG_ZLIB_DEFLATE) += zlib_deflate/
    > obj-$(CONFIG_REED_SOLOMON) += reed_solomon/
    > +obj-$(CONFIG_LZO1X_COMPRESS) += lzo1x/
    > +obj-$(CONFIG_LZO1X_DECOMPRESS) += lzo1x/
    >
    > obj-$(CONFIG_TEXTSEARCH) += textsearch.o
    > obj-$(CONFIG_TEXTSEARCH_KMP) += ts_kmp.o

    Hey there,

    Is this syntax OK for Makefile use? I'm only asking out of personal
    ignorance and because I'd use:

    Kconfig:

    config LZO1X_COMPRESS
    select LZO1X
    tristate

    config LZO1X_DECOMPRESS
    select LZO1X
    tristate

    config LZO1X
    tristate (or boolean??)

    Makefile:

    obj-$(CONFIG_LZO1X) += lzo1x/

    Thanks,

    Michael-Luke
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-05-29 10:21    [W:8.731 / U:0.304 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site