Page MenuHome
Differential D15891

Patches Optix SDK /include used twice
ClosedPublic
Actions

Authored by Josh Whelchel (soundofjw) on Sep 6 2022, 7:35 AM.

Details

Reviewers
Brecht Van Lommel (brecht)
Commits
rB74477149dddf: Fix T100845: wrong Cycles OptiX runtime compilation include path
Summary

This was causing kernel build fails for me on arch linux: see T100845

Diff Detail

Repository
rB Blender

Event Timeline

Josh Whelchel (soundofjw) requested review of this revision.Sep 6 2022, 7:35 AM
Josh Whelchel (soundofjw) created this revision.

see the value returned in get_optix_include_dir which already includes /include.
In my SDK (7.5 anyway) this is not a valid path.

Brecht Van Lommel (brecht) accepted this revision.Sep 6 2022, 3:45 PM
This revision is now accepted and ready to land.Sep 6 2022, 3:45 PM
Closed by commit rB74477149dddf: Fix T100845: wrong Cycles OptiX runtime compilation include path (authored by Josh Whelchel (soundofjw), committed by Brecht Van Lommel (brecht)). · Explain WhySep 6 2022, 4:23 PM
This revision was automatically updated to reflect the committed changes.
Brecht Van Lommel (brecht) added a commit: rB74477149dddf: Fix T100845: wrong Cycles OptiX runtime compilation include path.

Revision Contents
Changeset List

PathSize
intern/
cycles/
device/
optix/
2 lines

Diff 55345

View Options

intern/cycles/device/optix/device_impl.cpp

Loading...