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

generator.exe is now working #3497

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

generator.exe is now working #3497

wants to merge 3 commits into from

Conversation

ya4ept
Copy link
Contributor

@ya4ept ya4ept commented Jul 30, 2021

generator.exe does not work. At startup, it gives an error:
image
because it by default looks for file 1.pl which doesn't exist.

The generator.exe should actually run the generator.pl:
I fixed it:
image


Known issues:

  • The resulting file cannot pass command line parameters
  • replace "ActivePerl" with "StrawberryPerl" ?

@ya4ept ya4ept mentioned this pull request Jul 30, 2021
10 tasks
@ya4ept ya4ept added the bug label Jul 30, 2021
@ya4ept ya4ept requested a review from alisonrag July 30, 2021 08:56
@ya4ept ya4ept added this to In progress in OpenKore via automation Nov 22, 2021
@ya4ept ya4ept moved this from In progress to TODO in OpenKore Nov 22, 2021
@ya4ept
Copy link
Contributor Author

ya4ept commented Aug 29, 2022

mingw32-make.exe
g++ -Wall launcher.cpp utils.cpp core.cpp hstring.c ui.o -o launcher.exe -mno-cygwin -s
launcher.cpp: In function 'int main(int, char**)':
launcher.cpp:71: warning: deprecated conversion from string constant to 'char*'

but the launcher.exe file is generated

@alisonrag
Copy link
Contributor

some dlls is missing, maybe need to add flags in compile to include it
image

@alisonrag
Copy link
Contributor

what about?
https://stackoverflow.com/a/12922118

@alisonrag
Copy link
Contributor

some dlls is missing, maybe need to add flags in compile to include it image

@ya4ept

@ghost
Copy link

ghost commented Nov 16, 2022

I do recommend use library instead dlls. About import!
Then the suggestion is remove other functions and leave only necessary to run the code maybe after reach kernelbase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants