Address review feedback

- Consolidate error logging into single line
- Add MINIMUM_GLIBC_VERSION constant
- Handle version strings without dots in comparison logic

Signed-off-by: Anubhav Singh <anubhavsingh@Anubhavs-MacBook-Air.local>
Signed-off-by: Anubhav <anumukul009@gmail.com>

Author: anumukul

app/src/main/java/org/hyperledger/besu/cli/ConfigurationOverviewBuilder.java

@@ -69,6 +69,7 @@ public class ConfigurationOverviewBuilder {
   private boolean isParallelTxProcessingEnabled = false;
   private RocksDBCLIOptions.BlobDBSettings blobDBSettings;
   private Long targetGasLimit;
+  private static final String MINIMUM_GLIBC_VERSION = "2.28";
 
   /**
    * Create a new ConfigurationOverviewBuilder.
@@ -507,18 +508,17 @@ public String build() {
     lines.add("Maximum heap size: " + normalizeSize(Runtime.getRuntime().maxMemory()));
     lines.add("OS: " + PlatformDetector.getOS());
 
-   if (SystemInfo.getCurrentPlatform() == PlatformEnum.LINUX) {
-  final String glibcVersion = PlatformDetector.getGlibc();
-  
-  
-  checkGlibcVersion(glibcVersion);
-  
-  if (glibcVersion != null) {
-    lines.add("glibc: " + glibcVersion);
-  }
+    if (SystemInfo.getCurrentPlatform() == PlatformEnum.LINUX) {
+      final String glibcVersion = PlatformDetector.getGlibc();
 
-  detectJemalloc(lines);
-}
+      checkGlibcVersion(glibcVersion);
+
+      if (glibcVersion != null) {
+        lines.add("glibc: " + glibcVersion);
+      }
+
+      detectJemalloc(lines);
+    }
 
     final HardwareAbstractionLayer hardwareInfo = new SystemInfo().getHardware();
 
@@ -569,47 +569,51 @@ private void detectJemalloc(final List<String> lines) {
             });
   }
 
-
   /**
    * Checks if the glibc version meets the minimum required version.
    *
    * @param glibcVersion the detected glibc version
    */
   private void checkGlibcVersion(final String glibcVersion) {
     if (glibcVersion == null) {
-      // Cannot determine version, log warning but allow startup
-      logger.warn("Unable to determine glibc version. Minimum required version is 2.28");
+      logger.warn(
+          "Unable to determine glibc version. Minimum required version is {}",
+          MINIMUM_GLIBC_VERSION);
       return;
     }
 
-    final String minVersion = "2.28";
-    if (!isGlibcVersionSufficient(glibcVersion, minVersion)) {
-      logger.error("Insufficient glibc version detected.");
-      logger.error("Required: {} or higher", minVersion);
-      logger.error("Found: {}", glibcVersion);
-      logger.error("Please upgrade your system's glibc to version {} or higher.", minVersion);
+    if (!isGlibcVersionSufficient(glibcVersion, MINIMUM_GLIBC_VERSION)) {
+      logger.error(
+          "Insufficient glibc version detected. Required: {} or higher, Found: {}. Please upgrade your system's glibc.",
+          MINIMUM_GLIBC_VERSION,
+          glibcVersion);
       throw new RuntimeException(
-          "Besu requires glibc version " + minVersion + " or higher. Found: " + glibcVersion);
+          "Besu requires glibc version "
+              + MINIMUM_GLIBC_VERSION
+              + " or higher. Found: "
+              + glibcVersion);
     }
   }
 
   /**
-   
    * Compares glibc version strings.
    *
    * @param current the current glibc version
    * @param required the required minimum version
    * @return true if current version is sufficient, false otherwise
    */
-
   private boolean isGlibcVersionSufficient(final String current, final String required) {
     try {
       final String[] currentParts = current.split("\\.");
       final String[] requiredParts = required.split("\\.");
 
-      for (int i = 0; i < Math.min(currentParts.length, requiredParts.length); i++) {
-        final int currentNum = Integer.parseInt(currentParts[i].trim());
-        final int requiredNum = Integer.parseInt(requiredParts[i].trim());
+      final int compareLength = Math.max(currentParts.length, requiredParts.length);
+
+      for (int i = 0; i < compareLength; i++) {
+        final int currentNum =
+            i < currentParts.length ? Integer.parseInt(currentParts[i].trim()) : 0;
+        final int requiredNum =
+            i < requiredParts.length ? Integer.parseInt(requiredParts[i].trim()) : 0;
 
         if (currentNum > requiredNum) {
           return true;
@@ -618,15 +622,13 @@ private boolean isGlibcVersionSufficient(final String current, final String requ
           return false;
         }
       }
-      return true; // Versions are equal
+      return true;
     } catch (final NumberFormatException e) {
       logger.warn("Unable to parse glibc version: {}", current, e);
-      return true; // Assume sufficient if unable to parse
+      return true;
     }
   }
 
-
-
   /**
    * Normalize gas string.<br>
    * The implemented logic is<br>