Change CPP USE_SNICARHC to NO_SNICARHC

[apcraig]

PR checklist

• [X] Short (1 sentence) summary of your PR: Change CPP USE_SNICARHC to NO_SNICARHC
• [X] Developer(s): apcraig
• [X] Suggest PR reviewers from list in the column to the right.
• [X] Please copy the PR test results link or provide a summary of testing completed below. full test suite of Icepack (and CICE) run on cheyenne, all tests are bit-for-bit.
• How much do the PR code changes differ from the unmodified code?
  • [X] bit for bit
  • [ ] different at roundoff level
  • [ ] more substantial
• Does this PR create or have dependencies on CICE or any other models?
  • [X] Yes, USE_SNICARNC is going away, default will be to build the 5 band snicar table by default
  • [ ] No
• Does this PR add any new test cases?
  • [ ] Yes
  • [X] No
• Is the documentation being updated? ("Documentation" includes information on the wiki or in the .rst files from doc/source/, which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/.)
  • [X] Yes
  • [ ] No, does the documentation need to be updated at a later time?
    • [ ] Yes
    • [ ] No
• [X] Please provide any additional information or relevant details below:

Will PR equivalent update to CICE once this is merged. CICE will require updated Icepack.

[apcraig]

Sorry about the title, fixed that.

On the E3SM side, you will NOT have to do anything immediately. When you update Icepack, USE_SNICARHC will not play a role anymore and the snicar tables will be built automatically. So it should work just like it does not. At some point, you should remove the USE_SNICARHC from the E3SM build just to clean that up. You will only need a CPP in Icepack if you DON'T want to build the snicar 5 band tables.