Skip to content

Commit

Permalink
fixup! SConstruct : Ensure gaffer wrapper exists before calling `us…
Browse files Browse the repository at this point in the history
…dGenSchema`
  • Loading branch information
murraystevenson committed May 1, 2024
1 parent 6466ed7 commit cfd44d8
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions SConstruct
Original file line number Diff line number Diff line change
Expand Up @@ -1780,7 +1780,6 @@ for libraryName, libraryDef in libraries.items() :
if additionalFile in pythonFiles :
continue
additionalFileInstall = env.InstallAs( os.path.join( installRoot, additionalFile ), additionalFile )
env.Alias( "additionalFiles", additionalFileInstall )
env.Alias( "buildCore", additionalFileInstall )

# osl headers
Expand Down Expand Up @@ -1860,14 +1859,14 @@ for libraryName, libraryDef in libraries.items() :
# Then call `usdGenSchema` to write `generatedSchema.usda` and
# update `plugInfo.json` in place.

schemaCommandEnv = commandEnv.Clone()
schemaCommandEnv["ENV"]["PYTHONPATH"] += os.path.pathsep + schemaCommandEnv.subst( "$BUILD_DIR/python" )
subprocess.check_call(
[
shutil.which( "gaffer.cmd" if sys.platform == "win32" else "gaffer", path = env["ENV"]["PATH"] ),
"env",
"usdGenSchema.cmd" if sys.platform == "win32" else "usdGenSchema",
shutil.which( "usdGenSchema.cmd" if sys.platform == "win32" else "usdGenSchema", path = schemaCommandEnv["ENV"]["PATH"] ),
str( source[0] ), targetDir
],
env = env["ENV"]
env = schemaCommandEnv["ENV"]
)

schemaSource = os.path.join( "usdSchemas", libraryName + ".usda" )
Expand All @@ -1880,7 +1879,6 @@ for libraryName, libraryDef in libraries.items() :
schemaSource,
buildSchema
)
env.Depends( generatedSchema, "additionalFiles" )
commandEnv.Alias( "buildCore", generatedSchema )

env.Alias( "build", "buildCore" )
Expand Down

0 comments on commit cfd44d8

Please sign in to comment.