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

DTXV.exe requires xadec.dll to run #62

Closed
wants to merge 1 commit into from
Closed

DTXV.exe requires xadec.dll to run #62

wants to merge 1 commit into from

Conversation

987123879113
Copy link
Contributor

xadec.dll needs to be in the same folder as DTXV.exe or else you get an error trying to run DTXV.exe. It exists in the dll folder but DTXV.exe can't see that folder.

@ericpignet
Copy link
Collaborator

Thanks for noticing the issue, to be honest I forgot to try DTXV after the recent migration.
However I don't like having DLLs in the main directory...
DTXV.exe could set the DLL directory in the code with SetDllDirectory(), but I'm not sure where DTXV code is. Actually I'm not sure it's the right thing to include DTXV.exe in the DTXmaniaNX repository.
If it's free software, we should grab the source code and adapt it to work with our directory structure.
If it is not, we are probably not allowed to include it in the repo anyway.

If we don't want it in the repo (my preference) but we still want the users to have it to use with DTXCreator, it could still be added at packaging time...

I'm interested in @limyz / @fisyher opinion.

@limyz
Copy link
Owner

limyz commented Sep 29, 2020

We will need to include DTXViewer from the original dtxmania, it was not included in the verk fork for some reason.

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

3 participants