Summary: | Portage should provide some means to disable dir file removal in subdirs of /usr/share/info | ||
---|---|---|---|
Product: | Portage Development | Reporter: | Ulrich Müller <ulm> |
Component: | Core - Ebuild Support | Assignee: | Portage team <dev-portage> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | CC: | emacs, pms |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: |
https://bugs.gentoo.org/show_bug.cgi?id=416875 https://bugs.gentoo.org/show_bug.cgi?id=899898 |
||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 335925 | ||
Attachments: |
Patch that is implementing the functionality
Updated patch, accepting any file matching .keepinfodir* |
Description
Ulrich Müller
2009-02-01 17:43:57 UTC
A possible solution would be use an (empty) .keepinfodir file as a marker that tells portage that the dir file should be left alone. AFAICS, a test for this would then be needed in prepinfo and in chk_updated_info_files. Created attachment 245890 [details, diff]
Patch that is implementing the functionality
(In reply to comment #2) > Created an attachment (id=245890) [details] I think the reason that keepdir() uses a filename pattern like .keep_${CATEGORY}_${PN}-${SLOT} is to avoid file collisions, and it's handy to be able to see owners of the '.keep' files in the filenames (and makes the reference count plain to see). Should we do the same for '.keepinfodir' files? (In reply to comment #3) > I think the reason that keepdir() uses a filename pattern like > .keep_${CATEGORY}_${PN}-${SLOT} is to avoid file collisions, I don't think that this is an issue here: - In cases where the ".keepinfodir" files would collide, already the "dir" files would collide. - Keeping the Info dir file makes no sense in directories where more than one package will install Info files. > and it's handy to be able to see owners of the '.keep' files in the > filenames (and makes the reference count plain to see). > Should we do the same for '.keepinfodir' files? Do we need a special command ("keepinfodir"? "skipinfodir"?) for creating the file if we use such a complicated filename pattern? (In reply to comment #4) > Do we need a special command ("keepinfodir"? "skipinfodir"?) for creating the > file if we use such a complicated filename pattern? Well now that you mention it, I notice there isn't one. I guess the alternative is for ebuilds to be aware of the '.keepinfodir' concept and to create them manually? Maybe there's some other obvious way that I've missed? Created attachment 246057 [details, diff]
Updated patch, accepting any file matching .keepinfodir*
(In reply to comment #6) > Created an attachment (id=246057) [details] > Updated patch, accepting any file matching .keepinfodir* That looks good to me. That has been included in portage-2.1.9.1. Can this bug be closed? Yes, it's fixed now. |