Frans Pop <[EMAIL PROTECTED]> writes: > On Saturday 12 August 2006 10:35, Otavio Salvador wrote: >> Does anyone has comments about it? > > (Ignoring the debug stuff which was already removed in a later version of > the patch.) > > This patch gives me the feeling that functionality is being added because > it is possible, not because it is really needed. IMO there this patch at > least adds a certain amount of redundancy. > > I can understand the desire for a specific debootstrap script, but I feel > adding support for _preseeding_ INCLUDES and EXCLUDES is redundant, > especially when targeted at CDDs: > - CDDs should be able to use the include and exclude files instead of > preseeding;
That forces them to use CD installation. > - includes and excludes may not even be needed anymore now that you can > use a completely tailored debootstrap script instead; That will force us to change debootstrap script and that can be workarounded using included/excludes most of time. > - Colin recently already added the option to preseed installation of extra > packages in pkgsel (pkgsel/include), which is a much more safe place > to install additional packages. include and excludes aren't extra packages but packages providing base functionality. For example the use of nullmailer on desktop machines instead of exim4. > For the debootstrap script, I wonder if that really needs to be > preseedable or if it would be better to have base-installer look for a > script with a fixed name in a fixed place. It is only really of use to > people who will probably already be creating their own images. I didn't see what you mean here. > IMO with all these different options to include, exclude, customize, etc. > we are starting to run the risk of making d-i a maintenance and > documentation hell and also of not being able to support people anymore > who use these options incorrectly and thus create problems for > themselves. Do we really want to make it this easy for people to mess > with the base system installation? Well. If someone uses it then he's in his own risk. IMHO we shouldn't avoid the right of him to do that. > Of course, others may disagree with me. I have no problem with that. I do. > If the option to preseed includes and excludes _is_ added, I suggest to > make it so that both comma and space separated lists are accepted (see > Colin's implementation of pkgsel/include). Sure. I can improve it besides yestarday Joey comments too :-D -- O T A V I O S A L V A D O R --------------------------------------------- E-mail: [EMAIL PROTECTED] UIN: 5906116 GNU/Linux User: 239058 GPG ID: 49A5F855 Home Page: http://www.freedom.ind.br/otavio --------------------------------------------- "Microsoft gives you Windows ... Linux gives you the whole house." -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]