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

Deps #72

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Deps #72

wants to merge 4 commits into from

Conversation

gidden
Copy link
Member

@gidden gidden commented Jun 2, 2015

these are all the changes I needed to make to get a fully-functional cyclus-deps conda package

@@ -1,30 +1,3 @@
#/bin/bash
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe this just means I don't understand conda well enough, but why don't we need any of this anymore?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First, I think conda explicitly calls bash <script name>. Second, this was missing the shebang anyway and wouldn't have worked =/

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was wondering about the whole file being deleted but just tagged it on this line.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah (it is mostly being delted, not wholy, FWIW). This looks to have been
directly copied from the cyclus conda recipe. It assumes that cyclus was
actually installed (hence playing with the executable name). Cyclus-deps
does not install cyclus.

On Wed, Jun 3, 2015 at 10:40 AM, Paul Wilson [email protected]
wrote:

In conda-recipes/cyclus-deps/post-link.sh
#72 (comment):

@@ -1,30 +1,3 @@
-#/bin/bash

I was wondering about the whole file being deleted but just tagged it on
this line.


Reply to this email directly or view it on GitHub
https://github.com/cyclus/release/pull/72/files#r31636783.

Matthew Gidden, Ph.D.
Postdoctoral Associate, Nuclear Engineering
The University of Wisconsin -- Madison
Ph. 225.892.3192

@gidden
Copy link
Member Author

gidden commented Jun 3, 2015

ping

@gonuke
Copy link
Member

gonuke commented Jun 4, 2015

What's the best way to test this?

@gidden
Copy link
Member Author

gidden commented Jun 4, 2015

Feel free to upload some conda packages
On Jun 3, 2015 9:58 PM, "Paul Wilson" [email protected] wrote:

What's the best way to test this?


Reply to this email directly or view it on GitHub
#72 (comment).

@rwcarlsen
Copy link
Member

@gidden, @gonuke - is this still relevant and up to date?

@gidden
Copy link
Member Author

gidden commented May 8, 2016

As of ~1 year ago, this was what I needed to do to get cyclus to install from source using conda as the dependency package manager. I thought I saw something pass the list a while ago where @scopatz had also done some work in this regard? Perhaps not.. In any case, I don't know of any reason why this wouldn't work now.

@scopatz
Copy link
Member

scopatz commented May 9, 2016

I don't think it is up to date, but it is still relevant. I have been working on trying to get a cyclus conda package built via conda-forge, but cyclus has a lot of terrible dependencies (glib, etc) that make this difficult. Still hammering on it though

@rwcarlsen
Copy link
Member

I think glib comes via libxml2 :-/ Keep us posted on any progress @scopatz.

@scopatz
Copy link
Member

scopatz commented May 11, 2016

Well, glib is dependency of glibmm which is a dependency of libxml++. Currently libxml++ is building on conda-forge (finally _). All that is left before I can try a cyclus package are coin and gtest.

@gonuke
Copy link
Member

gonuke commented Nov 25, 2019

I think this has been superseded by other activity by @bam241 on conda/conda-forge

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.

4 participants