Page MenuHomeSolus

Initial commit of shaderc
ClosedPublic

Authored by YakoYakoYokuYoku on Mar 22 2021, 2:06 AM.
Tags
None
Referenced Files
F10828700: D10741.id25840.diff
Sun, Jun 4, 10:07 AM
F10805492: D10741.diff
Mon, May 29, 8:39 PM
F10777781: D10741.id25840.diff
Wed, May 24, 12:59 AM
F10776158: D10741.id25839.diff
Tue, May 23, 7:08 AM
F10775448: D10741.id25854.diff
Mon, May 22, 7:47 PM
F10773187: D10741.id26318.diff
Sun, May 21, 6:41 PM
F10769347: D10741.id26334.diff
Sat, May 20, 3:32 PM
F10748816: D10741.id25839.diff
Mon, May 15, 10:48 PM
Tokens
"Burninate" token, awarded by aleksvor.

Details

Summary

Requirement for both gst-plugins-bad and pytorch.

Depends on D10926.

Signed-off-by: Martin Reboredo <yakoyoku@gmail.com>

Test Plan

Compiled some shaders and built against both gst-plugins-bad and pytorch.

Diff Detail

Repository
R5204 shaderc
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

aleksvor added inline comments.
package.yml
16–17

Are you sure that these are needed as rundeps? Doesn't shaderc use only static libs from -devel subpackages?

YakoYakoYokuYoku marked an inline comment as done.

Some static libraries went missing...

I thought that shaderc calls either one of them, but that wasn't the case. libplacebo runs correctly with it.

JoshStrobl added a subscriber: JoshStrobl.

Should probably use a series file and %apply_patches, you'll be at 4 patches after my requested one. https://getsol.us/articles/packaging/packaging-practices/en/#applying-a-patch

package.yml
20

What is the intention of this? If it's just to get the build version, why don't you interpolate $version into the necessary string like v${version} and echo that?

23

I'd prefer this be a patch.

This revision now requires changes to proceed.Apr 25 2021, 3:25 PM
YakoYakoYokuYoku marked 2 inline comments as done.
YakoYakoYokuYoku edited the summary of this revision. (Show Details)

Bumped glslang and patched remarked issues.

Thanks for the quick fixes, LGTM, thanks!

This revision is now accepted and ready to land.Apr 25 2021, 5:55 PM
This revision was automatically updated to reflect the committed changes.