Re: MAKEDEV patch

[Eduardo Pereira Habkost <ehabkost@conectiva.com.br>]

[-- Attachment #1: Type: text/plain, Size: 1294 bytes --]


Hi, Tommy,

On Fri, Jun 11, 2004 at 01:13:38PM -0700, Tommy McCabe wrote:
> This patch uses sudo to do the mknods in MAKEDEV.
> 
> 
> --- ./elkscurr/elkscmd/rootfs_template/dev/MAKEDEV
> 2004-06-11 16:01:53.000000000 -0400
> +++ ./elks/elkscmd/rootfs_template/dev/MAKEDEV
> 2004-06-11 15:54:50.000000000 -0400
> @@ -13,7 +13,7 @@
>  # Refer to the MAKESET command, also in this
> directory, for usage details.
> 
>  LINK=ln
> -MKDEV=mknod
> +MKDEV=sudo -u root mknod
>  MKSET=./MAKESET

I think that making everybody use sudo to build the disk images *and*
breaking the MAKEDEV script (that is supposed to run inside elks, too),
is not a good thing.

We should instruct the users to build the disk images as root (or better:
find a way to build the disk images without the need of root privileges,
some day), but not inserting 'sudo' everywhere on the buildsystem. We
can have users that:

- Don't like sudo
- Don't use sudo, and don't want to configure it
- Don't know how to configure sudo
- Use sudo, and have a small password timeout, and don't wat to be asked
  for the password many times during the disk images building

If you want to use sudo to build the disk images, please use
'sudo make <sometarget>', it is easier and simpler.

--
Eduardo

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]