[vlc-devel] [PATCH] macosx: package: use ln -n instead of -h

Jean-Baptiste Kempf jb at videolan.org
Fri Apr 17 12:22:17 CEST 2020


LGTM

On Fri, Apr 17, 2020, at 12:20, Alexandre Janniaux wrote:
> The -h option is not portable, the -n is achieving the same feature but
> is also compatible with other ln implementation.
> ---
>  extras/package/macosx/package.mak | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/extras/package/macosx/package.mak 
> b/extras/package/macosx/package.mak
> index 8d64e0568a..49ea3d7b96 100644
> --- a/extras/package/macosx/package.mak
> +++ b/extras/package/macosx/package.mak
> @@ -8,9 +8,9 @@ endif
>  # Symlink a pseudo-bundle
>  pseudo-bundle:
>  	$(MKDIR_P) $(top_builddir)/bin/Contents/Resources/
> -	$(LN_S) -hf $(abs_top_builddir)/modules/gui/macosx/UI 
> $(top_builddir)/bin/Contents/Resources/Base.lproj
> -	$(LN_S) -hf $(abs_top_builddir)/share/macosx/Info.plist 
> $(top_builddir)/bin/Contents/Info.plist
> -	$(LN_S) -hf $(CONTRIB_DIR)/Frameworks
> +	$(LN_S) -nf $(abs_top_builddir)/modules/gui/macosx/UI 
> $(top_builddir)/bin/Contents/Resources/Base.lproj
> +	$(LN_S) -nf $(abs_top_builddir)/share/macosx/Info.plist 
> $(top_builddir)/bin/Contents/Info.plist
> +	$(LN_S) -nf $(CONTRIB_DIR)/Frameworks
>  	cd $(top_builddir)/bin/Contents/Resources/ && find 
> $(abs_top_srcdir)/modules/gui/macosx/Resources/ -type f -exec $(LN_S) 
> -f {} \;
>  
>  macos-install:
> -- 
> 2.26.1
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel

-- 
Jean-Baptiste Kempf -  President
+33 672 704 734


More information about the vlc-devel mailing list