[PATCH v3 0/7] prepare new nanddump options, defaults

Brian Norris computersforpeace at gmail.com
Fri Jun 24 13:37:29 EDT 2011


Hello,

On Thu, Jun 23, 2011 at 4:00 PM, Brian Norris
<computersforpeace at gmail.com> wrote:
> On Thu, Jun 23, 2011 at 8:52 AM, Mike Frysinger <vapier.adi at gmail.com> wrote:
>> On Thu, Jun 23, 2011 at 11:04, Artem Bityutskiy wrote:
>>> But
>>> even better we could make the version numbers to be the same as release
>>> number, then this process would be more automated and version would make
>>> some sense...
>>
>> the version isnt encoded anywhere.  so if you started using something
>> like the kernel where the VERSION was in the Makefile, we could use
>> that to create a header file which all the utils would include.
>> -mike
>
> I like Mike's idea on the VERSION thing.

I took a little look at doing this "kernel style," but I'm running
into problems. It seems like this would be a little more difficult
because of the less centralized nature of mtd-utils (in comparison
with the kernel). Plus, I'm not very good with the GNU make system and
don't wanna mess too deep into something that could easily break
various people's build systems...

Also, is it just me or is the mtd-utils build setup pretty ugly? Maybe
I'm just used to how simplistic, unified, and informative the kernel
build system is, whereas mtd-utils prints out all sorts of detailed
info at the expense of *clearly* showing what it's doing.

OK, end of complaint (for now) ; I have some tangential noobish
questions, if anyone would be kind enough to answer:

Are the "ubi-utils" and "tests" meant to have the option of compiling
separately from the other mtd-utils?
Are these tools ever distributed separately? If not, do they need to
be independently buildable? It looks to me like ubi-utils is dependent
on first compiling "lib", but otherwise, they can all be built
separately... Anyway, I think this kinda screws with the centralized
VERSION thing above.
Why is mtdinfo under ubi-utils? Perhaps it should be moved when it
replaces flash_info? I think this is a fair interpretation of Item 3
in the feature-removal plan...

Thanks for putting up with me.

Cheers,
Brian



More information about the linux-mtd mailing list