majic-scripts Changeset

Changeset - 8092a1881a49

Parent rev.

Child rev.

[Not reviewed]

0 1 0

Branko Majic (branko) - 1 month ago 2025-03-02 00:58:38
branko@majic.rs

[thebuggenie_hg_remote.py] Fixed linter errors:

- Mostly spacing-related.
- Use the range function instead of xrange (Python 2.7 -> Python 3.x).
- Script probably will not run correctly under Python 3.x, but at
least the linter will not get in the way when working on other
Python code in the project.

1 file changed with 20 insertions and 15 deletions:

hooks/thebuggenie_hg_remote.py

0 comments (0 inline, 0 general) First comment

hooks/thebuggenie_hg_remote.py

➞

Show inline comments

@@ @@ -25,7 +25,7 @@ @@
 # passkey = project_passkey
 # project = project_id
 # nosslverify = [true|false]
+#
+#
 # [hooks]
 # changegroup.buggenie = python:buggenie.hook
+#
@@ @@ -59,6 +59,7 @@ import subprocess @@
 import mercurial.node
 def call_program(program, arguments, url):
     """
     Calls the specified program, passing it the arguments. The URL is passed
@@ @@ -83,7 +84,7 @@ def call_program(program, arguments, url): @@
     arguments.insert(0, program)
     # Set-up the subprocess for execution.
-    process = subprocess.Popen(arguments, stdin = subprocess.PIPE, stdout = subprocess.PIPE, stderr = subprocess.PIPE)
     process = subprocess.Popen(arguments, stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
     # curl expects slightly different format when reading URL from stdin.
     if program == "curl":
@@ @@ -95,6 +96,7 @@ def call_program(program, arguments, url): @@
     # Return the exit code of process, and message that should be shown to user.
     return process.returncode, stdout, stderr
 def extract_report(repo, context):
     """
     Extract the commit report from provided context.
@@ @@ -141,12 +143,13 @@ def extract_report(repo, context): @@
     # Finally return the report.
     return report
 def submit_report(commit, repo, ui, program, program_params, base_url, project_id, passkey):
     """
     Submits report to TBG for a single commit.
     Arguments:
     commit - Mercurial commit identifier.
     repo - Mercurial repository object.
     # Set-up the base link for project.
     url = "%s/vcs_integration/report/%s/?" % (base_url, project_id)
     # Pass the GET arguments.
-    args = [ "%s=%s" % (quote(key), quote(value)) for key, value in report.iteritems()]
     args = ["%s=%s" % (quote(key), quote(value)) for key, value in report.iteritems()]
     url += "&".join(args)
     # Call the program.
         stdout += "\n"
         ui.warn(stdout)
 def hook(ui, repo, hooktype, node = None, url = None, **kwargs):
 def hook(ui, repo, hooktype, node=None, url=None, **kwargs):
     """
     Commit and changegroup hook for Mercurial.
     config = dict((param, value) for param, value in ui.configitems('buggenie'))
     # Make sure the required settings were specified.
-    if not "program" in config:
+    if "program" not in config:
         ui.warn("TBG HG Hook: No program was specified. Check your settings.\n")
         return
-    if not "url" in config:
+    if "url" not in config:
         ui.warn("TBG HG Hook: No base URL was specified. Check your settings.\n")
         return
-    if not "project" in config:
+    if "project" not in config:
         ui.warn("TBG HG Hook: No project ID was specified. Check your settings.\n")
         return
-    if not "passkey" in config:
+    if "passkey" not in config:
         ui.warn("TBG HG Hook: No passkey was specified. Check your settings.\n")
         return
     # Set-up parameters to be passed onto program.
     if config["program"] == "wget":
         # Use dots for progress, output to stdout, read link from stdin.
-        program_params = [ "--progress=dot", "-O", "-", "-i", "-" ]
         program_params = ["--progress=dot", "-O", "-", "-i", "-"]
         if config.get("nosslverify") == "true":
             program_params.append("--no-check-certificate")
     elif config["program"] == "curl":
         # Quiet, but show errors, fail on server error, and read link from stdin.
-        program_params = [ "-s", "-S", "-f", "-K", "-" ]
         program_params = ["-s", "-S", "-f", "-K", "-"]
         if config.get("nosslverify") == "true":
             program_params.append("--insecure")
     else:
         start = repo.changelog.rev(node_hash)
         end = len(repo.changelog)
         for commit in xrange(start, end):
             submit_report(commit, repo, ui, config["program"], program_params, config["url"], config["project"], config["passkey"])
         for commit in range(start, end):
             submit_report(commit, repo, ui,
                           config["program"], program_params, config["url"], config["project"], config["passkey"])
     # If the hook is called as a commit, hook, process just the single commit.
     elif hooktype == "commit":
         submit_report(commit, repo, ui, config["program"], program_params, config["url"], config["project"], config["passkey"])
         submit_report(commit, repo, ui,
                       config["program"], program_params, config["url"], config["project"], config["passkey"])

0 comments (0 inline, 0 general) First comment