Opened 3 years ago

Closed 3 years ago

#7961 closed defect (fixed)

Clipper fails to build in Windows - Nov 3 release candidate

Reported by: Tristan Croll Owned by: Zach Pearson
Priority: blocker Milestone:
Component: Build System Version:
Keywords: Cc: chimera-programmers
Blocked By: Blocking:
Notify when closed: Platform: all
Project: ChimeraX

Description

Installed the Nov 3 release candidate and attempted to rebuild Clipper, leading to the following error:

c:\program files\chimerax\bin\lib\site-packages\chimerax\atomic_lib\include\atomstruct\string_types.h(20): fatal error C1083: Cannot open include file: 'chutil/CString.h': No such file or directory

Here's what my dependency specification currently looks like. Do I need something new in there?

  <Dependencies>
    <Dependency name="ChimeraX-Core" build="true" version="&gt;=1.5rc202211032258, &lt;1.6"/>
    <Dependency name="ChimeraX-Atomic" version="~=1.36"/>
    <Dependency name="ChimeraX-AtomicLibrary" build="true" version="~=8.0"/>
    <Dependency name="ChimeraX-Arrays" build="true" version="~=1.0"/>
  </Dependencies>

Attachments (1)

make.out (2.1 MB ) - added by Tristan Croll 3 years ago.
Added by email2trac

Change History (19)

comment:1 by pett, 3 years ago

Cc: chimera-programmers added
Milestone: 1.5
Owner: changed from pett to Zach Pearson

comment:2 by Zach Pearson, 3 years ago

Can you post the full log here? This is surely something I changed recently in the Windows builds, but I'd like to see the invocation of the compiler so I know what directories were included.

in reply to:  3 ; comment:3 by Tristan Croll, 3 years ago

Here you go.

On Mon, Nov 7, 2022 at 5:13 PM ChimeraX <ChimeraX-bugs-admin@cgl.ucsf.edu>
wrote:

make.out

by Tristan Croll, 3 years ago

Attachment: make.out added

Added by email2trac

comment:4 by Zach Pearson, 3 years ago

We used to move the core headers and libraries up to the top-level application directory, but we have a new API to find header and library files that the core now uses with the new TOML bundle builder. XML bundle builder does not know how to find those libraries. I had made a half-baked attempt to teach it how, but I can see that I missed some edge cases. I'll download Clipper and ensure it builds on the daily build, then merge the changes into the release candidates to test again, and finally update this ticket when it builds on the RC.

Thanks for reporting this! I'd hate for this bug to have shipped.

in reply to:  6 comment:5 by Tristan Croll, 3 years ago

Maybe I should also be exploring the TOML option (after 1.5 has shipped, of
course), if that’s your preferred option for the future!

On Mon, 7 Nov 2022 at 18:43, ChimeraX <ChimeraX-bugs-admin@cgl.ucsf.edu>
wrote:

comment:6 by Zach Pearson, 3 years ago

I would hold off on it until we're sure that it works with the toolshed, but once that's done I'd encourage a switch.

comment:7 by Zach Pearson, 3 years ago

Status: assignedfeedback

I backported the slightly smarter lib finder I put in XML bundle builder then built clipper with the latest release/v1.5 code. Should work tomorrow (or you can build ChimeraX to test).

in reply to:  9 comment:8 by Tristan Croll, 3 years ago

I’ll wait for the new build. :)

On Mon, 7 Nov 2022 at 20:57, ChimeraX <ChimeraX-bugs-admin@cgl.ucsf.edu>
wrote:

in reply to:  10 ; comment:9 by Tristan Croll, 3 years ago

Tried it in today's build ( built: 2022-11-08 02:21:20 PST committed:
2022-11-08 00:09:12 PST). Still fails with the same error, I'm afraid.

On Mon, Nov 7, 2022 at 8:59 PM ChimeraX <ChimeraX-bugs-admin@cgl.ucsf.edu>
wrote:

in reply to:  11 ; comment:10 by Tristan Croll, 3 years ago

I can get a successful build after copying C:\Program
Files\ChimeraX\bin\Lib\site-packages\chimerax\core\include to C:\Program
Files\ChimeraX\include. Doesn't fix the bug, but gives me a working bundle
for now. :)

On Tue, Nov 8, 2022 at 4:22 PM ChimeraX <ChimeraX-bugs-admin@cgl.ucsf.edu>
wrote:

comment:11 by Zach Pearson, 3 years ago

OK, the bundle builder code was working but the core on the release branch did not have the get_include/lib code that the core on the develop branch had. *Now* it should work tomorrow. :)

comment:12 by Zach Pearson, 3 years ago

Did it build?

in reply to:  14 comment:13 by Tristan Croll, 3 years ago

Sorry - haven't done a build since last Tuesday (the one I got to work by
manually copying the include directory). Got a bit sidetracked by other
things. Will download the latest ChimeraX tomorrow and see how it goes.

On Wed, Nov 16, 2022 at 9:31 PM ChimeraX <ChimeraX-bugs-admin@cgl.ucsf.edu>
wrote:

in reply to:  15 ; comment:14 by Tristan Croll, 3 years ago

Still failing in the same way using the Nov 15 chimerax-rc build, I'm
afraid. Here's an example of the compiler command:


On Wed, Nov 16, 2022 at 9:38 PM ChimeraX <ChimeraX-bugs-admin@cgl.ucsf.edu>
wrote:

in reply to:  16 ; comment:15 by Tristan Croll, 3 years ago

Sorry... accidentally hit shift-return (which apparently equates to "send
now" in GMail). *Here's* an example of the compiler command:

"C:\Program Files\Microsoft Visual
Studio\2022\Professional\VC\Tools\MSVC\14.16.27023\bin\HostX86\x64\cl.exe"
/c /nologo /O2 /W3 /GL /DNDEBUG /MD -DCLIPPER_CX_EXPORTS -DCLIPPER_VERBOSE
-DNOMINMAX -DDYNAMIC_LIBRARY=1 "-Ic:\Program Files\ChimeraX\include"
"-Ic:\Program Files\ChimeraX\bin\lib\site-packages\numpy\core\include"
-Isrc/deps "-Ic:\Program
Files\ChimeraX\bin\lib\site-packages\chimerax\atomic_lib\include"
"-Ic:\Program Files\ChimeraX\bin\lib\site-packages\chimerax\arrays\include"
"-Ic:\Program Files\ChimeraX\bin\include" "-IC:\Program Files\Microsoft
Visual Studio\2022\Professional\VC\Tools\MSVC\14.16.27023\include"
"-IC:\Program Files\Microsoft Visual
Studio\2022\Professional\VC\Auxiliary\VS\include" "-IC:\Program Files
(x86)\Windows Kits\10\include\10.0.19041.0\ucrt" "-IC:\Program Files
(x86)\Windows Kits\10\\include\10.0.19041.0\\um" "-IC:\Program Files
(x86)\Windows Kits\10\\include\10.0.19041.0\\shared" "-IC:\Program Files
(x86)\Windows Kits\10\\include\10.0.19041.0\\winrt" "-IC:\Program Files
(x86)\Windows Kits\10\\include\10.0.19041.0\\cppwinrt" "-IC:\Program
Files\Microsoft Visual
Studio\2022\Professional\VC\Tools\MSVC\14.16.27023\include" "-IC:\Program
Files\Microsoft Visual Studio\2022\Professional\VC\Auxiliary\VS\include"
"-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\ucrt"
"-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.19041.0\\um"
"-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.19041.0\\shared"
"-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.19041.0\\winrt"
"-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.19041.0\\cppwinrt"
/EHsc /Tpsrc/clipper_ext/xtal_mgr.cpp /Fosrc/clipper_ext/xtal_mgr.obj

On Thu, Nov 17, 2022 at 9:31 AM Tristan Croll <tcroll@altoslabs.com> wrote:

comment:16 by Zach Pearson, 3 years ago

No problem, thanks for getting back to me. I can see that that core folder is still missing. I must not have pushed the change I made but
it's definitely there now so fingers crossed.

comment:17 by Zach Pearson, 3 years ago

Milestone: 1.5

comment:18 by Zach Pearson, 3 years ago

Resolution: fixed
Status: feedbackclosed

Tristan reported this morning that his bundle builds without his previous workaround.

Note: See TracTickets for help on using tickets.