Skip to content

fix error in macOS #2671

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Oct 17, 2023
Merged

fix error in macOS #2671

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
28a2c9f
fix error in macOS
sofide Oct 15, 2023
d5d41d6
move check_spell to a function
sofide Oct 16, 2023
4c30f11
remove multiprocessing hack and add docstring
sofide Oct 16, 2023
c54aa03
Merge branch '3.12' into fix/check_spell_on_macos
sofide Oct 16, 2023
1a59606
Update scripts/check_spell.py
sofide Oct 17, 2023
3c886da
Merge branch '3.12' into fix/check_spell_on_macos
sofide Oct 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
move check_spell to a function
  • Loading branch information
@sofide
sofide committed Oct 16, 2023
commit d5d41d61f55d3adef28427d8fe7824fce4969970
65 changes: 36 additions & 29 deletions scripts/check_spell.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,32 +9,39 @@

import pospell

# fix multiprocessing error loop on MacOS
if sys.platform == "darwin":
import multiprocessing
multiprocessing.set_start_method("fork")

# Read custom dictionaries
entries = set()
for filename in Path("dictionaries").glob("*.txt"):
with open(filename, "r") as f:
entries.update(
stripped_line
for stripped_line in (line.strip() for line in f.readlines())
if stripped_line
)

# Write merged dictionary file
output_filename = tempfile.mktemp(suffix="_merged_dict.txt")
with open(output_filename, "w") as f:
for e in entries:
f.write(e)
f.write("\n")

# Run pospell either against all files or the file given on the command line
po_files = sys.argv[1:]
if not po_files:
po_files = Path(".").glob("*/*.po")

errors = pospell.spell_check(po_files, personal_dict=output_filename, language="es_ES")
sys.exit(0 if errors == 0 else -1)

def check_spell(po_files=None):
# fix multiprocessing error loop on MacOS
if sys.platform == "darwin":
import multiprocessing
multiprocessing.set_start_method("fork")

# Read custom dictionaries
entries = set()
for filename in Path("dictionaries").glob("*.txt"):
with open(filename, "r") as f:
entries.update(
stripped_line
for stripped_line in (line.strip() for line in f.readlines())
if stripped_line
)

# Write merged dictionary file
output_filename = tempfile.mktemp(suffix="_merged_dict.txt")
with open(output_filename, "w") as f:
for e in entries:
f.write(e)
f.write("\n")

# Run pospell either against all files or the file given on the command line
if not po_files:
po_files = Path(".").glob("*/*.po")

detected_errors = pospell.spell_check(po_files, personal_dict=output_filename, language="es_ES")
return detected_errors


if __name__ == "__main__":
po_files = sys.argv[1:]
errors = check_spell(po_files)
sys.exit(0 if errors == 0 else -1)