[PATCH 1/1] UBUNTU: link-headers -- only link directories which do not already exist
Stefan Bader
stefan.bader at canonical.com
Thu Feb 26 09:57:53 UTC 2009
Andy Whitcroft wrote:
> Bug: #315252
>
> When building the flavour specific header link-balls we ensure we have
> all of the directories linked by iterating over those. However when
> the directory already exists due to a local file within it we do not
> notice the directory and link it regardless. This leaves us with
> stuttered links on install:
>
> /usr/src/linux-headers-2.6.28-8-generic/include/linux/linux
> /usr/src/linux-headers-2.6.28-8-generic/include/asm-x86/asm-x86
>
> This occurs because we check that the destination does not already
> exist before linking, but we only check that it is not a file. Existing
> directories are not matched and are relinked. Fix this to detect both.
>
> Signed-off-by: Andy Whitcroft <apw at canonical.com>
> ---
> debian/scripts/link-headers | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/debian/scripts/link-headers b/debian/scripts/link-headers
> index a6ce50a..9448bfe 100755
> --- a/debian/scripts/link-headers
> +++ b/debian/scripts/link-headers
> @@ -19,7 +19,7 @@ while read file; do
> dir=$file
> lastdir=$file
>
> - if [ -f "$hdrdir/$file" ]; then
> + if [ -f "$hdrdir/$file" -o -d "$hdrdir/$file" ]; then
> continue
> fi
>
Could that test in theory be "-e"?
Sounds like this is something to make sure quite a way back (at least Gutsy)
--
When all other means of communication fail, try words!
More information about the kernel-team
mailing list