Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 103332 - enhanced linuxrc script for fstab and unionfs
Summary: enhanced linuxrc script for fstab and unionfs
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Hosted Projects
Classification: Unclassified
Component: genkernel (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo Genkernel Maintainers
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2005-08-22 06:22 UTC by postmodern
Modified: 2005-11-22 12:07 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
linuxrc patch (linuxrc.patch,620 bytes, patch)
2005-08-22 06:22 UTC, postmodern 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description postmodern 2005-08-22 06:22:03 UTC 
the linuxrc script supplied with sys-kernel/genkernel-3.3.5 formats /etc/fstab
improperly when handling Unionfs mounts.

Reproducible: Always
Steps to Reproduce:
1. make a livecd and have the initramfs generated with genkerenl --unionfs-dev
2.
3.

Actual Results:  
/etc/fstab is always empty

Expected Results:  
a formated /etc/fstab
Comment 1 postmodern 2005-08-22 06:22:43 UTC 
Created attachment 66539 [details, diff]
linuxrc patch
Comment 2 Chris Gianelloni (RETIRED) gentoo-dev 2005-10-28 11:35:14 UTC 
Sorry if I am just being dense (as I'm sure that I am) but I don't see a
difference between what is there before the patch and what is there afterwards.
 The sed looks the same.  What exactly is this changing?
Comment 3 postmodern 2005-10-28 12:25:17 UTC 
Before the patch cat is reading the same file that the command is outputting to.
This produces an fstab that's empty.

All I did was output to "fstab.new" and mv's it to "fstab".
Comment 4 Chris Gianelloni (RETIRED) gentoo-dev 2005-10-28 12:51:49 UTC 
OK... so I was just being dense... makes perfect sense now... thanks...

I've added this to CVS, so it should show up in the next version of genkernel...
Comment 5 Eric Edgar (RETIRED) gentoo-dev 2005-11-22 12:07:36 UTC 
 fixed in 3.3.7