Bug 17593

Summary: [PATCH] Add KERNEL identifier to GENERIC
Product: Base System Reporter: Doug <Doug>
Component: kernAssignee: Marcel Moolenaar <marcel>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: 3.4-STABLE   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff
none
file.dat none

Description Doug 2000-03-25 06:40:01 UTC
	This small patch adds a kernel identifier to the GENERIC kernel file.
This allows the user to keep their GENERIC kernel up to date as a backup
while not influencing their "regular" kernel. It also has the benefit of
moving the old kernel.GENERIC to kernel.GENERIC.old during the
'make install' step.

Fix: Apply the following patch:

How-To-Repeat: 
	Track a -stable release, have your real kernel blow up, have
kernel.GENERIC not work because it's too out of date. :)
Comment 1 Sheldon Hearn freebsd_committer freebsd_triage 2000-03-29 11:12:29 UTC
Responsible Changed
From-To: freebsd-bugs->sheldonh

I'll take this one.  If the patch on 17536 works as advertised, I'd 
prefer that solution, but we'll see. 

Comment 2 Sheldon Hearn 2000-03-30 15:06:01 UTC
Hi Doug,

Here's what I sent the originator of PR 17536 regarding additional work
on the buildkernel and installkernel targets.  I hope this explains why
I haven't gone ahead with your patch.

Ciao,
Sheldon.
Comment 3 Sheldon Hearn 2000-04-11 10:42:53 UTC
Hi Doug,

Have you had time to test whether "makeoptions KERNEL?=kernel.GENERIC"
is allowed?

Ciao,
Sheldon.
Comment 4 Sheldon Hearn freebsd_committer freebsd_triage 2000-06-22 14:08:35 UTC
Responsible Changed
From-To: sheldonh->freebsd-bugs

Divest myself of another PR so that I can focus on my only 
goal for 5.0-RELEASE. 
Sorry about this, Doug.
Comment 5 Marcel Moolenaar freebsd_committer freebsd_triage 2000-07-12 20:06:53 UTC
Responsible Changed
From-To: freebsd-bugs->marcel

My plate as well.
Comment 6 Marcel Moolenaar freebsd_committer freebsd_triage 2000-07-23 18:49:49 UTC
State Changed
From-To: open->closed

We won't support installing multiple kernels with the 
installkernel target. With this path, the default case 
(ie KERNEL=GENERIC) would not install a kernel named 
'kernel'. This is potentially harmful. The problem the 
patch tries to solve is more pilot error than anything 
else. Remember that installing a new kernel always 
leaves the old kernel under 'kernel.old'...