Skip to content

New package: IDL#17451

Merged
adamjstewart merged 6 commits intospack:developfrom
francinelapid:idl-package
Jul 31, 2020
Merged

New package: IDL#17451
adamjstewart merged 6 commits intospack:developfrom
francinelapid:idl-package

Conversation

@francinelapid
Copy link
Copy Markdown
Contributor

@francinelapid francinelapid commented Jul 9, 2020 •
edited
Loading

#17199

  • requires manual download of IDL
    • will look for IDL tar in either a mirror or the current working directory

adamjstewart
adamjstewart requested changes Jul 10, 2020
View reviewed changes
@adamjstewart
Copy link
Copy Markdown
Member

adamjstewart commented Jul 10, 2020

I haven't used IDL in forever, once taught a course in IDL for satellite imagery processing.

@shahzebsiddiqui shahzebsiddiqui mentioned this pull request Jul 12, 2020
Open
19 tasks
adamjstewart
adamjstewart reviewed Jul 20, 2020
View reviewed changes
@adamjstewart
Copy link
Copy Markdown
Member

adamjstewart commented Jul 20, 2020

Flake8:

var/spack/repos/builtin/packages/idl/package.py:7: [F401] 'shutil' imported but unused
var/spack/repos/builtin/packages/idl/package.py:8: [F401] 'pathlib' imported but unused
var/spack/repos/builtin/packages/idl/package.py:9: [F401] 'os' imported but unused
var/spack/repos/builtin/packages/idl/package.py:14: [W293] blank line contains whitespace
var/spack/repos/builtin/packages/idl/package.py:16: [W291] trailing whitespace
var/spack/repos/builtin/packages/idl/package.py:26: [E265] block comment should start with '# '
var/spack/repos/builtin/packages/idl/package.py:40: [W293] blank line contains whitespace
var/spack/repos/builtin/packages/idl/package.py:44: [W293] blank line contains whitespace

@adamjstewart adamjstewart merged commit 32b070a into spack:develop Jul 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adamjstewart adamjstewart adamjstewart approved these changes

Assignees

No one assigned

Labels

new-package

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@francinelapid @adamjstewart