* Re: Another patch
[not found] <40F3C561.2080704@xs4all.nl>
@ 2004-07-13 12:06 ` Tommy McCabe
2004-07-27 13:54 ` Miguel Bolanos
0 siblings, 1 reply; 6+ messages in thread
From: Tommy McCabe @ 2004-07-13 12:06 UTC (permalink / raw)
To: linux-8086
--- Stefan de Konink <skinkie@xs4all.nl> wrote:
> Tommy McCabe wrote:
> > 2. Remove the Conrad Parallel Port Radio Clock
> (which
> > I think is some sort of easter egg) from MAKEDEV.
>
> This is a real device, why do you think its an
> easter egg :)
>
> Stefan
>
Because A), why would a radio clock need to hook up to
a parallel port anyway, and B), name one distro that
supports such a thing.
__________________________________________________
Do You Yahoo!?
Tired of spam? Yahoo! Mail has the best spam protection around
http://mail.yahoo.com
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: Another patch
2004-07-13 12:06 ` Another patch Tommy McCabe
@ 2004-07-27 13:54 ` Miguel Bolanos
0 siblings, 0 replies; 6+ messages in thread
From: Miguel Bolanos @ 2004-07-27 13:54 UTC (permalink / raw)
To: Tommy McCabe; +Cc: linux-8086
Greetings,
On Tue, 2004-07-13 at 06:06, Tommy McCabe wrote:
> --- Stefan de Konink <skinkie@xs4all.nl> wrote:
> > Tommy McCabe wrote:
> > > 2. Remove the Conrad Parallel Port Radio Clock
> > (which
> > > I think is some sort of easter egg) from MAKEDEV.
> >
> > This is a real device, why do you think its an
> > easter egg :)
> >
> > Stefan
> >
>
> Because A), why would a radio clock need to hook up to
> a parallel port anyway, and
> B), name one distro that
> supports such a thing.
>
ELKS?
Mike.
> __________________________________________________
> Do You Yahoo!?
> Tired of spam? Yahoo! Mail has the best spam protection around
> http://mail.yahoo.com
> -
> To unsubscribe from this list: send the line "unsubscribe linux-8086" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread
* Another patch
@ 2004-07-12 21:41 Tommy McCabe
2004-07-27 13:54 ` Miguel Bolanos
0 siblings, 1 reply; 6+ messages in thread
From: Tommy McCabe @ 2004-07-12 21:41 UTC (permalink / raw)
To: linux-8086
[-- Attachment #1: Type: text/plain, Size: 657 bytes --]
This patch:
1. Has eight partitions per hard disk instead of 64,
to save indoes and not flood the disk images. This
time I didn't change the commented-out stuff like the
secondary IDE channel.
2. Remove the Conrad Parallel Port Radio Clock (which
I think is some sort of easter egg) from MAKEDEV.
3. Fixed the "make clean" process.
4. Got rid of the sudos in the Makefile.
5. Got rid of the ridiculous business with a ROOTDIR
that didn't exist and commands that weren't referenced
anywhere in the Makefile.
__________________________________
Do you Yahoo!?
Read only the mail you want - Yahoo! Mail SpamGuard.
http://promotions.yahoo.com/new_mail
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patches.diff --]
[-- Type: text/x-diff; name="patches.diff", Size: 5397 bytes --]
diff -ruN ./elkscomp/elkscmd/Make.rules ./elkscurr/elkscmd/Make.rules
--- ./elkscomp/elkscmd/Make.rules 2002-05-29 14:23:03.000000000 -0400
+++ ./elkscurr/elkscmd/Make.rules 2004-07-12 17:22:55.000000000 -0400
@@ -68,8 +68,6 @@
#
# Directory to create the target tree in.
-ROOTDIR=$(BASEDIR)/RootFS
-
###############################################################################
#
# Command to copy the tree across.
diff -ruN ./elkscomp/elkscmd/Makefile ./elkscurr/elkscmd/Makefile
--- ./elkscomp/elkscmd/Makefile 2004-06-09 12:25:40.000000000 -0400
+++ ./elkscurr/elkscmd/Makefile 2004-07-12 17:27:44.000000000 -0400
@@ -46,12 +46,7 @@
$(FULL3_TARGET_FS) $(FULL5_TARGET_FS) || true
rm -f $(COMB_TARGET_FS) $(ROOT_TARGET_FS) $(ROOT_NET_TARGET_FS) \
$(FULL3_TARGET_FS) $(FULL5_TARGET_FS) boot images.zip core
- rm -rf rootfs_template $(ROOTDIR)
@for i in $(DIRS); do make -C $$i clean ; done
- -echo FIXME: make clean is temporarily broken, because there is no
- -echo rootfs_template.tar anymore
- tar xf rootfs_template.tar
-
###############################################################################
#
# Include standard packaging commands.
@@ -84,30 +79,6 @@
###############################################################################
#
-# Create standard packages.
-
-basepkg:
- rm -fr $(ROOTDIR)
- mkdir $(ROOTDIR)
- make mkbase
-
-netpkg:
- rm -fr $(ROOTDIR)
- mkdir $(ROOTDIR)
- make mknet
-
-stdpkg:
- rm -fr $(ROOTDIR)
- mkdir $(ROOTDIR)
- make mkstd
-
-maxpkg:
- rm -fr $(ROOTDIR)
- mkdir $(ROOTDIR)
- make mkmax
-
-###############################################################################
-#
# Create relevant parts of ELKS package.
$(ELKS_DIR)/.config:
@@ -154,42 +125,36 @@
mount_combfs: $(COMB_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
$(MKFS) $(MKFS_OPTS) $(COMB_TARGET_FS) $(COMB_TARGET_BLKS)
- sudo -u root mount $(COMB_TARGET_FS) $(TARGET_MNT) $(LOOP)
(cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
(cd $(TARGET_MNT) ; tar xpvf -)
(cd $(TARGET_MNT)/dev ; ./MAKEDEV)
mount_full3fs: $(FULL3_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
$(MKFS) $(MKFS_OPTS) $(FULL3_TARGET_FS) $(FULL3_TARGET_BLKS)
- sudo -u root mount $(FULL3_TARGET_FS) $(TARGET_MNT) $(LOOP)
(cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
(cd $(TARGET_MNT) ; tar xpvf -)
(cd $(TARGET_MNT)/dev ; ./MAKEDEV)
mount_full5fs: $(FULL5_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
$(MKFS) $(MKFS_OPTS) $(FULL5_TARGET_FS) $(FULL5_TARGET_BLKS)
- sudo -u root mount $(FULL5_TARGET_FS) $(TARGET_MNT) $(LOOP)
(cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
(cd $(TARGET_MNT) ; tar xpvf -)
(cd $(TARGET_MNT)/dev ; ./MAKEDEV)
mount_rootfs: $(ROOT_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
$(MKFS) $(MKFS_OPTS) $(ROOT_TARGET_FS) $(ROOT_TARGET_BLKS)
- sudo -u root mount $(ROOT_TARGET_FS) $(TARGET_MNT) $(LOOP)
(cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
(cd $(TARGET_MNT) ; tar xpvf -)
(cd $(TARGET_MNT)/dev ; ./MAKEDEV)
mount_rootnetfs: $(ROOT_NET_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
$(MKFS) $(MKFS_OPTS) $(ROOT_NET_TARGET_FS) $(ROOT_NET_TARGET_BLKS)
- sudo -u root mount $(ROOT_NET_TARGET_FS) $(TARGET_MNT) $(LOOP)
(cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
(cd $(TARGET_MNT) ; tar xpvf -)
(cd $(TARGET_MNT)/dev ; ./MAKEDEV)
mount_sibofs: $(SIBO_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
$(MKFS) $(MKFS_OPTS) $(SIBO_TARGET_FS) $(SIBO_TARGET_BLKS)
- sudo -u root mount $(SIBO_TARGET_FS) $(TARGET_MNT) $(LOOP)
(cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
(cd $(TARGET_MNT) ; tar xpvf -)
(cd $(TARGET_MNT) ; rm -rf home boot root var)
diff -ruN ./elkscomp/elkscmd/rootfs_template/dev/MAKEDEV ./elkscurr/elkscmd/rootfs_template/dev/MAKEDEV
--- ./elkscomp/elkscmd/rootfs_template/dev/MAKEDEV 2004-06-03 23:07:55.000000000 -0400
+++ ./elkscurr/elkscmd/rootfs_template/dev/MAKEDEV 2004-07-12 17:33:09.000000000 -0400
@@ -97,8 +97,8 @@
##############################################################################
# Direct IDE disks, Primary channel.
- $MKSET 0 63 $MKDEV hda b 5 # Currently.
- $MKSET 64 63 $MKDEV hdb b 5 # Currently.
+ $MKSET 0 7 $MKDEV hda b 5 # Currently.
+ $MKSET 8 7 $MKDEV hdb b 5 # Currently.
# $MKSET 0 63 $MKDEV hda b 3 # Ought to be.
# $MKSET 64 63 $MKDEV hdb b 3 # Ought to be.
@@ -189,10 +189,10 @@
##############################################################################
# BIOS hard disks.
- $MKSET 0 63 $MKDEV bda b 3 # Currently
- $MKSET 64 63 $MKDEV bdb b 3 # Currently
- $MKSET 128 63 $MKDEV bdc b 3 # Currently
- $MKSET 192 63 $MKDEV bdd b 3 # Currently
+ $MKSET 0 7 $MKDEV bda b 3 # Currently
+ $MKSET 8 7 $MKDEV bdb b 3 # Currently
+ $MKSET 16 7 $MKDEV bdc b 3 # Currently
+ $MKSET 24 7 $MKDEV bdd b 3 # Currently
# $MKSET 0 63 $MKDEV bda b 14 # Ought to be
# $MKSET 64 63 $MKDEV bdb b 14 # Ought to be
@@ -256,9 +256,4 @@
# $MKSET 0 7 $MKDEV i2c c 89
##############################################################################
-# Conrad Electronic Parallel Port Radio Clocks. These are not yet supported.
-
-# $MKSET 0 1 $MKDEV pcfclock c 181
-
-##############################################################################
# EOF.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Another patch
2004-07-12 21:41 Tommy McCabe
@ 2004-07-27 13:54 ` Miguel Bolanos
2004-07-27 15:15 ` Eduardo Pereira Habkost
0 siblings, 1 reply; 6+ messages in thread
From: Miguel Bolanos @ 2004-07-27 13:54 UTC (permalink / raw)
To: Tommy McCabe; +Cc: linux-8086
Greetings,
On Mon, 2004-07-12 at 15:41, Tommy McCabe wrote:
> This patch:
>
> 1. Has eight partitions per hard disk instead of 64,
> to save indoes and not flood the disk images. This
> time I didn't change the commented-out stuff like the
> secondary IDE channel.
I would like to hear out some votes about this before actually applying
it.. in my personal opinion it should go, but perhaps anyone have a
different opinion? (Allan? Harry? Claudio? ... ?)
>
> 2. Remove the Conrad Parallel Port Radio Clock (which
> I think is some sort of easter egg) from MAKEDEV.
>
sort of an easter egg... interesting definition.
Honestly i can't just a feature with that justification.
> 3. Fixed the "make clean" process.
>
Applied the rootfs_template.tar fix.
> 4. Got rid of the sudos in the Makefile.
>
Definitly applied, BUT.. keeping the mount commands.
> 5. Got rid of the ridiculous business with a ROOTDIR
> that didn't exist and commands that weren't referenced
> anywhere in the Makefile.
>
NOT applied.
best wishes
Mike
>
>
> __________________________________
> Do you Yahoo!?
> Read only the mail you want - Yahoo! Mail SpamGuard.
> http://promotions.yahoo.com/new_mail
>
> ______________________________________________________________________
> diff -ruN ./elkscomp/elkscmd/Make.rules ./elkscurr/elkscmd/Make.rules
> --- ./elkscomp/elkscmd/Make.rules 2002-05-29 14:23:03.000000000 -0400
> +++ ./elkscurr/elkscmd/Make.rules 2004-07-12 17:22:55.000000000 -0400
> @@ -68,8 +68,6 @@
> #
> # Directory to create the target tree in.
>
> -ROOTDIR=$(BASEDIR)/RootFS
> -
> ###############################################################################
> #
> # Command to copy the tree across.
> diff -ruN ./elkscomp/elkscmd/Makefile ./elkscurr/elkscmd/Makefile
> --- ./elkscomp/elkscmd/Makefile 2004-06-09 12:25:40.000000000 -0400
> +++ ./elkscurr/elkscmd/Makefile 2004-07-12 17:27:44.000000000 -0400
> @@ -46,12 +46,7 @@
> $(FULL3_TARGET_FS) $(FULL5_TARGET_FS) || true
> rm -f $(COMB_TARGET_FS) $(ROOT_TARGET_FS) $(ROOT_NET_TARGET_FS) \
> $(FULL3_TARGET_FS) $(FULL5_TARGET_FS) boot images.zip core
> - rm -rf rootfs_template $(ROOTDIR)
> @for i in $(DIRS); do make -C $$i clean ; done
> - -echo FIXME: make clean is temporarily broken, because there is no
> - -echo rootfs_template.tar anymore
> - tar xf rootfs_template.tar
> -
> ###############################################################################
> #
> # Include standard packaging commands.
> @@ -84,30 +79,6 @@
>
> ###############################################################################
> #
> -# Create standard packages.
> -
> -basepkg:
> - rm -fr $(ROOTDIR)
> - mkdir $(ROOTDIR)
> - make mkbase
> -
> -netpkg:
> - rm -fr $(ROOTDIR)
> - mkdir $(ROOTDIR)
> - make mknet
> -
> -stdpkg:
> - rm -fr $(ROOTDIR)
> - mkdir $(ROOTDIR)
> - make mkstd
> -
> -maxpkg:
> - rm -fr $(ROOTDIR)
> - mkdir $(ROOTDIR)
> - make mkmax
> -
> -###############################################################################
> -#
> # Create relevant parts of ELKS package.
>
> $(ELKS_DIR)/.config:
> @@ -154,42 +125,36 @@
>
> mount_combfs: $(COMB_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
> $(MKFS) $(MKFS_OPTS) $(COMB_TARGET_FS) $(COMB_TARGET_BLKS)
> - sudo -u root mount $(COMB_TARGET_FS) $(TARGET_MNT) $(LOOP)
> (cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
> (cd $(TARGET_MNT) ; tar xpvf -)
> (cd $(TARGET_MNT)/dev ; ./MAKEDEV)
>
> mount_full3fs: $(FULL3_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
> $(MKFS) $(MKFS_OPTS) $(FULL3_TARGET_FS) $(FULL3_TARGET_BLKS)
> - sudo -u root mount $(FULL3_TARGET_FS) $(TARGET_MNT) $(LOOP)
> (cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
> (cd $(TARGET_MNT) ; tar xpvf -)
> (cd $(TARGET_MNT)/dev ; ./MAKEDEV)
>
> mount_full5fs: $(FULL5_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
> $(MKFS) $(MKFS_OPTS) $(FULL5_TARGET_FS) $(FULL5_TARGET_BLKS)
> - sudo -u root mount $(FULL5_TARGET_FS) $(TARGET_MNT) $(LOOP)
> (cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
> (cd $(TARGET_MNT) ; tar xpvf -)
> (cd $(TARGET_MNT)/dev ; ./MAKEDEV)
>
> mount_rootfs: $(ROOT_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
> $(MKFS) $(MKFS_OPTS) $(ROOT_TARGET_FS) $(ROOT_TARGET_BLKS)
> - sudo -u root mount $(ROOT_TARGET_FS) $(TARGET_MNT) $(LOOP)
> (cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
> (cd $(TARGET_MNT) ; tar xpvf -)
> (cd $(TARGET_MNT)/dev ; ./MAKEDEV)
>
> mount_rootnetfs: $(ROOT_NET_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
> $(MKFS) $(MKFS_OPTS) $(ROOT_NET_TARGET_FS) $(ROOT_NET_TARGET_BLKS)
> - sudo -u root mount $(ROOT_NET_TARGET_FS) $(TARGET_MNT) $(LOOP)
> (cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
> (cd $(TARGET_MNT) ; tar xpvf -)
> (cd $(TARGET_MNT)/dev ; ./MAKEDEV)
>
> mount_sibofs: $(SIBO_TARGET_FS) $(ELKSCMD_DIR)/rootfs_template/etc/issue
> $(MKFS) $(MKFS_OPTS) $(SIBO_TARGET_FS) $(SIBO_TARGET_BLKS)
> - sudo -u root mount $(SIBO_TARGET_FS) $(TARGET_MNT) $(LOOP)
> (cd $(ELKSCMD_DIR)/rootfs_template ; tar cf - --exclude CVS *) | \
> (cd $(TARGET_MNT) ; tar xpvf -)
> (cd $(TARGET_MNT) ; rm -rf home boot root var)
> diff -ruN ./elkscomp/elkscmd/rootfs_template/dev/MAKEDEV ./elkscurr/elkscmd/rootfs_template/dev/MAKEDEV
> --- ./elkscomp/elkscmd/rootfs_template/dev/MAKEDEV 2004-06-03 23:07:55.000000000 -0400
> +++ ./elkscurr/elkscmd/rootfs_template/dev/MAKEDEV 2004-07-12 17:33:09.000000000 -0400
> @@ -97,8 +97,8 @@
> ##############################################################################
> # Direct IDE disks, Primary channel.
>
> - $MKSET 0 63 $MKDEV hda b 5 # Currently.
> - $MKSET 64 63 $MKDEV hdb b 5 # Currently.
> + $MKSET 0 7 $MKDEV hda b 5 # Currently.
> + $MKSET 8 7 $MKDEV hdb b 5 # Currently.
>
> # $MKSET 0 63 $MKDEV hda b 3 # Ought to be.
> # $MKSET 64 63 $MKDEV hdb b 3 # Ought to be.
> @@ -189,10 +189,10 @@
> ##############################################################################
> # BIOS hard disks.
>
> - $MKSET 0 63 $MKDEV bda b 3 # Currently
> - $MKSET 64 63 $MKDEV bdb b 3 # Currently
> - $MKSET 128 63 $MKDEV bdc b 3 # Currently
> - $MKSET 192 63 $MKDEV bdd b 3 # Currently
> + $MKSET 0 7 $MKDEV bda b 3 # Currently
> + $MKSET 8 7 $MKDEV bdb b 3 # Currently
> + $MKSET 16 7 $MKDEV bdc b 3 # Currently
> + $MKSET 24 7 $MKDEV bdd b 3 # Currently
>
> # $MKSET 0 63 $MKDEV bda b 14 # Ought to be
> # $MKSET 64 63 $MKDEV bdb b 14 # Ought to be
> @@ -256,9 +256,4 @@
> # $MKSET 0 7 $MKDEV i2c c 89
>
> ##############################################################################
> -# Conrad Electronic Parallel Port Radio Clocks. These are not yet supported.
> -
> -# $MKSET 0 1 $MKDEV pcfclock c 181
> -
> -##############################################################################
> # EOF.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Another patch
2004-07-27 13:54 ` Miguel Bolanos
@ 2004-07-27 15:15 ` Eduardo Pereira Habkost
2004-07-27 16:09 ` Tommy McCabe
0 siblings, 1 reply; 6+ messages in thread
From: Eduardo Pereira Habkost @ 2004-07-27 15:15 UTC (permalink / raw)
To: linux-8086
[-- Attachment #1: Type: text/plain, Size: 2024 bytes --]
On Tue, Jul 27, 2004 at 07:54:25AM -0600, Miguel Bolanos wrote:
> Greetings,
>
> On Mon, 2004-07-12 at 15:41, Tommy McCabe wrote:
> > This patch:
> >
> > 1. Has eight partitions per hard disk instead of 64,
> > to save indoes and not flood the disk images. This
> > time I didn't change the commented-out stuff like the
> > secondary IDE channel.
>
> I would like to hear out some votes about this before actually applying
> it.. in my personal opinion it should go, but perhaps anyone have a
> different opinion? (Allan? Harry? Claudio? ... ?)
The patch breaks the major/minor numbers on MAKEDEV.
>
> >
> > 2. Remove the Conrad Parallel Port Radio Clock (which
> > I think is some sort of easter egg) from MAKEDEV.
> >
>
> sort of an easter egg... interesting definition.
> Honestly i can't just a feature with that justification.
I agree. We can comment it, as the device is not used. Our problem is
that MAKEDEV makes too many devices by default and waste inodes on
the root disk. Comment out devices that are not obviously used, can
help. Maybe we can have a "--few-inodes" option or something like that,
for building the root disks.
>
> > 3. Fixed the "make clean" process.
> >
>
> Applied the rootfs_template.tar fix.
Did you removed the warnings? The warning is there to tell the user that
make clean is a little broken (it doesn't restore the previous state of
rootfs_template), so we need to fix that, not remove the warning.
>
> > 4. Got rid of the sudos in the Makefile.
> >
>
> Definitly applied, BUT.. keeping the mount commands.
Of course.
>
> > 5. Got rid of the ridiculous business with a ROOTDIR
> > that didn't exist and commands that weren't referenced
> > anywhere in the Makefile.
> >
Huh? I didn't get your point, Tommy. Please detail your changes, and
explain the real reason, not just because you think it looks better,
or it is "ridiculous".
>
> NOT applied.
>
> best wishes
>
> Mike
>
--
Eduardo
[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Another patch
2004-07-27 15:15 ` Eduardo Pereira Habkost
@ 2004-07-27 16:09 ` Tommy McCabe
0 siblings, 0 replies; 6+ messages in thread
From: Tommy McCabe @ 2004-07-27 16:09 UTC (permalink / raw)
To: linux-8086
--- Eduardo Pereira Habkost
<ehabkost@conectiva.com.br> wrote:
> On Tue, Jul 27, 2004 at 07:54:25AM -0600, Miguel
> Bolanos wrote:
> > Greetings,
> >
> > On Mon, 2004-07-12 at 15:41, Tommy McCabe wrote:
> > > This patch:
> > >
> > > 1. Has eight partitions per hard disk instead of
> 64,
> > > to save indoes and not flood the disk images.
> This
> > > time I didn't change the commented-out stuff
> like the
> > > secondary IDE channel.
> >
> > I would like to hear out some votes about this
> before actually applying
> > it.. in my personal opinion it should go, but
> perhaps anyone have a
> > different opinion? (Allan? Harry? Claudio? ... ?)
>
> The patch breaks the major/minor numbers on MAKEDEV.
I don't know what a major-minor number is, but the
number I changed were arguments to MAKESET that said
how many devices to make. Besides, I tested it myself
and the hard disk drivers don't even notice.
> >
> > >
> > > 2. Remove the Conrad Parallel Port Radio Clock
> (which
> > > I think is some sort of easter egg) from
> MAKEDEV.
> > >
> >
> > sort of an easter egg... interesting definition.
> > Honestly i can't just a feature with that
> justification.
>
> I agree. We can comment it, as the device is not
> used. Our problem is
> that MAKEDEV makes too many devices by default and
> waste inodes on
> the root disk. Comment out devices that are not
> obviously used, can
> help. Maybe we can have a "--few-inodes" option or
> something like that,
> for building the root disks.
>
> >
> > > 3. Fixed the "make clean" process.
> > >
> >
> > Applied the rootfs_template.tar fix.
>
> Did you removed the warnings? The warning is there
> to tell the user that
> make clean is a little broken (it doesn't restore
> the previous state of
> rootfs_template), so we need to fix that, not remove
> the warning.
The patched Makefile doesn't alter rootfs_template at
all during make clean; but unless someone deliberately
modified what was in there it wouldn't be altered in
the first place.
> >
> > > 4. Got rid of the sudos in the Makefile.
> > >
> >
> > Definitly applied, BUT.. keeping the mount
> commands.
>
> Of course.
>
> >
> > > 5. Got rid of the ridiculous business with a
> ROOTDIR
> > > that didn't exist and commands that weren't
> referenced
> > > anywhere in the Makefile.
> > >
>
> Huh? I didn't get your point, Tommy. Please detail
> your changes, and
> explain the real reason, not just because you think
> it looks better,
> or it is "ridiculous".
The lines between 90-107 of the Makefile reference
commands that are not used anywhere else, and thus are
useless. They do, however, reference the variable
ROOTDIR, which is defined in Make.rules as a directory
that doesn't exist. So, I got rid of ROOTDIR and the
commands which aren't used.
> >
> > NOT applied.
> >
> > best wishes
> >
> > Mike
> >
>
> --
> Eduardo
>
> ATTACHMENT part 2 application/pgp-signature
__________________________________
Do you Yahoo!?
Yahoo! Mail is new and improved - Check it out!
http://promotions.yahoo.com/new_mail
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2004-07-27 16:09 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <40F3C561.2080704@xs4all.nl>
2004-07-13 12:06 ` Another patch Tommy McCabe
2004-07-27 13:54 ` Miguel Bolanos
2004-07-12 21:41 Tommy McCabe
2004-07-27 13:54 ` Miguel Bolanos
2004-07-27 15:15 ` Eduardo Pereira Habkost
2004-07-27 16:09 ` Tommy McCabe
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox