[vlc-devel] [PATCH] contrib: libssp: build libssp on Win32 so we can patch it to use bcrypt

Rémi Denis-Courmont remi at remlab.net
Wed Apr 8 11:43:40 CEST 2020


I mean that overriding parts, but not all, of the toolchain should not be done by default. It's just calling for problems when somebody uses a toolchain that's not exactly like yours.

Le 8 avril 2020 09:52:34 GMT+03:00, Steve Lhomme <robux4 at ycbcr.xyz> a écrit :
>On 2020-04-07 16:06, Rémi Denis-Courmont wrote:
>> Le tiistaina 7. huhtikuuta 2020, 16.11.37 EEST Steve Lhomme a écrit :
>>> diff --git a/contrib/src/libssp/rules.mak
>b/contrib/src/libssp/rules.mak
>>> new file mode 100644
>>> index 00000000000..42fed0c714b
>>> --- /dev/null
>>> +++ b/contrib/src/libssp/rules.mak
>>> @@ -0,0 +1,28 @@
>>> +# libssp
>>> +LIBSSP_VERSION := 9_2_0
>>> +LIBSSP_SVNURL :=
>>> svn://gcc.gnu.org/svn/gcc/tags/gcc_$(LIBSSP_VERSION)_release/libssp
>+
>>> +ifdef HAVE_WIN32
>>> +PKGS += libssp
>>> +endif
>> 
>> This really just a more general and abstract respin of Martin's
>question, but
>> I think overriding a toolchain library is a very bad idea.
>> 
>> Conversely, *if* it's really necessary, then it should be disabled by
>default.
>
>The move to bcrypt is needed for winstore/uwp/uap builds. For win32 it 
>really depends if we want to keep a chance to support XP or not (I vote
>
>no). We may also decide that the one from the toolchain (which is not 
>officially part of LLVM ? A package detection would be nice) is good
>enough.
>
>I'll submit the patch to move away from wincrypt, so hopefully we won't
>
>even need to patch in the future.
>
>As for "it should be disabled by default" do you mean bcrypt in libssp 
>or libssp in VLC ?
>
>>> +
>>> +$(TARBALLS)/libssp-$(LIBSSP_VERSION).tar.gz:
>>> +	rm -rf libssp-$(LIBSSP_VERSION) libssp
>>> +	$(SVN) checkout -q $(LIBSSP_SVNURL)
>>> +	rm -rf libssp/.svn
>>> +	mv libssp libssp-$(LIBSSP_VERSION)
>>> +	tar czf $@ libssp-$(LIBSSP_VERSION)
>>> +
>>> +.sum-libssp: libssp-$(LIBSSP_VERSION).tar.gz
>>> +
>>> +libssp: libssp-$(LIBSSP_VERSION).tar.gz .sum-libssp
>>> +	$(UNPACK)
>>> +	$(APPLY)
>>>
>$(SRC)/libssp/0001-favor-bcrypt-over-wincrypt-for-the-random-generator-.pat
>>> ch +	$(APPLY)
>>> $(SRC)/libssp/0002-allow-building-outside-of-the-gcc-tree.patch
>+	$(MOVE)
>>> +
>>> +.libssp: libssp
>>> +	$(RECONF)
>>> +	cd $< && $(HOSTVARS) ./configure $(HOSTCONF)
>>> +	cd $< && $(MAKE) install
>>> +	touch $@
>> 
>> 
>> -- 
>> Rémi Denis-Courmont
>> Hagalund ny stad, Nylands republik
>> 
>> 
>> 
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel
>> 
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20200408/c46d6e4e/attachment.html>


More information about the vlc-devel mailing list