<div dir="ltr"><div dir="ltr">Hi Tim,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El mar., 20 oct. 2020 a las 21:12, Tim Ooms (<<a href="mailto:tim.ooms@aeronomie.be">tim.ooms@aeronomie.be</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Thomas,<br>
<br>
Yes, I did.<br>
<br>
The goal of the patch is to not break the non-kallithea hook scripts <br>
that may be present when doing a rescan and reinstall (after an update <br>
for example) and at the same time supporting the execution of <br>
third-party hook scripts next to the kallithea ones.<br>
<br>
When using the force parameter, instead of overwriting the non-kallithea <br>
hook script, it renames the script by adding .<date>.<kallithea-version> <br>
to it (can off course be something else).<br>
<br>
The new version of the kallithea hook script will execute all scripts in <br>
the hook-directory matching <hook-name>.*.* (should just match previous <br>
pattern). This way, it executes the previously installed hook script and <br>
possible other hook scripts the admin added matching the pattern.<br></blockquote><div><br></div><div>Thanks for the background.</div><div>In general, we encourage contributors to write clear commit messages with all necessary info to understand the what/why/how of changes. So info like the above would be great to have in such commit message.<br></div><div><br></div><div>For these changes, I think they can be seen as two separate things and should therefore better be two separate patches:</div><div>- support running additional git hooks in addition to the kallithea one</div><div>- rename existing git hooks rather than overwriting them, when 'force' is given.</div><div><br></div><div>I'm not sure if the second part is really necessary: it should normally only occur once, the code adds quite some complexity for such mundane task of renaming a file. Perhaps it could be enough to have clear enough documentation or messages?</div><div>Let's see what Mads Kiilerich thinks.<br></div><div><br></div><div>For the first part, I can see that it can be useful.</div><div>See also the comments on following proposed change, for a recent discussion regarding the git hooks and how it is implemented internally:</div><div><a href="https://kallithea-scm.org/repos/kallithea-incoming/changeset/c63dd3aeacdbbd77349bb2b9100dee5ba0c3fda6">https://kallithea-scm.org/repos/kallithea-incoming/changeset/c63dd3aeacdbbd77349bb2b9100dee5ba0c3fda6</a></div><div> </div>Best regards,</div><div class="gmail_quote">Thomas<br></div></div>