update bacio, crtm module vars in common/modules_*.yaml

[AlexanderRichert-NOAA]

Description

This PR updates the bacio vars to use $BACIO_LIB and $BACIO_INC instead of being suffixed with 4/8, reflecting the removal of the type-differentiated library versions following bacio 2.6.0. It also changes $CRTM_LIB to point to the shared library.

Dependencies

none

Issues addressed

Fixes #1894

Applications affected

most applications are affected

Systems affected

all

Testing

• CI: Note whether the automatic tests (GitHub actions tests that run automatically for every commit) pass or not
  • GitHub actions CI tests pass
  • GitHub actions CI tests do not pass (provide explanation)
  • GitHub actions CI tests skipped (provide explanation if necessary)
• Additional testing: Add information on any additional tests conducted
  • test on Acorn: install crtm 3.1.3 and bacio 2.6.0 with module files and verify that BACIO_LIB/BACIO_INC/CRTM_LIB point to existing paths.

Checklist

• This PR addresses one issue/problem/enhancement or has a very good reason for not doing so.
• These changes have been tested on the affected systems and applications.
• All dependency PRs/issues have been resolved and this PR can be merged.
• All necessary updates to the documentation (spack-stack wiki) will be made when this PR is merged

[AlexanderRichert-NOAA]

@RussTreadon-NOAA @DavidHuber-NOAA please let me know if the paths here make sense based on what you need. I'll also test on at least one system to make sure the paths line up.
Ideally downstream applications would find and configure these dependencies via CMake/CMAKE_PREFIX_PATH. Also we may wish to consider incorporating these into the various package.py's to avoid system-specific differences in lib vs. lib64 etc.

[DavidHuber-NOAA]

These look reasonable to me. I'm happy to see fewer _4 and _8's in the paths.

[climbfuji]

I'll merge this after #1924