zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Added line limit exemptions for 'variant' and 'version' directives.

Browse Source 
Updated the 'scotch' package to use oneline 'variant' directives.
This commit is contained in:
Joseph Ciurej 2016-05-16 12:14:57 -07:00
parent 52c8bedb9c
commit 761c5c8450
2 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
share/spack/qa/run-flake8
View File

@ -20,12 +20,17 @@ fi
# Check if changed files are flake8 conformant [framework] # Check if changed files are flake8 conformant [framework]
changed=$(git diff --name-only develop... | grep '.py$') changed=$(git diff --name-only develop... | grep '.py$')
# Exempt url lines in changed packages from overlong line errors. # Add approved style exemptions to the changed packages.
# Exempt functions defined with '@when' decorator from redefinition errors.
for file in $changed; do for file in $changed; do
if [[ $file = *package.py ]]; then if [[ $file = *package.py ]]; then
cp "$file" "$file~" cp "$file" "$file~"
# Exempt lines with urls and descriptions from overlong line errors.
perl -i -pe 's/^(\s*url\s*=.*)$/\1 # NOQA: ignore=E501/' $file perl -i -pe 's/^(\s*url\s*=.*)$/\1 # NOQA: ignore=E501/' $file
perl -i -pe 's/^(\s*version\(.*\).*)$/\1 # NOQA: ignore=E501/' $file
perl -i -pe 's/^(\s*variant\(.*\).*)$/\1 # NOQA: ignore=E501/' $file
# Exempt '@when' decorated functions from redefinition errors.
perl -i -pe 's/^(\s*\@when\(.*\).*)$/\1 # NOQA: ignore=F811/' $file perl -i -pe 's/^(\s*\@when\(.*\).*)$/\1 # NOQA: ignore=F811/' $file
fi fi
done done

12
var/spack/repos/builtin/packages/scotch/package.py
View File

@ -40,14 +40,10 @@ class Scotch(Package):
version('6.0.0', 'c50d6187462ba801f9a82133ee666e8e') version('6.0.0', 'c50d6187462ba801f9a82133ee666e8e')
version('5.1.10b', 'f587201d6cf5cf63527182fbfba70753') version('5.1.10b', 'f587201d6cf5cf63527182fbfba70753')
variant('mpi', default=False, variant('mpi', default=False, description='Activate the compilation of parallel libraries')
description='Activate the compilation of parallel libraries') variant('compression', default=True, description='Activate the posibility to use compressed files')
variant('compression', default=True, variant('esmumps', default=False, description='Activate the compilation of esmumps needed by mumps')
description='Activate the posibility to use compressed files') variant('shared', default=True, description='Build a shared version of the library')
variant('esmumps', default=False,
description='Activate the compilation of esmumps needed by mumps')
variant('shared', default=True,
description='Build a shared version of the library')
depends_on('flex') depends_on('flex')
depends_on('bison') depends_on('bison')