Retour à une version plus stable

src/fr/monlouyan/bakefile/BakeEngine.java

@@ -3,7 +3,6 @@ package fr.monlouyan.bakefile;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.Set;
 
 /**
  * Moteur principal de l'application Bake.
@@ -86,12 +85,4 @@ public class BakeEngine {
             executor.execute(rule);
         }
     }
-
-	/**
-	 * Récupère les noms de toutes les règles.
-	 * @return Un ensemble contenant les noms de toutes les règles
-	 */
-	public static Set<String> getAllRuleNames() {
-		return ruleMap.keySet();
-	}
 }

src/fr/monlouyan/bakefile/BakefileParser.java

@@ -26,10 +26,9 @@ public class BakefileParser {
 	/**
      * Regex pour détecter les targets et leurs dépendances.
      * Format : "nom1 nom2 nom3 : dépendance1 dépendance2"
-     * La nouvelle regex assure que la ligne ne commence pas par une tabulation
+     * La nouvelle regex gère plusieurs cibles séparées par des espaces
-     * et vérifie que le premier caractère non-espace n'est pas un symbole de commentaire
      */
-    private static final Pattern TARGET_PATTERN = Pattern.compile("^([^\\t:#][^:#]+?)\\s*:\\s*([^#]*?)\\s*(?:#.*)?$");
+    private static final Pattern TARGET_PATTERN = Pattern.compile("^([^:#]+?)\\s*:\\s*([^#]*?)\\s*(?:#.*)?$");
 
 	/**
      * Regex pour détecter les lignes de commande associées à une target.
@@ -95,19 +94,8 @@ public class BakefileParser {
             Matcher contMatcher = CONTINUATION_PATTERN.matcher(line);
             
             if (contMatcher.matches()) {
-				// Ajouter la ligne sans le backslash mais conserver le contenu entier
+                // Ajouter la ligne sans le backslash
-				// Ne pas ajouter d'espace après certains opérateurs comme &&
+                combinedLine.append(contMatcher.group(1).trim()).append(" ");
-				String content = contMatcher.group(1);
-				combinedLine.append(content);
-				
-				// Si la ligne ne se termine pas déjà par un opérateur tel que &&, ajouter un espace
-				if (!content.trim().endsWith("&&") && !content.trim().endsWith("|") && 
-					!content.trim().endsWith(";")) {
-					combinedLine.append(" ");
-				} else {
-					// Si elle se termine par &&, |, ou ;, ajouter juste un espace après
-					combinedLine.append(" ");
-				}
                 i++;
             } else {
                 // Ajouter la dernière ligne et sortir
@@ -121,9 +109,6 @@ public class BakefileParser {
 
 	/**
 	 * Remplacer les variables dans une chaîne.
-	 * Cette méthode remplace toutes les références de variables (${VAR} ou $(VAR))
-	 * par leur valeur. Si une variable n'est pas définie, elle est remplacée par une chaîne vide.
-	 * 
 	 * @param input Chaîne à traiter
 	 * @return Chaîne avec les variables remplacées
 	 */
@@ -131,25 +116,36 @@ public class BakefileParser {
         if (input == null) return null;
         
         String result = input;
+        Set<String> processedVars = new HashSet<>();
+        boolean changed;
         
-		// Détecter et remplacer toutes les occurrences de variables
+        do {
+            changed = false;
             Matcher matcher = VARIABLE_REFERENCE.matcher(result);
             StringBuffer sb = new StringBuffer();
             
             while (matcher.find()) {
                 String varName = matcher.group(1) != null ? matcher.group(1) : matcher.group(2);
-			// Remplacer par la valeur de la variable si elle existe, sinon par une chaîne vide
+                if (!processedVars.contains(varName) && variables.containsKey(varName)) {
-			String replacement = variables.containsKey(varName) ? variables.get(varName) : "";
+                    String replacement = variables.get(varName);
                     matcher.appendReplacement(sb, Matcher.quoteReplacement(replacement));
+                    changed = true;
+                    processedVars.add(varName);
+                }
             }
             matcher.appendTail(sb);
             result = sb.toString();
             
-		// Vérifier les références imbriquées de variables et continuer à remplacer si nécessaire
+            // Si aucun changement n'a été fait dans ce passage, arrêter
-		if (VARIABLE_REFERENCE.matcher(result).find()) {
+            if (!changed) {
-			result = replaceVariables(result); // Appel récursif pour gérer les variables imbriquées
+                break;
             }
             
+            // Réinitialiser processedVars pour le prochain passage si nécessaire
+            processedVars.clear();
+            
+        } while (changed);
+        
         return result.trim();
     }
 
@@ -205,10 +201,9 @@ public class BakefileParser {
     public List<Rule> parse() {
         List<Rule> rules = new ArrayList<>();
         Set<String> phonyTargets = new HashSet<>();
-        List<String> allTargetNames = new ArrayList<>(); // Pour suivre l'ordre des cibles
     
         if (!Files.exists(Paths.get(filename))) {
-            System.out.println("*** No targets specified and no bakefile found.  Stop.");
+            System.out.println("*** No targets specified and no makefile found.  Stop.");
             System.exit(2);
         }       
     
@@ -218,18 +213,6 @@ public class BakefileParser {
             List<String> dependencies = new ArrayList<>();
             List<String> commands = new ArrayList<>();
             
-            // Première passe : collecter toutes les cibles PHONY
-            for (String line : lines) {
-                if (line.trim().isEmpty()) continue;
-                
-                Matcher phonyMatcher = PHONY_PATTERN.matcher(line);
-                if (phonyMatcher.matches()) {
-                    String[] phonies = phonyMatcher.group(1).trim().split("\\s+");
-                    Collections.addAll(phonyTargets, phonies);
-                }
-            }
-            
-            // Deuxième passe : analyser les règles en tenant compte des PHONY
             for (int i = 0; i < lines.size(); i++) {
                 String line = lines.get(i);
                 
@@ -260,7 +243,8 @@ public class BakefileParser {
                 Matcher phonyMatcher = PHONY_PATTERN.matcher(line);
                 
                 if (phonyMatcher.matches()) {
-                    // Déjà traité dans la première passe
+                    String[] phonies = phonyMatcher.group(1).trim().split("\\s+");
+                    Collections.addAll(phonyTargets, phonies);
                     continue;
                 }
                 
@@ -283,8 +267,9 @@ public class BakefileParser {
                                 phonyTargets.contains(resolvedTarget)
                             ));
                             
-                            // Enregistrer le nom de la cible pour suivre l'ordre
+                            if (firstTarget == null && !phonyTargets.contains(resolvedTarget)) {
-                            allTargetNames.add(resolvedTarget);
+                                firstTarget = resolvedTarget;
+                            }
                         }
                     }
     
@@ -309,32 +294,15 @@ public class BakefileParser {
                         phonyTargets.contains(resolvedTarget)
                     ));
                     
-                    // Enregistrer le nom de la cible pour suivre l'ordre
+                    if (firstTarget == null && !phonyTargets.contains(resolvedTarget)) {
-                    allTargetNames.add(resolvedTarget);
+                        firstTarget = resolvedTarget;
                     }
                 }
-    
-            // Définir la première cible (similaire à Make)
-            // Make prend la première cible non-PHONY, ou la première cible si toutes sont PHONY
-            if (!allTargetNames.isEmpty()) {
-                // Chercher d'abord une cible non-PHONY dans l'ordre d'apparition
-                for (String targetName : allTargetNames) {
-                    if (!phonyTargets.contains(targetName)) {
-                        firstTarget = targetName;
-                        break;
-                    }
-                }
-                
-                // Si toutes les cibles sont PHONY, prendre simplement la première
-                if (firstTarget == null) {
-                    firstTarget = allTargetNames.get(0);
-                }
             }
     
         } catch (IOException e) {
             e.printStackTrace();
         }
-        
         return rules;
     }
 

src/fr/monlouyan/bakefile/DependencyResolver.java

@@ -79,14 +79,6 @@ public class DependencyResolver {
                 continue;
             }
             
-            // Ignorer les options de compilation qui pourraient avoir été mal interprétées comme des règles
-            if (ruleName.startsWith("-") || ruleName.contains(":")) {
-                if (debug) {
-                    System.out.println("Debug: Skipping compiler option: " + ruleName);
-                }
-                continue;
-            }
-            
             Rule rule = ruleMap.get(ruleName);
             if (rule != null) {
                 rulesToBuild.add(rule);
@@ -116,12 +108,6 @@ public class DependencyResolver {
             if (dep.startsWith("~")) {
                 continue;
             }
-            
-            // Ignorer les options de compilation qui pourraient avoir un ':' dedans
-            if (dep.startsWith("-") || dep.contains(":")) {
-                continue;
-            }
-            
             topologicalSort(dep, processed, buildOrder);
         }
         
@@ -162,11 +148,6 @@ public class DependencyResolver {
                         continue;
                     }
                     
-                    // Ignorer les options de compilation qui pourraient avoir un ':' dedans
-                    if (dependency.startsWith("-") || dependency.contains(":")) {
-                        continue;
-                    }
-                    
                     if (ruleMap.containsKey(dependency)) {
                         detectCycle(dependency, visited, stack, ruleName);
                     }

src/fr/monlouyan/bakefile/Rule.java

@@ -1,7 +1,6 @@
 package fr.monlouyan.bakefile;
 
 import java.io.File;
-import java.util.ArrayList;
 import java.util.List;
 
 /**
@@ -95,6 +94,21 @@ public class Rule {
             return false;
         }
     
+        // Vérifier d'abord toutes les dépendances avant d'exécuter quoi que ce soit
+        for (String dependency : dependencies) {
+            // Skip dependencies with tilde in path
+            if (dependency.startsWith("~")) {
+                continue;
+            }
+            
+            File depFile = new File(dependency);
+            boolean hasRule = BakeEngine.hasRule(dependency);
+            if (!depFile.exists() && !dependency.isEmpty() && !hasRule) {
+                System.out.println("bake: *** No rule to make target `" + dependency + "', needed by `" + name + "'.  Stop.");
+                System.exit(2);
+            }
+        }
+
         // Si le fichier cible n'existe pas et qu'il y a des commandes, il doit être mis à jour
         File targetFile = new File(name);
         if (!targetFile.exists() && !commands.isEmpty()) {
@@ -104,53 +118,11 @@ public class Rule {
             return true;
         }
     
-		// Vérifier d'abord toutes les dépendances avant d'exécuter quoi que ce soit
-		for (String dependency : dependencies) {
-			// Skip dependencies with tilde in path
-			if (dependency.startsWith("~")) {
-				continue;
-			}
-			
-			// Ignorer les options de compilation qui pourraient avoir un ':' dedans
-			if (dependency.startsWith("-") || dependency.contains(":")) {
-				continue;
-			}
-			
-			File depFile = new File(dependency);
-			boolean hasRule = BakeEngine.hasRule(dependency);
-			
-			if (!depFile.exists() && !dependency.isEmpty() && !hasRule) {
-				// Vérifier si on est en situation de dépendance circulaire déjà traitée
-				boolean isPartOfCircularDependency = false;
-				
-				// Vérifier si cette dépendance est impliquée dans une relation circulaire
-				for (Rule rule : getAllRules()) {
-					if (rule.getName().equals(dependency) && rule.getDependencies().contains(name)) {
-						if (BakeCLI.isDebug()) {
-							System.out.println("Debug: Found circular dependency between " + name + " and " + dependency);
-						}
-						isPartOfCircularDependency = true;
-						break;
-					}
-				}
-				
-				if (isPartOfCircularDependency) {
-					if (BakeCLI.isDebug()) {
-						System.out.println("Debug: Ignoring circular dependency: " + dependency);
-					}
-					continue;
-				}
-				
-				System.out.println("bake: *** No rule to make target `" + dependency + "', needed by `" + name + "'.  Stop.");
-				System.exit(2);
-			}
-		}
-
         // Si la règle n'a pas de commandes, on vérifie seulement si une dépendance doit être mise à jour
         if (commands.isEmpty()) {
             for (String dependency : dependencies) {
-				// Skip dependencies with tilde in path or compiler options
+                // Skip dependencies with tilde in path
-				if (dependency.startsWith("~") || dependency.startsWith("-") || dependency.contains(":")) {
+                if (dependency.startsWith("~")) {
                     continue;
                 }
                 
@@ -179,8 +151,8 @@ public class Rule {
         long currentTime = System.currentTimeMillis();
         
         for (String dependency : dependencies) {
-			// Skip dependencies with tilde in path or compiler options
+            // Skip dependencies with tilde in path
-			if (dependency.startsWith("~") || dependency.startsWith("-") || dependency.contains(":")) {
+            if (dependency.startsWith("~")) {
                 continue;
             }
             
@@ -226,19 +198,4 @@ public class Rule {
         
         return false;
     }
-
-	/**
-	 * Permet de récupérer la liste de toutes les règles
-	 * @return La liste de toutes les règles
-	 */
-	private List<Rule> getAllRules() {
-		List<Rule> allRules = new ArrayList<>();
-		for (String ruleName : BakeEngine.getAllRuleNames()) {
-			Rule rule = BakeEngine.getRule(ruleName);
-			if (rule != null) {
-				allRules.add(rule);
-			}
-		}
-		return allRules;
-	}
 }