[ros-diffs] [reactos] 07/09: [WINESYNC] Use the same 'winesync' author as for all the other commits the script generates.

[Hermès Bélusca-Maïto]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=1d92196d2afbe210328e1…

commit 1d92196d2afbe210328e1147229da62b121ccb5f
Author:     Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org>
AuthorDate: Tue Sep 12 13:21:36 2023 +0200
Commit:     Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org>
CommitDate: Mon Dec 18 22:21:39 2023 +0100

    [WINESYNC] Use the same 'winesync' author as for all the other commits the
script generates.
    
    This now correctly sets the commit author as 'winesync' (before it was
    the committer's developer name) for the following types of commits:
    
    `[WINESYNC]: revert wine-staging patchset for <MODULE_NAME>`
    
    and
    
    `[WINESYNC]: <MODULE_NAME> is now in sync with wine-staging <WINE_TAG>`
---
 sdk/tools/winesync/winesync.py | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/sdk/tools/winesync/winesync.py b/sdk/tools/winesync/winesync.py
index 63cafd60a2b..7335a8a37ac 100644
--- a/sdk/tools/winesync/winesync.py
+++ b/sdk/tools/winesync/winesync.py
@@ -46,6 +46,9 @@ class wine_sync:
         self.wine_staging_repo = pygit2.Repository(self.wine_staging_src)
         self.reactos_repo = pygit2.Repository(self.reactos_src)
 
+        # the standard author signature we will use
+        self.winesync_author_signature = pygit2.Signature('winesync',
&#39;ros-dev(a)reactos.org&#39;)
+
         # read the index from the reactos tree
         self.reactos_index = self.reactos_repo.index
         self.reactos_index.read()
@@ -245,8 +248,9 @@ class wine_sync:
         else:
             commit_msg += f'wine commit id {str(wine_commit.id)} by
{wine_commit.author.name} <{wine_commit.author.email}>'
 
-        self.reactos_repo.create_commit('HEAD',
-            pygit2.Signature('winesync', &#39;ros-dev(a)reactos.org&#39;)rg'),
+        self.reactos_repo.create_commit(
+            'HEAD',
+            self.winesync_author_signature,
             self.reactos_repo.default_signature,
             commit_msg,
             self.reactos_index.write_tree(),
@@ -310,7 +314,7 @@ class wine_sync:
 
         self.reactos_repo.create_commit(
             'HEAD',
-            self.reactos_repo.default_signature,
+            self.winesync_author_signature,
             self.reactos_repo.default_signature,
             f'[WINESYNC]: revert wine-staging patchset for {self.module}',
             self.reactos_index.write_tree(),
@@ -376,7 +380,7 @@ class wine_sync:
             self.reactos_index.write()
             self.reactos_repo.create_commit(
                 'HEAD',
-                self.reactos_repo.default_signature,
+                self.winesync_author_signature,
                 self.reactos_repo.default_signature,
                 f'[WINESYNC]: {self.module} is now in sync with wine-staging
{wine_tag}',
                 self.reactos_index.write_tree(),