added Linux support - Githubissues

nicogodet / telemac-mascaret-feedstock

https://anaconda.org/nicogodet/opentelemac

2 stars 2 forks source link

added Linux support #6

Closed tomsail closed 11 months ago

tomsail commented 11 months ago

MUMPS works fine added docs and examples for linux switched to v8p5

@nicogodet major change: recipe/meta.yaml > line 42 (build >> host) build.sh script is not recognised on run part (for Linux at least)

Have a look if this stills works on windows. Otherwise we'll have to differenciate this as well (with #[win]/#[not win] flags)

I'll add all changes to my repo as well, adding my fork here as upstream branch

nicogodet commented 11 months ago

Could you resolve conflict and tell me if this is ready for merge ?

tomsail commented 11 months ago

Could you resolve conflict and tell me if this is ready for merge ?

Yes, I just need to rebase (you've changed the paths of the scripts).

tomsail commented 11 months ago

Actually then if you want me to resolve the conflicts that will occur anyway in meta.yaml, I'll be happy to do it. You just need to give me write access to this git

nicogodet commented 11 months ago

I sent you an invite

tomsail commented 11 months ago

Thanks. It seems anyway that the rebase fixed it :)

tomsail commented 11 months ago

Concluding:

For Linux: MUMPS, GOTM, AED with API have implemented. MED still doesn't work.
For macOs: I'll ask @brey to try and compile on his mac when the PR has been merged.
For windows: I didn't follow

You can close #7 with this PR

tomsail commented 11 months ago

I'll also set this repo as upstream branch on my repo, so that we are syncronized