Lori A. Burns

btw, don't exert much effort on anything only used by optking as hopefully that gets deleted by end of year.

> I'm having second thoughts about this PR now. Should we deprecate the old constructors first? This PR will require changes in Forte as well. options as I see them....

> I vote deprecate, just in case we change our mind on this decision. fine by me

>> I do think we should try to get the changes into one psi4 release cycle. > Regarding the deprecation message/schedule: do you mean that they should be deprecated now...

Installing `share/` files into `/stage` is normal from https://github.com/psi4/psi4/blob/master/psi4/CMakeLists.txt#L354-L358 . If the `%%DATADIR%%` is literal, could you be not getting `CMAKE_INSTALL_DATADIR` set correctly from `GNUInstallDirs`?

> Your internal stage directory isn't copied properly into the real stage directory for some reason. That's strange -- it's a pretty straightforward copy, https://github.com/psi4/psi4/blob/master/CMakeLists.txt#L315-L318 from internal /stage/ to final...

> What version of cmake do you use? Personally, I use a near-latest. The Psi4 minimum is 3.15. We could go higher, but at least one of the dependencies doesn't...

I've just tested the following: * cloned to `/external_disk/gits/psi4` * from there, configured to `/tmp` via `cmake -S. ... -DCMAKE_INSTALL_PREFIX=/home/auser/installs/psi4 -B/tmp/compile-psi4` * build in `/tmp/compile-psi4` via `cmake --build .` *...

> I'm not comfortable merging this in while the dfocc cleanup is ongoing. Your thoughts? Somehow I had the impression this was in occ, so it was fine. I agree...

`undefined symbol: _ZNK3psi12TwoBodyAOInt22shell_pair_max_densityEiii` in case you hadn't clicked through to it yet