Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 205183 - sys-apps/baselayout - /etc/init.d/bootmisc shouldn't wipe .keep* files in /var/run
Summary: sys-apps/baselayout - /etc/init.d/bootmisc shouldn't wipe .keep* files in /va...
Status: VERIFIED DUPLICATE of bug 192682
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] baselayout (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-10 12:33 UTC by Jakub Moc (RETIRED)
Modified: 2008-01-19 21:20 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jakub Moc (RETIRED) gentoo-dev 2008-01-10 12:33:32 UTC
There are created there by lots of ebuilds so that portage leaves the subdirectories alone and doesn't unmerge them, don't wipe the .keep* files in subdirs please.
Comment 1 SpanKY gentoo-dev 2008-01-14 08:41:36 UTC
i've backported `checkpath` to baselayout-1.12 for people to use

*** This bug has been marked as a duplicate of bug 192682 ***
Comment 2 Jakub Moc (RETIRED) gentoo-dev 2008-01-14 08:54:02 UTC
I'm afraid I really don't understand this solution. Every baselayout out there wipes .keep*, including 2.0.0_rc6-r1. 

# Clean up /var/run and create /var/run/utmp so we can login.
        for x in $(find /var/run ! -type d ! -name utmp ! -name innd.pid \
            ! -name random-seed ! -name ld-elf.so.hints ! -name ld.so.hints); do
            [ ! -f "${x}" ] && continue
            ...
                rm -f "${x}"
        done
Comment 3 Jakub Moc (RETIRED) gentoo-dev 2008-01-18 12:30:57 UTC
(In reply to comment #1)
> i've backported `checkpath` to baselayout-1.12 for people to use

Reopen, this is completely unrelated per discussion w/ Uberlord. bootmisc doesn't use checkpath in any way.
Comment 4 Roy Marples 2008-01-18 14:36:56 UTC
I think the argument is that per package init scripts should use checkpath to ensure the directory exists with the correct uid/gid and permissions.
Comment 5 Jakub Moc (RETIRED) gentoo-dev 2008-01-18 14:44:07 UTC
OK, so you suggest going across the entire tree looking for ebuilds that keepdir subdirectories in /var/run with specific permissions - instead of simply not nuking stuff that's not supposed to be nuked at all in the first place?

Well have fun with that, I plain don't have time for this. These ebuilds create the directories for a reason and it takes like 5 seconds to fix baselayout's bootmisc initscript to leave .keep* stuff alone.

Thanks.
Comment 6 SpanKY gentoo-dev 2008-01-19 20:37:43 UTC
then dont open bugs if you dont have time for that

*** This bug has been marked as a duplicate of bug 192682 ***
Comment 7 Jakub Moc (RETIRED) gentoo-dev 2008-01-19 20:45:27 UTC
(In reply to comment #6)
> then dont open bugs if you dont have time for that

Yup. Since for every trivial problem, there's always a complicated solution that will take months to implement and involve tens of people instead of choosing a trivial fix that would take care of this in one minute. This bug proves it once again. 

So yeah, thanks very much again, sorry for bothering you.

Comment 8 SpanKY gentoo-dev 2008-01-19 21:20:33 UTC
the choices are:
 - implement a small change in one place that doesnt account for all situations and is not correct but allows most people to go on in ignorance
 - correctly implement changes in every place that it is needed and accounts for all situations and allows for all people to go on in ignorance

i obviously choose the latter, as this bug proves once again