#494 Fix TypeError Which Was Breaking All Builds
Closed 9 months ago by thebeanogamer. Opened 9 months ago by thebeanogamer.
thebeanogamer/FedoraReview fixtypeerror  into  master

file modified
+1 -1
@@ -334,7 +334,7 @@ 

          except ValueError:

              # E. g., when given '(contains no files)'

              continue

-         mode = int(mode, 8)

+         mode = int(mode[0], 8)

          if mode & 0o40000:

              dirs.append(path)

          else:

Since 8eeadf2, mode has been a list. However, int is expecting a string. Because of this mismatch, all runs of fedora-review are currently failing the "Package must build in mock" check.

This overlaps with #492, I think either implementation will work.

#492 was merged, this is now obsolete

Pull-Request has been closed by thebeanogamer

9 months ago
Metadata