[PATCH modular 2/3] Fix spelling.

[Gaetan Nadon]

On Tue, 2010-10-05 at 10:55 +0200, Julien Cristau wrote:

> > $ util/modular/build.sh $PREFIX --cmd "git pull"
> > 
> > Would you consider it an example of having built every
> > module/component? It isn't strictly correct anymore to say the only
> > thing build.sh can do is build items. "can only be processed" is a
> > more general and correct phrase to explain the operation of
> build.sh.
> > 
> > Although, since it's not strictly a spelling correction, the two
> > should probably be submitted as separate patches.
> 
> You can do git pull on any OS, the only thing you can't do everywhere
> is
> build the driver.
> 

There is an ambiguity here. New functions have been added, so the script
can do 
things that aren't strictly building. On other hand, this is a "build"
script. It's main purpose in life
is to "build" the software. Added functions are accessories to
"building". 

The reason why there are case statements is because the software won't
build on the specified platforms,
not because it cannot be downloaded or even configured. Using "process",
being more abstract,
can claim to be more correct, it cannot claim to be more useful.

I would stick with the current comment as I think is the one that will
be the most useful to readers.
It won't matter to 99% of users if the Sun legacy video driver can be
downloaded on a MAC.
I am in the 1% as I work mostly on configuration, so I don't care if it
builds or not on my platform.
Ideally, for me, there should be an option to by-pass the platform
check. 

I have pushed the 3 patches in the first series.
[PATCH modular 1/3] Sanity check required cmdline arguments.
[PATCH modular 2/3] Add extra check for 'prefix'.
[PATCH modular 3/3] Better cmdline error parsing.

I suggest a new series be posted which takes into account the feedback
received so far for:

[PATCH modular 1/3] Process a user-supplied list of module/components.
[PATCH modular 2/3] Fix spelling.
[PATCH modular 3/3] Enable progress on git clone.

This will make easier for reviewers.





-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20101005/dc49195b/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20101005/dc49195b/attachment.pgp>