<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN"> <HTML> <HEAD> <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8"> <META NAME="GENERATOR" CONTENT="GtkHTML/3.26.0"> </HEAD> <BODY> On Mon, 2010-10-04 at 06:54 -0400, Trevor Woerner wrote: <BLOCKQUOTE TYPE=CITE> <TT>From: Trevor Woerner <<A HREF="mailto:twoerner@gmail.com">twoerner@gmail.com</A>></TT> <TT>New feature. Allow the user to supply, on the cmdline, a space-separated list</TT> <TT>of module/components to process, the script will process only those listed</TT> <TT>module/components. The list can be in any arbitrary order, the script takes</TT> <TT>care to process them in their correct, dependency order. Warn the user should</TT> <TT>there be any module/components from the list which aren't processed (probably</TT> <TT>due to misspelling).</TT> <TT>Suggested usage:</TT> <TT>1. run the script, only generating the list of modules, redirected to a file</TT> <TT>$ build.sh -L > list</TT> <TT>2. edit list, removing any unneeded module/components</TT> <TT>3. run the build as follows:</TT> <TT>$ build.sh $PREFIX --modlist "`cat list`" ...</TT> </BLOCKQUOTE> Looks great. There may be some improvements to be done on the cmd line error checking. Not that the problems are introduced by the patch, but because the patch makes the usage of the cmd line less trivial than it use to be, it needs to be more robust. For example, if I invoke the script from the modular directory, with $PREFIX unset: <BLOCKQUOTE> <PRE> ./build.sh --modlist </PRE> </BLOCKQUOTE> I get: shift: 1072: can't shift that many <BLOCKQUOTE> <PRE> ./build.sh --bad </PRE> </BLOCKQUOTE> attempts to run the whole build and claims at the end that no modules were available. If I invoke the script from $PREFIX directory (as it should be): <BLOCKQUOTE> <PRE> ./build.sh --modlist </PRE> </BLOCKQUOTE> I get: shift: 1072: can't shift that many <BLOCKQUOTE> <PRE> ./build.sh --bad </PRE> </BLOCKQUOTE> attempts to run the whole build and I get: <BLOCKQUOTE> <PRE> aclocal: couldn't open directory `--bad/share/aclocal': No such file or directory </PRE> </BLOCKQUOTE> This is not going to be easy to figure out for casual users. Suggestion: it would make life easier if there was a way so provide the filename containing the module list. It's unlikely someone will be typing anything more than one module name. Gaetan </BODY> </HTML>