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

Suggested fix for getting the correct registry key for XE2 C++ and up… #57

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Suggested fix for getting the correct registry key for XE2 C++ and up… #57

wants to merge 1 commit into from

Conversation

MHumm
Copy link
Contributor

@MHumm MHumm commented Apr 16, 2019

… as of Mantis issue 6394
http://issuetracker.delphi-jedi.org/view.php?id=6394
The discussion here gives the impression that \Win32 is needed in the C++ XE2 and above registry key.

@obones
Copy link
Member

obones commented Apr 17, 2019

This suggested fix cannot work, there are multiple sub keys in the registry, of which two are of interest to the JCL (and JVCL for that matter): Win32 and Win64

So this cannot be placed in a constant.
I've had a look at the registry for the installs I have here, and the LibraryPath is already populated properly (ie, in both subkeys) for BDS11 and upper.
The SearchPath, however, seems to be missing the subkey.

A more thorough investigation needs to be done in order to write a proper fix.

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.

2 participants