Fix pkg_config sdl2 string

[Damirados]

Needed to make this change for it to compile after disabling precompiled dependency.

[mat-hek]

Suggested change

	{:pkg_config, "SDL2"}
	{:pkg_config, "sdl2"}
	{:pkg_config, "SDL2"},
	{:pkg_config, "sdl2"}

I guess the .pc may be named differently on different systems/vendors. Please try leaving them both, if one fails, bundlex should try another

[Damirados]

I have tried it now, but it doesn't seem to work.

==> membrane_sdl_plugin
Bundlex: Building natives: player
warning: Bundlex: couldn't load ["SDL2"] libraries with pkg-config due to:
	** (BundlexError) pkg-config error:
	Code: 1
	Package SDL2 was not found in the pkg-config search path.
	Perhaps you should add the directory containing `SDL2.pc'
	to the PKG_CONFIG_PATH environment variable
	Package 'SDL2', required by 'virtual:world', not found

[Damirados]

Oh, it works it just emits a warning now.

[Damirados]

It is here.

[mat-hek]

Right, I guess we can't get away without it. You can switch the order so the warning will be emitted in the other case :P

[Damirados]

But it can be fixed in bundlex. I don't see a reason why to emit warning there, if all providers fail it will error anyway.

[mat-hek]

In this case yes, but when fetching a precompiled dep fails and we fall back to pkg-config, we should inform the user what's happening. Maybe we should only warn if the kind of dependency changes, like from pkg-config to precompiled and vice versa?

[Damirados]

That seems reasonable, maybe I'll find some time today and try to make PR on bundlex repo.

[Damirados]

membraneframework/bundlex#134