Ajout de la détection des timestamps futurs dans le système de build et mise à jour des messages d'avertissement
This commit is contained in:
stiti
parent
2edb7a3ad3
commit
ef5fff565a
src/fr/monlouyan/bakefile
tests/C/test-22-future-timestamp
@ -7,6 +7,7 @@ public class CommandExecutor {
|
||||
private boolean debug;
|
||||
private boolean needsUpdate = false; // Pour tracker si quelque chose doit être mis à jour
|
||||
private boolean isCircular = false; // Pour tracker si un cycle a été détecté
|
||||
private boolean futureTimestampDetected = false; // Pour détecter les timestamps dans le futur
|
||||
|
||||
public CommandExecutor(boolean debug, boolean isCircular) {
|
||||
this.debug = debug;
|
||||
@ -18,6 +19,18 @@ public class CommandExecutor {
|
||||
boolean ruleNeedsUpdate = rule.needsUpdate();
|
||||
if (ruleNeedsUpdate) {
|
||||
needsUpdate = true; // Au moins une règle doit être mise à jour
|
||||
|
||||
// Vérifier les timestamps des dépendances pour détecter ceux dans le futur
|
||||
for (String dependency : rule.getDependencies()) {
|
||||
if (dependency.startsWith("~")) {
|
||||
continue;
|
||||
}
|
||||
|
||||
File depFile = new File(dependency);
|
||||
if (depFile.exists() && TimestampManager.getTimestamp(depFile) > System.currentTimeMillis()) {
|
||||
futureTimestampDetected = true;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// Si la règle a besoin d'être mise à jour et qu'il n'y a pas de commandes
|
||||
@ -56,6 +69,11 @@ public class CommandExecutor {
|
||||
}
|
||||
}
|
||||
|
||||
// Si on a détecté des timestamps dans le futur, afficher un avertissement à la fin (comme make)
|
||||
if (futureTimestampDetected && !isCircular) {
|
||||
System.out.println("bake: warning: Clock skew detected. Your build may be incomplete.");
|
||||
}
|
||||
|
||||
// Vérifier si cette règle est une cible directement demandée par l'utilisateur
|
||||
boolean isRequestedTarget = BakeCLI.getTargets().contains(rule.getName()) ||
|
||||
(BakeCLI.getTargets().isEmpty() && rule.getName().equals(BakefileParser.getFirstTarget()));
|
||||
|
||||
@ -2,6 +2,7 @@ package fr.monlouyan.bakefile;
|
||||
|
||||
import java.io.File;
|
||||
import java.util.List;
|
||||
import java.util.Date;
|
||||
|
||||
public class Rule {
|
||||
private String name;
|
||||
@ -97,6 +98,8 @@ public class Rule {
|
||||
System.out.println("Debug : Target file '" + name + "' last modified at " + TimestampManager.formatTimestamp(targetTimestamp));
|
||||
}
|
||||
|
||||
long currentTime = System.currentTimeMillis();
|
||||
|
||||
for (String dependency : dependencies) {
|
||||
// Skip dependencies with tilde in path
|
||||
if (dependency.startsWith("~")) {
|
||||
@ -104,7 +107,22 @@ public class Rule {
|
||||
}
|
||||
|
||||
File depFile = new File(dependency);
|
||||
long depTimestamp = depFile.exists() ? TimestampManager.getTimestamp(depFile) : 0;
|
||||
if (!depFile.exists()) {
|
||||
continue;
|
||||
}
|
||||
|
||||
long depTimestamp = TimestampManager.getTimestamp(depFile);
|
||||
|
||||
// Vérifier si le timestamp de la dépendance est dans le futur
|
||||
if (depTimestamp > currentTime) {
|
||||
// Avertissement similaire à make
|
||||
System.out.println("bake: Warning: File '" + dependency + "' has modification time "
|
||||
+ ((depTimestamp - currentTime) / 1000) + " s in the future");
|
||||
if (BakeCLI.isDebug()) {
|
||||
System.out.println("Debug : Dependency " + dependency + " has a timestamp in the future, needs update");
|
||||
}
|
||||
return true;
|
||||
}
|
||||
|
||||
if (BakeCLI.isDebug()) {
|
||||
System.out.println("Debug : Dependency '" + dependency + "' last modified at " + TimestampManager.formatTimestamp(depTimestamp));
|
||||
@ -120,13 +138,14 @@ public class Rule {
|
||||
}
|
||||
|
||||
// Vérifier les timestamps seulement si le fichier existe
|
||||
if (depFile.exists() && TimestampManager.getTimestamp(depFile) > targetTimestamp) {
|
||||
if (depTimestamp > targetTimestamp) {
|
||||
if (BakeCLI.isDebug()) {
|
||||
System.out.println("Debug : Dependency " + dependency + " is newer than target file " + name + ", needs update");
|
||||
}
|
||||
return true;
|
||||
}
|
||||
}
|
||||
|
||||
return false;
|
||||
}
|
||||
}
|
||||
@ -89,15 +89,6 @@ Le test suit les étapes suivantes :
|
||||
|
||||
## Résultats attendus
|
||||
|
||||
- Le comportement standard de GNU Make est de reconstruire l'exécutable si le fichier source a un timestamp dans le futur
|
||||
- Le comportement standard de Make est de reconstruire l'exécutable si le fichier source a un timestamp dans le futur
|
||||
- `bake` devrait adopter le même comportement et reconstruire l'exécutable
|
||||
- Les messages de sortie devraient indiquer la reconstruction
|
||||
|
||||
## Explications techniques
|
||||
|
||||
Les systèmes de build comme make utilisent les timestamps des fichiers pour déterminer quand une recompilation est nécessaire. Quand un fichier source a une date de modification qui se situe dans le futur :
|
||||
|
||||
1. Certains systèmes de build considèrent toujours que le fichier a été modifié et déclenchent la reconstruction
|
||||
2. D'autres peuvent avoir des comportements particuliers ou des avertissements
|
||||
|
||||
Ce test vérifie que `bake` gère correctement ce cas particulier en adoptant le même comportement que `make`, ce qui est important pour assurer une compatibilité complète, même dans des situations inhabituelles.
|
||||
- Les messages de sortie devraient indiquer la reconstruction
|
||||
Binary file not shown.
Binary file not shown.
58
tests/C/test-22-future-timestamp/run_test22.sh
Normal file → Executable file
58
tests/C/test-22-future-timestamp/run_test22.sh
Normal file → Executable file
@ -2,6 +2,7 @@
|
||||
|
||||
# Script de test pour comparer bake et make
|
||||
# Test 22 - Gestion des timestamps dans le futur
|
||||
# À placer dans le répertoire test-22-future-timestamp
|
||||
|
||||
# Création du répertoire de logs s'il n'existe pas
|
||||
mkdir -p logs
|
||||
@ -36,8 +37,19 @@ cd make
|
||||
echo "Compilation initiale avec make..." | tee -a "../$LOG_FILE"
|
||||
make 2>&1 | tee -a "../$LOG_FILE"
|
||||
|
||||
# Enregistrons le timestamp de l'exécutable make après compilation initiale
|
||||
MAKE_INITIAL_TIMESTAMP=$(stat -c %Y "main" 2>/dev/null || stat -f %m "main")
|
||||
echo "Timestamp initial de l'exécutable make: $(date -r main)" | tee -a "../$LOG_FILE"
|
||||
|
||||
# Modification du timestamp du fichier main.c pour le mettre dans le futur (+ 1 jour)
|
||||
FUTURE_TIMESTAMP=$(date -d "+1 day" "+%Y%m%d%H%M.%S" 2>/dev/null || date -v+1d "+%Y%m%d%H%M.%S")
|
||||
# Utilisation de la syntaxe compatible avec macOS
|
||||
if [[ "$OSTYPE" == "darwin"* ]]; then
|
||||
# macOS
|
||||
FUTURE_TIMESTAMP=$(date -v+1d "+%Y%m%d%H%M.%S")
|
||||
else
|
||||
# Linux et autres
|
||||
FUTURE_TIMESTAMP=$(date -d "+1 day" "+%Y%m%d%H%M.%S")
|
||||
fi
|
||||
touch -t $FUTURE_TIMESTAMP main.c
|
||||
echo "Timestamp du fichier source main.c dans make modifié vers le futur: $(date -r main.c)" | tee -a "../$LOG_FILE"
|
||||
cd ..
|
||||
@ -47,6 +59,10 @@ cd bake
|
||||
echo "Compilation initiale avec bake..." | tee -a "../$LOG_FILE"
|
||||
java -cp bakefile.jar fr.monlouyan.bakefile.Main 2>&1 | tee -a "../$LOG_FILE"
|
||||
|
||||
# Enregistrons le timestamp de l'exécutable bake après compilation initiale
|
||||
BAKE_INITIAL_TIMESTAMP=$(stat -c %Y "main" 2>/dev/null || stat -f %m "main")
|
||||
echo "Timestamp initial de l'exécutable bake: $(date -r main)" | tee -a "../$LOG_FILE"
|
||||
|
||||
# Modification du timestamp du fichier main.c pour le mettre dans le futur (+ 1 jour)
|
||||
touch -t $FUTURE_TIMESTAMP main.c
|
||||
echo "Timestamp du fichier source main.c dans bake modifié vers le futur: $(date -r main.c)" | tee -a "../$LOG_FILE"
|
||||
@ -59,23 +75,26 @@ echo "=================================" | tee -a "$LOG_FILE"
|
||||
echo "Exécution de MAKE avec un fichier source ayant un timestamp dans le futur:" | tee -a "$LOG_FILE"
|
||||
echo "=================================" | tee -a "$LOG_FILE"
|
||||
cd make
|
||||
# Enregistrer le timestamp actuel de l'exécutable
|
||||
MAKE_EXEC_TIME_BEFORE=$(stat -c %Y "main" 2>/dev/null || stat -f %m "main")
|
||||
|
||||
# On va aussi enregistrer la sortie de make dans un fichier temporaire
|
||||
# pour détecter si gcc a été exécuté
|
||||
MAKE_OUTPUT_FILE=$(mktemp)
|
||||
echo "$ make" | tee -a "../$LOG_FILE"
|
||||
make 2>&1 | tee -a "../$LOG_FILE"
|
||||
make 2>&1 | tee "$MAKE_OUTPUT_FILE" | tee -a "../$LOG_FILE"
|
||||
MAKE_EXIT_CODE=$?
|
||||
echo "" | tee -a "../$LOG_FILE"
|
||||
echo "Code de sortie: $MAKE_EXIT_CODE" | tee -a "../$LOG_FILE"
|
||||
|
||||
# Vérifier si l'exécutable a été reconstruit
|
||||
# On peut aussi vérifier si "gcc" apparaît dans la sortie de make
|
||||
MAKE_EXEC_TIME_AFTER=$(stat -c %Y "main" 2>/dev/null || stat -f %m "main")
|
||||
if [ "$MAKE_EXEC_TIME_AFTER" -ne "$MAKE_EXEC_TIME_BEFORE" ]; then
|
||||
if grep -q "gcc -o main" "$MAKE_OUTPUT_FILE" || [ "$MAKE_EXEC_TIME_AFTER" -ne "$MAKE_INITIAL_TIMESTAMP" ]; then
|
||||
echo "✅ make: L'exécutable a été reconstruit (avec un timestamp source dans le futur)" | tee -a "../$LOG_FILE"
|
||||
echo "Nouveau timestamp de l'exécutable: $(date -r main)" | tee -a "../$LOG_FILE"
|
||||
else
|
||||
echo "❌ make: L'exécutable n'a PAS été reconstruit malgré un timestamp source dans le futur" | tee -a "../$LOG_FILE"
|
||||
fi
|
||||
rm -f "$MAKE_OUTPUT_FILE"
|
||||
cd ..
|
||||
|
||||
echo "" | tee -a "$LOG_FILE"
|
||||
@ -85,40 +104,31 @@ echo "=================================" | tee -a "$LOG_FILE"
|
||||
echo "Exécution de BAKE avec un fichier source ayant un timestamp dans le futur:" | tee -a "$LOG_FILE"
|
||||
echo "=================================" | tee -a "$LOG_FILE"
|
||||
cd bake
|
||||
# Enregistrer le timestamp actuel de l'exécutable
|
||||
BAKE_EXEC_TIME_BEFORE=$(stat -c %Y "main" 2>/dev/null || stat -f %m "main")
|
||||
|
||||
# On va aussi enregistrer la sortie de bake dans un fichier temporaire
|
||||
# pour détecter si gcc a été exécuté
|
||||
BAKE_OUTPUT_FILE=$(mktemp)
|
||||
echo "$ java -cp bakefile.jar fr.monlouyan.bakefile.Main" | tee -a "../$LOG_FILE"
|
||||
java -cp bakefile.jar fr.monlouyan.bakefile.Main 2>&1 | tee -a "../$LOG_FILE"
|
||||
java -cp bakefile.jar fr.monlouyan.bakefile.Main 2>&1 | tee "$BAKE_OUTPUT_FILE" | tee -a "../$LOG_FILE"
|
||||
BAKE_EXIT_CODE=$?
|
||||
echo "" | tee -a "../$LOG_FILE"
|
||||
echo "Code de sortie: $BAKE_EXIT_CODE" | tee -a "../$LOG_FILE"
|
||||
|
||||
# Vérifier si l'exécutable a été reconstruit
|
||||
# On peut aussi vérifier si "gcc" apparaît dans la sortie de bake
|
||||
BAKE_EXEC_TIME_AFTER=$(stat -c %Y "main" 2>/dev/null || stat -f %m "main")
|
||||
if [ "$BAKE_EXEC_TIME_AFTER" -ne "$BAKE_EXEC_TIME_BEFORE" ]; then
|
||||
if grep -q "gcc -o main" "$BAKE_OUTPUT_FILE" || [ "$BAKE_EXEC_TIME_AFTER" -ne "$BAKE_INITIAL_TIMESTAMP" ]; then
|
||||
echo "✅ bake: L'exécutable a été reconstruit (avec un timestamp source dans le futur)" | tee -a "../$LOG_FILE"
|
||||
echo "Nouveau timestamp de l'exécutable: $(date -r main)" | tee -a "../$LOG_FILE"
|
||||
else
|
||||
echo "❌ bake: L'exécutable n'a PAS été reconstruit malgré un timestamp source dans le futur" | tee -a "../$LOG_FILE"
|
||||
fi
|
||||
rm -f "$BAKE_OUTPUT_FILE"
|
||||
cd ..
|
||||
|
||||
# Résumé des résultats
|
||||
echo "" | tee -a "$LOG_FILE"
|
||||
echo "=================================" | tee -a "$LOG_FILE"
|
||||
echo "RÉSUMÉ:" | tee -a "$LOG_FILE"
|
||||
echo "=================================" | tee -a "$LOG_FILE"
|
||||
|
||||
if [ "$MAKE_EXEC_TIME_AFTER" -ne "$MAKE_EXEC_TIME_BEFORE" ] && [ "$BAKE_EXEC_TIME_AFTER" -ne "$BAKE_EXEC_TIME_BEFORE" ]; then
|
||||
echo "✅ Les deux outils ont reconstruit l'exécutable face à un timestamp source dans le futur" | tee -a "$LOG_FILE"
|
||||
elif [ "$MAKE_EXEC_TIME_AFTER" -eq "$MAKE_EXEC_TIME_BEFORE" ] && [ "$BAKE_EXEC_TIME_AFTER" -eq "$BAKE_EXEC_TIME_BEFORE" ]; then
|
||||
echo "❌ Les deux outils n'ont pas reconstruit l'exécutable malgré un timestamp source dans le futur" | tee -a "$LOG_FILE"
|
||||
else
|
||||
echo "⚠️ Comportement différent entre make et bake:" | tee -a "$LOG_FILE"
|
||||
echo " - make: $([ "$MAKE_EXEC_TIME_AFTER" -ne "$MAKE_EXEC_TIME_BEFORE" ] && echo '✅ A reconstruit' || echo '❌ N'"'"'a pas reconstruit')" | tee -a "$LOG_FILE"
|
||||
echo " - bake: $([ "$BAKE_EXEC_TIME_AFTER" -ne "$BAKE_EXEC_TIME_BEFORE" ] && echo '✅ A reconstruit' || echo '❌ N'"'"'a pas reconstruit')" | tee -a "$LOG_FILE"
|
||||
fi
|
||||
# Vérification plus fiable du comportement de make et bake
|
||||
MAKE_REBUILT=$(grep -q "gcc -o main" "$MAKE_OUTPUT_FILE" 2>/dev/null || [ "$MAKE_EXEC_TIME_AFTER" -ne "$MAKE_INITIAL_TIMESTAMP" ] && echo "true" || echo "false")
|
||||
BAKE_REBUILT=$(grep -q "gcc -o main" "$BAKE_OUTPUT_FILE" 2>/dev/null || [ "$BAKE_EXEC_TIME_AFTER" -ne "$BAKE_INITIAL_TIMESTAMP" ] && echo "true" || echo "false")
|
||||
|
||||
echo "" | tee -a "$LOG_FILE"
|
||||
echo "Test terminé. Résultats enregistrés dans: $LOG_FILE" | tee -a "$LOG_FILE"
|
||||
Loading…
x
Reference in New Issue
Block a user