fletch icon indicating copy to clipboard operation
fletch copied to clipboard

Qt 5.11.2 fails to build in Visual Studio 2017

Open mleotta opened this issue 6 years ago • 7 comments

Fletch build fails with Qt 5.11.2 on Windows (Visual Studio 2017). The error seems to be this known issue:

https://bugreports.qt.io/browse/QTBUG-66652?attachmentSortBy=dateTime

I suppose we will need to find a patch for this.

mleotta avatar Sep 27 '18 20:09 mleotta

According to that link, the build should fail when the path to the QT base directory is 63 or more characters. Our dashboard comes in at 67 characters... I shortened the path and I am running the dashboard now to confirm that this is indeed the problem. Looks like Qt doesn't even have a patch for this problem (yet).

Qt5 in KWIVER is in a weird state for me currently, I cannot seem to get version 5.11.2 to compile successfully on any platforms (win/mac/linux). I'd like to know if any others experience the same troubles..

Cookt2 avatar Oct 01 '18 20:10 Cookt2

I have built it with the latest msvc 2017 several times now

On Mon, Oct 1, 2018, 4:50 PM Taylor J. Cook [email protected] wrote:

According to that link, the build should fail when the path to the QT base directory is 63 or more characters. Our dashboard comes in at 67 characters... I shortened the path and I am running the dashboard now to confirm that this is indeed the problem. Looks like Qt doesn't even have a patch for this problem (yet).

Qt5 in KWIVER is in a weird state for me currently, I cannot seem to get version 5.11.2 to compile successfully on any platforms (win/mac/linxu). I'd like to know if any others experience the same troubles..

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Kitware/fletch/issues/451#issuecomment-426058048, or mute the thread https://github.com/notifications/unsubscribe-auth/AadLH_ppfPY5hzLW4B2MqfaxY-Pad0ebks5ugoAHgaJpZM4W9Xxw .

aaron-bray avatar Oct 01 '18 21:10 aaron-bray

Taylor is correct. The build will work unless you extend the length of your build directory. There is apparently a setting in Windows 10 preview(?) that allows you to disable that restriction, though I can't imagine why it's not just the default.

https://mspoweruser.com/ntfs-260-character-windows-10/

FWIW, we've had the same issue with Boost for a while now. I don't know if there is really any good solution other than, shorten your path or disable the NTFS silliness. It's a Windows limitation, not Qt or our build. At some point, every package will fail because if it, provided you make the build path too long.

On Mon, Oct 1, 2018 at 5:29 PM Aaron Bray [email protected] wrote:

I have built it with the latest msvc 2017 several times now

On Mon, Oct 1, 2018, 4:50 PM Taylor J. Cook [email protected] wrote:

According to that link, the build should fail when the path to the QT base directory is 63 or more characters. Our dashboard comes in at 67 characters... I shortened the path and I am running the dashboard now to confirm that this is indeed the problem. Looks like Qt doesn't even have a patch for this problem (yet).

Qt5 in KWIVER is in a weird state for me currently, I cannot seem to get version 5.11.2 to compile successfully on any platforms (win/mac/linxu). I'd like to know if any others experience the same troubles..

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Kitware/fletch/issues/451#issuecomment-426058048, or mute the thread < https://github.com/notifications/unsubscribe-auth/AadLH_ppfPY5hzLW4B2MqfaxY-Pad0ebks5ugoAHgaJpZM4W9Xxw

.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Kitware/fletch/issues/451#issuecomment-426069711, or mute the thread https://github.com/notifications/unsubscribe-auth/AFRAWrzeSLrOrUjKxMS-4J_sm2VGw8cuks5ugolJgaJpZM4W9Xxw .

-- David Stoup Principal Engineer

Kitware, Inc. 28 Corporate Drive Clifton Park, NY. 12065 518-881-4949 (W) 518-312-3946 (M) 518-371-4573 (F)

dstoup avatar Oct 02 '18 11:10 dstoup

My path to Qt has 62 characters, unless I'm counting wrong, and I hit this issue.

C:\Users\matt.leotta\Software\fletch\bld-vs15-rel\build\src\Qt

I could shorten my path, but I expect other KWIVER users would easily hit this bug if they put fletch in their home directory like I did.

On Tue, Oct 2, 2018 at 7:42 AM David Stoup [email protected] wrote:

Taylor is correct. The build will work unless you extend the length of your build directory. There is apparently a setting in Windows 10 preview(?) that allows you to disable that restriction, though I can't imaging why it's not just the default.

https://mspoweruser.com/ntfs-260-character-windows-10/

On Mon, Oct 1, 2018 at 5:29 PM Aaron Bray [email protected] wrote:

I have built it with the latest msvc 2017 several times now

On Mon, Oct 1, 2018, 4:50 PM Taylor J. Cook [email protected] wrote:

According to that link, the build should fail when the path to the QT base directory is 63 or more characters. Our dashboard comes in at 67 characters... I shortened the path and I am running the dashboard now to confirm that this is indeed the problem. Looks like Qt doesn't even have a patch for this problem (yet).

Qt5 in KWIVER is in a weird state for me currently, I cannot seem to get version 5.11.2 to compile successfully on any platforms (win/mac/linxu). I'd like to know if any others experience the same troubles..

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Kitware/fletch/issues/451#issuecomment-426058048, or mute the thread <

https://github.com/notifications/unsubscribe-auth/AadLH_ppfPY5hzLW4B2MqfaxY-Pad0ebks5ugoAHgaJpZM4W9Xxw

.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Kitware/fletch/issues/451#issuecomment-426069711, or mute the thread < https://github.com/notifications/unsubscribe-auth/AFRAWrzeSLrOrUjKxMS-4J_sm2VGw8cuks5ugolJgaJpZM4W9Xxw

.

-- David Stoup Principal Engineer

Kitware, Inc. 28 Corporate Drive Clifton Park, NY. 12065 518-881-4949 (W) 518-312-3946 (M) 518-371-4573 (F)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Kitware/fletch/issues/451#issuecomment-426242609, or mute the thread https://github.com/notifications/unsubscribe-auth/AAybMawU6UZ1zFSeK4U4H5uz1dVoSHyIks5ug1EYgaJpZM4W9Xxw .

mleotta avatar Oct 02 '18 12:10 mleotta

Qt 5.11.2 is building successfully on VS2017. This success looks to be a result of limiting the build path, and a few patches to master.

Nightly: https://open.cdash.org/buildSummary.php?buildid=5586897

Can this issue be closed?

Cookt2 avatar Oct 19 '18 18:10 Cookt2

If the solution requires that you build Fletch in a path of limited character length, then I think we need to document that requirement in the build instructions before closing this issue.

mleotta avatar Oct 22 '18 01:10 mleotta

I agree with Matt. Further, as long as that's the solution, I would prefer this ticket remain open as a reminder that the real solution needs to be found and integrated, even if it's just monitoring upstream to see if they have finally addressed it.

dstoup avatar Oct 22 '18 12:10 dstoup