Skip to content

Commit

Permalink
No not do appconfig name check on mono
Browse files Browse the repository at this point in the history
Speculatively fixes #279.

The fix from #241 should not be needed on Mono (and in fact cannot work at all in its current form!).
  • Loading branch information
atruskie committed Feb 10, 2020
1 parent 9e49f83 commit ae7501e
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions src/AnalysisPrograms/Production/MainEntryUtilities.cs
Original file line number Diff line number Diff line change
Expand Up @@ -198,6 +198,13 @@ internal static void BeforeExecute(MainArgs main, CommandLineApplication applica
/// <returns><value>True</value> if modifications were made.</returns>
internal static bool CheckAndUpdateApplicationConfig()
{
if (AppConfigHelper.IsMono)
{
// this issue *should* not occur when AP.exe is invoked with a mono prefix...
Log.Verbose("Skipping app config name check for mono.");
return false;
}

// TODO: DOTNET CORE - we can expect most of this method to not work, it needs to
// tested again.
var executableName = Process.GetCurrentProcess().MainModule.ModuleName;
Expand Down

0 comments on commit ae7501e

Please sign in to comment.