[Git][reproducible-builds/diffoscope][master] Return None, not the empty string, if no command is found.

Chris Lamb (@lamby) gitlab at salsa.debian.org
Tue Oct 26 10:48:45 UTC 2021



Chris Lamb pushed to branch master at Reproducible Builds / diffoscope


Commits:
8cbe1ab6 by Chris Lamb at 2021-10-26T11:48:32+01:00
Return None, not the empty string, if no command is found.

- - - - -


1 changed file:

- diffoscope/tools.py


Changes:

=====================================
diffoscope/tools.py
=====================================
@@ -48,7 +48,7 @@ def find_executable(cmd):
     command. Will also try the command with some common suffixes (eg.
     `dumppdf.py`) to support distributions that strip or retain them.
 
-    Returns an empy string (``) if no command is found.
+    Returns `None` if no command is found.
     """
 
     for suffix in ("", ".py"):
@@ -56,8 +56,6 @@ def find_executable(cmd):
         if val:
             return val
 
-    return ""
-
 
 def get_tools(only_missing=False):
     """Return the tool configuration in a dict"""



View it on GitLab: https://salsa.debian.org/reproducible-builds/diffoscope/-/commit/8cbe1ab6c3e30259bdd2bd9f0768d6e8c1573209

-- 
View it on GitLab: https://salsa.debian.org/reproducible-builds/diffoscope/-/commit/8cbe1ab6c3e30259bdd2bd9f0768d6e8c1573209
You're receiving this email because of your account on salsa.debian.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.reproducible-builds.org/pipermail/rb-commits/attachments/20211026/52b7df7d/attachment.htm>


More information about the rb-commits mailing list