- changed status to open
- removed comment
ExternalThorns/pciutils ignores most option list options
Issue #1657
closed
It does not contain a configure script and its Makefile hard-codes the compiler and linker to be gcc. This is an issue if LDFLAGS (or CFLAGS possibly) contain -openmp like they do when using the intel compiler. Also using mkl could (should) be done with the -mkl switch to the intel compiler but gcc (as used in pciutils) will naturally not accept this.
Keyword: pciutils
Comments (5)
-
reporter
-
Changing priority to minor as the thorn has been removed from the ET thornlist. As always, please re-raise if you disagree.
-
- changed status to open
- removed comment
Looks good without trying: please apply.
-
- changed status to resolved
- removed comment
applied in r28.
-
reporter
- edited description
- changed status to closed
- Log in to comment
- Assignee
- –
- Type
- Priority
- Status
- closed
- Component
- EinsteinToolkit thorn
- Milestone
- –
- Version
- development version
- Votes
- 0
- Watchers
- 0
This patch removes the hard-coded CC and AR from the Makefile and builds pciutils using MKL and -openmp flags with the intel compiler. It also builds on a stock Linux (Debian Squeeze) system where gcc is used. I have not tested this on a Mac (see
#1658).