threads / patch / 38509

patch, 2 partsMakefile: Also add a symlink fallback to installing ALL_PROGRAMS

Subject: [PATCH 2/2] Makefile: Also add a symlink fallback to installing ALL_PROGRAMS

## tl;dr

2 messages between Feb 5, 2015 and Feb 5, 2015. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Sebastian Schuberth· Feb 5, 2015, 15:52 UTC · lore

We do this for BUILT_INS and REMOTE_CURL_ALIASES, so we should do so here, too.

Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
---
 Makefile | 1 +
 1 file changed, 1 insertion(+)

Show changes to Makefile +1 −0

diff --git a/Makefile b/Makefile
index 21f23cb..d2849c3 100644
--- a/Makefile
+++ b/Makefile
@@ -2258,6 +2258,7 @@ endif
         $(RM) "$$execdir/$$p" && \
         test -z "$(NO_INSTALL_HARDLINKS)$(NO_CROSS_DIRECTORY_HARDLINKS)" && \
         ln "$$bindir/$$p" "$$execdir/$$p" 2>/dev/null || \
+        ln -s "$$bindir/$$p" "$$execdir/$$p" 2>/dev/null || \
         cp "$$bindir/$$p" "$$execdir/$$p" || exit; \
       done; \
     } && \

-- 
2.1.2-mingw-1

Kyle J. McKay· Feb 5, 2015, 17:49 UTC · re: Sebastian Schuberth · lore

Re: [PATCH 2/2] Makefile: Also add a symlink fallback to installing ALL_PROGRAMS

On Feb 5, 2015, at 07:52, Sebastian Schuberth wrote:

Show 19 quoted lines

> We do this for BUILT_INS and REMOTE_CURL_ALIASES, so we should do so  
> here,
> too.
>
> Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
> ---
> Makefile | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Makefile b/Makefile
> index 21f23cb..d2849c3 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -2258,6 +2258,7 @@ endif
>         $(RM) "$$execdir/$$p" && \
>         test -z "$(NO_INSTALL_HARDLINKS)$ 
> (NO_CROSS_DIRECTORY_HARDLINKS)" && \
>         ln "$$bindir/$$p" "$$execdir/$$p" 2>/dev/null || \
> +        ln -s "$$bindir/$$p" "$$execdir/$$p" 2>/dev/null || \

Can we not start installing absolute path symlinks please.

-Kyle