Not rewrites hooks if no changes

Author: ilyavy

README.md

@@ -13,7 +13,7 @@ the project is rebuild. Be aware that for hooks to be installed to git, any `mvn
 like `compile` or `test`. It means that after editing git-hooks-maven-plugin configuration in `pom.xml`,
 it's necessary to manually run `mvn compile` or any other maven goal, on the initializing step of which git hooks will be installed. 
 
-The example with *pre-commit* and *pre-push* hooks configured, will look like it:
+The example with *pre-commit* and *pre-push* hooks configured, will look like this:
 ```xml
 <plugins>
  <plugin>

pom.xml

@@ -6,7 +6,7 @@
 
  <groupId>io.github.pepperkit</groupId>
  <artifactId>git-hooks-maven-plugin</artifactId>
- <version>1.0.0</version>
+ <version>1.1.0</version>
  <packaging>maven-plugin</packaging>
  <name>Git Hooks Maven Plugin</name>
  <description>Maven plugin, which enables control on git hooks via easy configuration.</description>
@@ -321,7 +321,7 @@
  <plugin>
  <groupId>io.github.pepperkit</groupId>
  <artifactId>git-hooks-maven-plugin</artifactId>
- <version>1.0.0</version>
+ <version>1.1.0</version>
  <configuration>
  <hooks>
  <pre-commit>mvn -B checkstyle:checkstyle</pre-commit>

src/main/java/io/github/pepperkit/githooks/ExecuteHooksMojo.java

@@ -24,7 +24,7 @@ public class ExecuteHooksMojo extends AbstractMojo {
  @Parameter(property = "hookName")
  public String hookName;
 
- GitHooksManager gitHooksManager = new GitHooksManager(getLog());
+ GitHooksManager gitHooksManager = new GitHooksManager(this);
 
  @Override
  public void execute() throws MojoExecutionException {

src/main/java/io/github/pepperkit/githooks/GitHooksManager.java

@@ -16,8 +16,6 @@
 import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.nio.file.attribute.PosixFilePermission;
-import java.time.ZoneId;
-import java.time.format.DateTimeFormatter;
 import java.util.Arrays;
 import java.util.Collections;
 import java.util.HashSet;
@@ -28,8 +26,7 @@
 import java.util.concurrent.Executors;
 import java.util.stream.Collectors;
 
-import org.apache.maven.plugin.logging.Log;
-import org.apache.maven.plugin.logging.SystemStreamLog;
+import org.apache.maven.plugin.Mojo;
 
 /**
  * Manages all the work with git hooks.
@@ -69,24 +66,14 @@ public class GitHooksManager {
  PosixFilePermission.OWNER_EXECUTE
  )));
 
- private final Log logger;
-
- private final DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd_HH-mm-ss")
- .withZone(ZoneId.systemDefault());
+ private final Mojo mojo;
 
  /**
- * Creates GitHooksManager with a newly created logger.
+ * Creates GitHooksManager with the provided mojo. Mojo is used to obtain the correct logger.
+ * @param mojo mojo, which initiated the work with GitHooksManager
  */
- public GitHooksManager() {
- logger = new SystemStreamLog();
- }
-
- /**
- * Creates GitHooksManager with the provided mojo logger.
- * @param logger mojo logger
- */
- public GitHooksManager(Log logger) {
- this.logger = logger;
+ public GitHooksManager(Mojo mojo) {
+ this.mojo = mojo;
  }
 
  /**
@@ -143,16 +130,23 @@ List<File> getExistingHookFiles() {
  */
  void createHook(String hookName, String hookValue) throws IOException {
  String hookPath = getHookPath(hookName);
-
- try (BufferedWriter writer = new BufferedWriter(new FileWriter(hookPath))) {
- logger.info("Writing `" + hookName + "` hook");
- writer.write(SHEBANG + "\n" + hookValue.replaceAll("[ ]{2,}", ""));
-
- Path hookFilePath = Paths.get(hookPath);
- if (hookFilePath.getFileSystem().supportedFileAttributeViews().contains("posix")) {
- Set<PosixFilePermission> currentPermissions = Files.getPosixFilePermissions(hookFilePath);
- if (!currentPermissions.containsAll(HOOK_FILE_PERMISSIONS)) {
- Files.setPosixFilePermissions(hookFilePath, HOOK_FILE_PERMISSIONS);
+ String fullHookValue = SHEBANG + "\n" + hookValue.replaceAll("[ ]{2,}", "");
+
+ Optional<String> existingHookValue = readHook(hookName);
+ if (existingHookValue.isPresent() && existingHookValue.get().equals(fullHookValue)) {
+ mojo.getLog().info("The hook `" + hookName + "` has not changed, skipping");
+
+ } else {
+ try (BufferedWriter writer = new BufferedWriter(new FileWriter(hookPath))) {
+ mojo.getLog().info("Writing `" + hookName + "` hook");
+ writer.write(fullHookValue);
+
+ Path hookFilePath = Paths.get(hookPath);
+ if (hookFilePath.getFileSystem().supportedFileAttributeViews().contains("posix")) {
+ Set<PosixFilePermission> currentPermissions = Files.getPosixFilePermissions(hookFilePath);
+ if (!currentPermissions.containsAll(HOOK_FILE_PERMISSIONS)) {
+ Files.setPosixFilePermissions(hookFilePath, HOOK_FILE_PERMISSIONS);
+ }
  }
  }
  }
@@ -170,7 +164,7 @@ private String getHookPath(String hookName) {
  */
  boolean printHook(String hookName) throws IOException {
  Optional<String> hookValue = readHook(hookName);
- hookValue.ifPresent(h -> logger.info(
+ hookValue.ifPresent(h -> mojo.getLog().info(
  "`" + hookName + "` -> The following commands will be invoked: \n" + h));
  return hookValue.isPresent();
  }
@@ -199,14 +193,14 @@ boolean executeHook(String hookName) throws InterruptedException, IOException {
 
  Optional<String> hook = readHook(hookName);
  if (hook.isPresent()) {
- logger.info(">>>>> Executing hook `" + hookName + "` <<<<<");
+ mojo.getLog().info(">>>>> Executing hook `" + hookName + "` <<<<<");
  Process process = Runtime.getRuntime().exec("sh -c " + getHookPath(hookName));
  Executors.newSingleThreadExecutor().submit(() -> new BufferedReader(
- new InputStreamReader(process.getInputStream())).lines().forEach(logger::info));
+ new InputStreamReader(process.getInputStream())).lines().forEach(mojo.getLog()::info));
 
  int exitCode = process.waitFor();
- logger.info("Exit code is " + exitCode);
- logger.info(">>>>> The hook `" + hookName + "` was executed with the "
+ mojo.getLog().info("Exit code is " + exitCode);
+ mojo.getLog().info(">>>>> The hook `" + hookName + "` was executed with the "
  + (exitCode == 0 ? "SUCCESS" : "ERROR") + " result <<<<<");
  }
  return hook.isPresent();

src/main/java/io/github/pepperkit/githooks/InitHooksMojo.java

@@ -29,7 +29,7 @@ public class InitHooksMojo extends AbstractMojo {
  @Parameter(defaultValue = "null")
  public Map<String, String> hooks;
 
- GitHooksManager gitHooksManager = new GitHooksManager(getLog());
+ GitHooksManager gitHooksManager = new GitHooksManager(this);
 
  /**
  * Helper method, is used to set hooks to null if nothing is provided by the user.

src/main/java/io/github/pepperkit/githooks/PrintHooksMojo.java

@@ -23,7 +23,7 @@ public class PrintHooksMojo extends AbstractMojo {
  @Parameter(property = "hookName")
  public String hookName;
 
- GitHooksManager gitHooksManager = new GitHooksManager(getLog());
+ GitHooksManager gitHooksManager = new GitHooksManager(this);
 
  @Override
  public void execute() throws MojoExecutionException {