Skip to content
This repository has been archived by the owner on Feb 24, 2024. It is now read-only.
/ registry Public archive

Initial Commit of apfel #99

Closed
wants to merge 1 commit into from
Closed

Initial Commit of apfel #99

wants to merge 1 commit into from

Conversation

alecbcs
Copy link
Contributor

@alecbcs alecbcs commented Mar 16, 2021

No description provided.

@alecbcs
Copy link
Contributor Author

alecbcs commented Mar 17, 2021

@vsoch what do you make of this? It seems like it must be built with python instead of python just being a variant? I'm I missing something?

@vsoch
Copy link
Collaborator

vsoch commented Mar 17, 2021

if Python is a variant there must be a use case of not wanting it - but I don't know what that would be :) I would just add +python (this line) https://github.com/autamus/registry/pull/99/files#diff-cf0503e86d12eb335abda11215497c99b55d68409cdb6d009bff1e13e63424b1R23.

@alecbcs
Copy link
Contributor Author

alecbcs commented Mar 17, 2021

Right, I think the problem might be that right now we have a good way of defining custom Spack install options. This is something I'm currently working on for when we add the ability for custom multi-package containers. I'll hold off on this PR until then and then I'm guessing we'll just make an apfel container with this package installed.

@alecbcs
Copy link
Contributor Author

alecbcs commented Apr 15, 2021

I'm going to close this PR until we've implemented multi-package support if that's alright.

@alecbcs alecbcs closed this Apr 15, 2021
@alecbcs alecbcs deleted the include-apfel branch April 15, 2021 06:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants