Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create recipe for igphyml #18930

Closed
wants to merge 7 commits into from
Closed

Create recipe for igphyml #18930

wants to merge 7 commits into from

Conversation

PertuyF
Copy link
Contributor

@PertuyF PertuyF commented Nov 26, 2019

Bioconda requires reviews prior to merging pull-requests (PRs). To facilitate this, once your PR is passing tests and ready to be merged, please add the please review & merge label so other members of the bioconda community can have a look at your PR and either make suggestions or merge it. Note that if you are not already a member of the bioconda project (meaning that you can't add this label), please ping @bioconda/core so that your PR can be reviewed and merged (please note if you'd like to be added to the bioconda project). Please see #15332 for more details.

  • I have read the guidelines for bioconda recipes.
  • This PR adds a new recipe.
  • AFAIK, this recipe is directly relevant to the biological sciences (otherwise, please submit to the more general purpose conda-forge channel).
  • This PR updates an existing recipe.
  • This PR does something else (explain below).

@dpryan79
Copy link
Contributor

I have no clue why automake is constructing a borked Makefile on OSX.

@PertuyF
Copy link
Contributor Author

PertuyF commented Nov 27, 2019

Hi @dpryan79 ,
Thanks for your efforts with OSX build! there are specific instructions for this in the doc.

Likely CC is already defined in the docker image at build time, but MACLLVM might be missing?

@dpryan79
Copy link
Contributor

Yeah, CC is set, but MACLLVM not. This package is quite the mess though, I wonder if it's easier to not use its build script.

@PertuyF
Copy link
Contributor Author

PertuyF commented Nov 27, 2019

As you see fit, you have much more experience than me in this domain.
If you can help with what to do, I can find some time to test.

@dpryan79 dpryan79 mentioned this pull request Nov 27, 2019
5 tasks
@PertuyF
Copy link
Contributor Author

PertuyF commented Nov 27, 2019

Closing as replaced by #18945

@PertuyF PertuyF closed this Nov 27, 2019
@PertuyF PertuyF deleted the igphyml branch November 27, 2019 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants