Source-Changes archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
re: CVS commit: src/etc/rc.d
On Feb 24, 12:10pm, mrg%eterna.com.au@localhost (matthew green) wrote:
-- Subject: re: CVS commit: src/etc/rc.d
|
|
| Module Name: src
| Committed By: christos
| Date: Sun Feb 24 01:05:35 UTC 2008
|
| Modified Files:
| src/etc/rc.d: fsck
|
| Log Message:
| cleanup fsck error handling:
| 1. explain what each error means
| 2. remove 130 (128 + 2) which was presumably there because fsck_msdos did
| not handle SIGINT properly
|
|
| the '130' case has been there since rev 1.1 on /etc/rc -- ie,
| it comes from much older and i'm not sure removing it is the
| right answer anyway.
Now that fsck* always returns consistent error codes it should not
be there. Anyway, falling in the default case just produces a
different message and boot still stops, so there is no change in
functionality. I don't like having weird hacks in the startup files
that are there because of cargo-cult programming. Unless you know
of a reason why 130 is valid and should be there. I will gladly
document it and put it back.
christos
Home |
Main Index |
Thread Index |
Old Index