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

Running HPX applications on Windows should not require setting %PATH% #503

Closed
brycelelbach opened this issue Aug 22, 2012 · 3 comments

Comments

Projects
None yet
3 participants
@brycelelbach
Copy link
Member

commented Aug 22, 2012

Users shouldn't have to set the %PATH% environment variable to run HPX applications on Windows.

Someone should be able to download HPX, run cmake, build it, and then run it, without doing anything else.

@sithhell

This comment has been minimized.

Copy link
Member

commented Aug 22, 2012

On Tuesday, August 21, 2012 18:41:20 Bryce Adelstein-Lelbach wrote:

Users shouldn't have to set the %PATH% environment variable to run HPX
applications on Windows.
Someone should be able to download HPX, run cmake, build it, and then run it,
without doing anything else.

Reply to this email directly or view it on GitHub.

Couldn't cmake set the PATH in this case?

@brycelelbach

This comment has been minimized.

Copy link
Member Author

commented Aug 22, 2012

Copying over Hartmut's comment from #451:

I'm not sure if this is a good idea.

  • we should maintain the same installation structure on all platforms
  • letting Windows rely on finding the DLL's in the local directory is exactly what causes 'DLL Hell' and is deemed to be general design flaw in Windows.

I'd rather suggest adding an application manifest or an application configuration file (or both) allowing to direct Windows to find the DLL's where they are (in a private application assembly (see http://msdn.microsoft.com/en-us/library/aa375674(v=vs.110).aspx)

@hkaiser

This comment has been minimized.

Copy link
Member

commented Sep 13, 2014

This has been fixed by #1204

@hkaiser hkaiser closed this Sep 13, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.