Skip to content

Commit

Permalink
Merge pull request #17331 from edsantiago/makedocs_parallel_safe
Browse files Browse the repository at this point in the history
make hack/markdown-preprocess parallel-safe
  • Loading branch information
openshift-merge-robot authored Feb 3, 2023
2 parents 51dd01c + bd8937c commit 4c8ad63
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion hack/markdown-preprocess
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ class Preprocessor():
cross-referencing all the man pages in which they're used.
"""
for optionfile in self.used_by:
tmpfile = optionfile + '.tmp'
tmpfile = optionfile + '.tmp.' + str(os.getpid())
with open(optionfile, 'r', encoding='utf-8') as fh_in, open(tmpfile, 'w', encoding='utf-8', newline='\n') as fh_out:
fh_out.write("####> This option file is used in:\n")
used_by = ', '.join(x for x in self.used_by[optionfile])
Expand Down

3 comments on commit 4c8ad63

@lsm5
Copy link
Member

@lsm5 lsm5 commented on 4c8ad63 Feb 6, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lsm5
Copy link
Member

@lsm5 lsm5 commented on 4c8ad63 Feb 6, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the real culprit is likely something else. All other repos are failing too.

@cevich
Copy link
Member

@cevich cevich commented on 4c8ad63 Feb 6, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coming full-circle here: The problem was a change in the Cirrus GraphQL API that broke a unit-test included in "prebuild.sh". Fix: #17383

Please sign in to comment.