Add TODO

NomicFoundation · Oct 5, 2022 · 98e552e · 98e552e
1 parent 6d497ab
commit 98e552e
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/packages/hardhat-core/src/internal/solidity/compiler/downloader.ts b/packages/hardhat-core/src/internal/solidity/compiler/downloader.ts
@@ -82,6 +82,12 @@ export interface ICompilerDownloader {
  */
 export class CompilerDownloader implements ICompilerDownloader {
   public static getCompilerPlatform(): CompilerPlatform {
+    // TODO: This check is seriously wrong. It doesn't take into account
+    //  the architecture nor the toolchain. This should check the triplet of
+    //  system instead (see: https://wiki.osdev.org/Target_Triplet).
+    //
+    //  The only reason this downloader works is that it validates if the
+    //  binaries actually run.
     switch (os.platform()) {
       case "win32":
         return CompilerPlatform.WINDOWS;