Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Commit

Permalink
Fix dist-check for LilyPond 2.23.5
Browse files Browse the repository at this point in the history
There are now so many .ly files that Python complains if passing
all of them as a list to os.popen. Instead use a find invocation
to get the list dynamically.
  • Loading branch information
hahnjo authored and PhilHolmes committed Nov 28, 2021
1 parent f687c4c commit 319b672
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions test-lily/dist-check.py
Original file line number Diff line number Diff line change
Expand Up @@ -82,13 +82,7 @@ def check_files (tarball, repo):
files = [f.strip () for f in files]

## .ly files
ly_files = [f for f in files
if re.search (r'\.ly$', f)]


ly_file_str = ' '.join (ly_files)

no_version = popen (r"cd %(dir)s && grep '\\version' -L %(ly_file_str)s" % locals ()).readlines ()
no_version = popen (r"cd %(dir)s && grep '\\version' -L $(find . -name '*.ly')" % locals ()).readlines ()
if no_version:
printf ('Files without \\version: ')
printf ('\n'.join (no_version))
Expand Down

0 comments on commit 319b672

Please sign in to comment.