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