Merge pull request #819 from fortran-lang/gnikit/issue813

feat(linter): improve GNU regex for modern options

Author: gnikit

CHANGELOG.md

@@ -78,6 +78,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Fixed
 
+- Fixed GFortran version regex to allow for semver + build metadata
+  ([#813](https://github.com/fortran-lang/vscode-fortran-support/issues/813))
 - Fixed broken badges and replaced them with shields.io
   ([[#815](https://github.com/fortran-lang/vscode-fortran-support/issues/815)])
 - Fixed regular expression for parsing version of GFortran in linter

src/features/linter-provider.ts

@@ -99,7 +99,7 @@ export class LinterSettings {
     // seemed to optimise away the stdout and regex would return null
     // The words between the parenthesis can have all sorts of special characters
     // account for all of them just to be safe
-    const regex = /^GNU Fortran \([\S ]+\) (?<version>.*)$/gm;
+    const regex = /^GNU Fortran \([\S ]+\) (?<msg>(?<version>\d+\.\d+\.\d+).*)$/gm;
     const output = child.stdout.toString();
     const match = regex.exec(output);
     const version = match ? match.groups.version : undefined;
@@ -108,9 +108,9 @@ export class LinterSettings {
       this.logger.info(`[lint] Found GNU Fortran version ${version}`);
       this.logger.debug(`[lint] Using Modern GNU Fortran diagnostics: ${this.modernGNU}`);
       return version;
-    } else {
-      this.logger.error(`[lint] invalid compiler version extracted ${version} from ${output}`);
     }
+    this.logger.warn(`[lint] Unable to extract semver version ${match.groups.msg} from ${output}`);
+    this.logger.warn(`[lint] Using GFortran with fallback options`);
   }
 
   public get version(): string {