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.

Sign up for GitHub

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

Jump to bottom

start emake release #107

Merged
merged 38 commits into from
Apr 9, 2020
Merged

start emake release #107

Changes from 1 commit
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
124df96
start emake release
Mar 20, 2020
d39c172
build emake
Mar 23, 2020
81c9cb3
merge master
Mar 23, 2020
8d96b91
update submodule
Mar 23, 2020
610fa38
Update ci-build.sh
fundies Mar 23, 2020
435a11a
Update ServerPlugin.cpp
fundies Mar 24, 2020
fbecc6e
Update CMakeLists.txt
fundies Mar 24, 2020
7fd0034
Update CMakeLists.txt
fundies Mar 24, 2020
cadb4a1
Update CMakeLists.txt
fundies Mar 24, 2020
2792f7b
Update ServerPlugin.cpp
fundies Mar 24, 2020
ade17aa
Update ci-build.sh
fundies Mar 24, 2020
d29092f
Update CMakeLists.txt
fundies Mar 24, 2020
d86b543
Append path, not replace it.
RobertBColton Apr 1, 2020
1200733
Actually capture MSYS2 environment.
RobertBColton Apr 1, 2020
1bedc92
oopsie
RobertBColton Apr 1, 2020
68465f7
Actually capture the MSYS2 environment.
RobertBColton Apr 1, 2020
dd94de7
best guess at hack
fundies Apr 1, 2020
82882bb
Update azure-jobs.yml
fundies Apr 1, 2020
289d739
noquote
RobertBColton Apr 2, 2020
f2d1ba0
Update CMakeLists.txt
fundies Apr 3, 2020
fec0ba0
Log QProcess standard output & error
RobertBColton Apr 5, 2020
140fdf3
Update ServerPlugin.cpp
RobertBColton Apr 6, 2020
638b461
Update ServerPlugin.cpp
RobertBColton Apr 6, 2020
05974fb
Update ServerPlugin.cpp
RobertBColton Apr 7, 2020
f0f64ff
Update ServerPlugin.cpp
RobertBColton Apr 7, 2020
7265a15
Update ServerPlugin.cpp
RobertBColton Apr 7, 2020
e200f26
Update ServerPlugin.cpp
RobertBColton Apr 7, 2020
f882884
Update ServerPlugin.cpp
RobertBColton Apr 7, 2020
5712ff5
Update ServerPlugin.cpp
RobertBColton Apr 7, 2020
1c0c3e2
Update ServerPlugin.cpp
RobertBColton Apr 7, 2020
1ab7ca0
Update ServerPlugin.cpp
RobertBColton Apr 7, 2020
dec11be
Update ServerPlugin.cpp
RobertBColton Apr 7, 2020
4463e12
Update ServerPlugin.cpp
RobertBColton Apr 8, 2020
1e945b4
oopsie
RobertBColton Apr 8, 2020
c055532
use boolean operators
RobertBColton Apr 9, 2020
dc8723e
Use cmake_dependent_option
RobertBColton Apr 9, 2020
1dba8d6
Update CMakeLists.txt
fundies Apr 9, 2020
77d6c85
update submodule
Apr 9, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Plugins/ServerPlugin.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ ServerPlugin::ServerPlugin(MainWindow& mainWindow) : RGMPlugin(mainWindow) {
qDebug().noquote() << "Environmental variable \"MSYS_ROOT\" is not set defaulting MSYS path to: " + msysPath;
} else msysPath = env.value("MSYS_ROOT");

msys2Proc->start(msysPath + "/msys2_shell.cmd");
msys2Proc->start(msysPath + "/msys2_shell.cmd -mingw64");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This did not fix the missing draw_* completions. However, I looked some more and noticed that all of the graphics_mandatory.h functions are available, such as screen_refresh and the like. All of the window functions are also available, leading me to believe we just have the graphics system set to None or something.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did it fix all those incude errrs for map n etc in jdi log?

Copy link
Contributor

@RobertBColton RobertBColton Apr 6, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All I know is I compared the log with regular normal emake that I know works and the one from RGM is way longer. I notice a bunch of stuff about var4 but I did not see anything about map in there.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://pastebin.com/Fjw9gshc like line 73 in your earlier paste. are you still seeing those errors?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two files are identical according to online diff viewer.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

im fairly certain thats the issue. i doubt youll see those errors launching emake normally

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try 638b461

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still nothing, are you sure it's not just the JDI STL hacks to fix string or something?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did 1c0c3e2 which just printed empty. So that means the environment is not copying anything. As I said, I believe the process is not having enough time to actually initialize.

msys2Proc->waitForStarted();
QProcessEnvironment msys2Env = msys2Proc->processEnvironment();
process->setProcessEnvironment(msys2Env);
Expand Down