Re: [OS-BUILD PATCH 0/2] Reduce disk space requirements of the kernel build

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Ondrej Mosnáček on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1950#note_1081919148

> Would it make sense to use a hardlink here instead of a symlink?

I figured a symlink would be safer, since hardlinks only work between files on
the same filesystem. It's not obvious that there is such guarantee for the
various build dirs, and even if there was, it's better to minimize such hidden
assumptions IMHO.

> My only other concern would be someone trying to debug a build failure and
finding those intermediate files gone, but I suppose they could hack up the
spec to preserve them in that particular use case.

If the build fails after the `rm`, then the kernel image is already built and
there should be no need to work with those files to debug it. And even if the
user wants to re-run the make steps for some reason, the build system should
be able to rebuild the missing files as needed. So I'd argue the impact would
be minimal here.
_______________________________________________
kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/kernel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux