Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2025-02-11 01:57:00 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1467148 - exclude .DS_Store file from list of subdirs to check for license info, r=ted

Browse Source 
vendor_rust.py checks all dirs in third_party/rust/ for license information, but it wasn't distinguishing between subdirs and files when listing the contents of that directory, and the script earlier causes macOS to create a .DS_Store file in the directory, which the script then treats as a subdir, causing an IOError "not a directory". This change excludes the .DS_Store file (and other non-dirs) from the list of subdirs of third_party/rust/ to check for license info.

MozReview-Commit-ID: 4CrJH9tZLnG

--HG--
extra : rebase_source : 464ad17f4c9b76303f9024d8eaad475b1fc248f7
This commit is contained in:
Myk Melez 2018-07-02 14:27:22 -07:00
parent bdde614ed0
commit cb3429d30b
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
python/mozbuild/mozbuild/vendor_rust.py
View File

@ -272,7 +272,7 @@ license file's hash.
# Force all of the packages to be checked for license information
# before reducing via `all`, so all license issues are found in a
# single `mach vendor rust` invocation.
results = [check_package(p) for p in os.listdir(vendor_dir)]
results = [check_package(p) for p in os.listdir(vendor_dir) if os.path.isdir(os.path.join(vendor_dir, p))]
return all(results)
def vendor(self, ignore_modified=False,