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

Update index.rst #960

Merged
merged 1 commit into from Feb 9, 2016
Merged

Update index.rst #960

merged 1 commit into from Feb 9, 2016

Conversation

saha84
Copy link
Contributor

@saha84 saha84 commented Feb 8, 2016

For the path of the shared library in Linux, it is enough to put the 'libCoolProp.so' in '/usr/lib/scilab' and use the sample.sce file as it is now. I don't know the equivalent folder in windows.

note: As I say in my first comment, "English is not my mother tongue; please excuse any errors on my part."

For the path of the shared library in Linux, it is enough to put the 'libCoolProp.so' in '/usr/lib/scilab' and use the sample.sce file as it is now. I don't know the equivalent folder in windows.

note: As I say in my first comment, "English is not my mother tongue; please excuse any errors on my part."
@ibell
Copy link
Contributor

ibell commented Feb 9, 2016

Can you also add that comment about the shared library? Didn't know about that. I'm merging this PR.

ibell added a commit that referenced this pull request Feb 9, 2016
@ibell ibell merged commit ef400b9 into CoolProp:master Feb 9, 2016
@saha84
Copy link
Contributor Author

saha84 commented Feb 9, 2016

hello,

I just updated the comment about the shared library, but I don't see it anywhere.

Sorry this is the first time I do this thing in github

@ibell
Copy link
Contributor

ibell commented Feb 9, 2016

I think you need to make a new pull request. The .rst files will get
compiled into the website automatically, so your changes you made in the
last pull are here:
http://www.coolprop.dreamhosters.com/binaries/sphinx/coolprop/wrappers/Scilab/index.html#scilab

On Tue, Feb 9, 2016 at 3:31 AM, saha84 notifications@github.com wrote:

hello,

I just updated the comment about the shared library, but I don't see it
anywhere.

Sorry this is the first time I do this thing in github


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

@ibell ibell added this to the v5.1.3 milestone Feb 9, 2016
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.

None yet

2 participants