Mac: Update build scripts to handle new Profiling configuration#375
Mac: Update build scripts to handle new Profiling configuration#375nickgra merged 2 commits intomicrosoft:masterfrom
Conversation
|
|
||
| # If we're building the Profiling(Release) configuration, remove Profiling() for building .NET code | ||
| if [ "$CONFIGURATION" == "Profiling(Release)" ]; then | ||
| CONFIGURATION=Release |
There was a problem hiding this comment.
Shouldn't this be DOTNETCONFIGURATION?
There was a problem hiding this comment.
Oh I see, DOTNETCONFIGURATION hasn't been set yet. I would probably clean this up so we're deciding how to set DOTNETCONFIGURATION rather than changing CONFIGURATION
There was a problem hiding this comment.
We have to set DOTNETCONFIGURATION for the Debug/default case too so I'd rather have two separate conversions. One to reset 'Profiling(Release)' to Release and then another to unconditionally add '.Mac' to the configuration, regardless of if it's Debug/Release/malformed input.
| xcodebuild -configuration $CONFIGURATION -workspace $PROJFS/PrjFS.xcworkspace build -scheme PrjFS -derivedDataPath $ROOTDIR/BuildOutput/ProjFS.Mac/Native || exit 1 | ||
|
|
||
| # If we're building the Profiling(Release) configuration, remove Profiling() for building .NET code | ||
| if [ "$CONFIGURATION" == "Profiling(Release)" ]; then |
There was a problem hiding this comment.
Use DOTNETCONFIGURATION for consistency?
Do we need a change in the MirrorProvider scripts too?
There was a problem hiding this comment.
We do need a change in the MirrorProvider, adding that. On making that change, I discovered that the MirrorProvider's script does something different (doesn't bother making a DOTNETCONFIGURATION variable so I'll make that change in the BuildGVFSForMac script too.
2 participants
Update build scripts to be able to handle the 'Profiling(Release)' configuration. Only pass that configuration value into xcodebuild as .NET requires 'Release.Mac' to be the configuration.