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

Use Alien::HDF4 in PDLA::IO::HDF build #15

Merged
merged 1 commit into from Aug 29, 2015
Merged

Use Alien::HDF4 in PDLA::IO::HDF build #15

merged 1 commit into from Aug 29, 2015

Conversation

mohawk2
Copy link
Member

@mohawk2 mohawk2 commented Aug 29, 2015

No description provided.

@zmughal
Copy link
Member

zmughal commented Aug 29, 2015

Could you add Alien::HDF4 to the appveyor.yml as well?

@mohawk2
Copy link
Member Author

mohawk2 commented Aug 29, 2015

Instead I've been bold and added it as a CONFIGURE_REQUIRES dep. That'll mean wanting to run it through Travis again as well as Appveyor.

@zmughal
Copy link
Member

zmughal commented Aug 29, 2015

Looks good.

We will need to return to why the dmake test target fails in Appveyor because failing at the build stage will mask any failures in all our other changes. A little worrying, no?

Merge: Aye

@mohawk2
Copy link
Member Author

mohawk2 commented Aug 29, 2015

It's an issue but it's probably my fault on some level. I have dmake available here for fast dev when we reach that stage. Merging.

@mohawk2 mohawk2 merged commit 2edeb0e into master Aug 29, 2015
@mohawk2 mohawk2 deleted the alienhdf4 branch August 29, 2015 20:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants