crash when importing files from directory with too long path on Windows about plater HOT 14 OPEN

some add. info: seems also happening if path + filename in total get too long

from plater.

Can you try the binaries from here:
https://drive.google.com/open?id=1USaJTZzJk_7KIQr8ctoPnoI9gQ6FYcY9

from plater.

@combolek what fix did you make?

It can be also related to special chars (here the ö)

from plater.

Hi, thanks for the download @combolek . But the given executable has the same problem. Indeed i've overseen the special character Ö which is typical in europe. In general UTF-8 might be good solution because many countries have à á â and other chars in their daily used names.

I now renamed the folder and it works (but in my case a bad solution)

from plater.

Ok @vmario89, I'm not saying you shouldn't use UTF-8 characters, but just trying to isolate what makes the program crash
Do you confirm that the same PATH with and without the special char have different behavior?

from plater.

hi @Gregwar, yes the "ö" in path is the problem. I tested it with

"C:\Users\mario\Seafile\cloud.123456-STLEXPOR.DD\Gedöns\123456Design\chainy_v2.stl" -> fail
"C:\Users\mario\Seafile\cloud.123456-STLEXPOR.DD\Gedons\123456Design\chainy_v2.stl" -> good

regards, Mario

from plater.

I was all wrong, ignore me.

from plater.

For the native MSVC build the following workaround works for me:

--- a/gui/main.cpp
+++ b/gui/main.cpp
@@ -13,6 +13,10 @@ Q_IMPORT_PLUGIN(QWindowsVistaStylePlugin);
 
 int main(int argc, char *argv[])
 {
+#if defined(_MSC_VER)
+    // set default encoding for fopen/chdir/etc.
+    setlocale(LC_CTYPE, ".utf8");
+#endif
     QApplication a(argc, argv);
 
     QGLFormat glf = QGLFormat::defaultFormat();

@vmario89 can you try these binaries?
https://drive.google.com/file/d/1ZdfkoLtuxTc07pidsPaVaUHsK0e3Y6kh/view?usp=sharing

But even if this works, it's only a partial solution because it cannot handle command line arguments (so it won't work for the plater.exe command line app). Also, I think the official build uses mingw (?) and this solution does not work then either.

from plater.

I figured out a better way to fix it, see PR #30. It requires Windows 10 1903 or later but it also handles paths on the command line.

@vmario89 can you try these updated binaries?
https://drive.google.com/drive/folders/1USaJTZzJk_7KIQr8ctoPnoI9gQ6FYcY9

from plater.

Stopped crashing for me with your latest updated binaries @combolek

from plater.

Great, thanks for testing. Let's hope PR #30 will be merged.

from plater.

It is done

from plater.

Oh nice! Would it be possible to provide a recent binary download / release set?

regards, Mario :)

from plater.

Did you try the ones from the link above?

from plater.