Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CAM no longer builds with intel-oneapi compilers #35

Closed
gdicker1 opened this issue Mar 8, 2024 · 6 comments
Closed

CAM no longer builds with intel-oneapi compilers #35

gdicker1 opened this issue Mar 8, 2024 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@gdicker1
Copy link
Contributor

gdicker1 commented Mar 8, 2024

@dazlich discovered that the update to include ESCOMP/CAM cam6_3_148 in EarthWorksOrg/CAM #14 broke builds with the Intel-OneAPI compilers

Example of error message:

ERROR: Command: '/glade/u/home/dazlich/ewv2.1/post_relv21/components/cam/bld/configure -s -fc_type intel-oneapi -dyn mpas -hgrid mpasa120 -cpl nuopc -usr_src /glade/derecho/scratch/dazlich/frametest.intel-oneapi/SourceMods/src.cam -spmd -nosmp -ocn mpaso -dyn mpas -nlev=32 -mpas_libdir /glade/derecho/scratch/dazlich/frametest.intel-oneapi/bld/atm/obj/mpas' failed with error 'ERROR: intel-oneapi is not a valid value for parameter fc_type: valid values are cray,pgi,intel,gnu,pathscale,ibm,nag,nvhpc,pgi-gpu,nvhpc-gpu' from dir '/glade/derecho/scratch/dazlich/frametest.intel-oneapi/Buildconf/camconf'

This seems to be due to edits to bld/configure (near line 1685) that were introduced in the commit 14b6813 in ESCOMP/CAM as part of the changes in cam6_3_147. I think this could be fixed by either reverting this change or replacing the logic with correct Perl substring matching.

@gdicker1 gdicker1 added the bug Something isn't working label Mar 8, 2024
@gdicker1 gdicker1 self-assigned this Mar 8, 2024
@gdicker1
Copy link
Contributor Author

gdicker1 commented Mar 8, 2024

Example fix from this StackOverflow comment:

if (index($str, $substr) != -1) {
    print "$str contains $substr\n";
} 

Instead of

if ($fc_type eq "oneapi") {$fc_type = 'intel'; }

@dazlich
Copy link
Contributor

dazlich commented Mar 8, 2024 via email

@gdicker1
Copy link
Contributor Author

gdicker1 commented Mar 8, 2024

Jim Edwards just had a look and concurs with the example fix.

@dazlich, nice! Do you know if he's opening an issue about it in ESCOMP/CAM, or should I?

@dazlich
Copy link
Contributor

dazlich commented Mar 8, 2024 via email

@gdicker1
Copy link
Contributor Author

gdicker1 commented Mar 8, 2024

Gotcha, thanks Don.

@gdicker1
Copy link
Contributor Author

Closed by #32

gdicker1 added a commit to gdicker1/EarthWorks that referenced this issue Aug 22, 2024
8f78adc Bump to 0.8.4
1ee334a Merge pull request EarthWorksOrg#58 from ESMCI/fix/hashid
e705f72 fix hash id and tag parsing
36b4b92 Bump to 0.8.3
654d233 Merge pull request EarthWorksOrg#57 from ESMCI/fix_test_output
0a4ea71 update the way tags are identified
6ee78d6 correct count of test fails
75a0337 Bump to 0.8.2
7136066 Merge pull request EarthWorksOrg#54 from ESMCI/fix/tag-not-found
a90de4f update to fetch tag first
a6e43fc Bump to 0.8.1
229ba03 Merge pull request EarthWorksOrg#53 from ESMCI/fix/ssh_access
e9cd2be better fix for issue with ssh access
c624b61 Bump to 0.8.0
6ccf617 Merge pull request EarthWorksOrg#51 from ESMCI/fix/recursive
53fce57 no need to make submodule objects persist
dae1c82 now working with issue EarthWorksOrg#50
ded91fd all tests passing, issue EarthWorksOrg#50 fixed
62a338b all pytests now passing
143abac tests a-d all pass
2b021fd add documentation to submodule.py
72211d2 add the new submodule class
82dd7a4 test_b_update now working
a7d8b4e all pytests passing
5322019 Bump to 0.7.9
1f07cb6 Merge pull request EarthWorksOrg#49 from ESMCI/fix/fxDONOTUSEurl
31ce71b use poetry update to update poetry.lock file
46043f0 add code of conduct (issue EarthWorksOrg#35)
0c3d683 dependabot changes and issue EarthWorksOrg#48

git-subtree-dir: .lib/git-fleximod
git-subtree-split: 8f78adcb0386cf496de377b9a92705e44f3f9a37
gdicker1 added a commit to gdicker1/EarthWorks that referenced this issue Aug 28, 2024
8f78adc Bump to 0.8.4
1ee334a Merge pull request EarthWorksOrg#58 from ESMCI/fix/hashid
e705f72 fix hash id and tag parsing
36b4b92 Bump to 0.8.3
654d233 Merge pull request EarthWorksOrg#57 from ESMCI/fix_test_output
0a4ea71 update the way tags are identified
6ee78d6 correct count of test fails
75a0337 Bump to 0.8.2
7136066 Merge pull request EarthWorksOrg#54 from ESMCI/fix/tag-not-found
a90de4f update to fetch tag first
a6e43fc Bump to 0.8.1
229ba03 Merge pull request EarthWorksOrg#53 from ESMCI/fix/ssh_access
e9cd2be better fix for issue with ssh access
c624b61 Bump to 0.8.0
6ccf617 Merge pull request EarthWorksOrg#51 from ESMCI/fix/recursive
53fce57 no need to make submodule objects persist
dae1c82 now working with issue EarthWorksOrg#50
ded91fd all tests passing, issue EarthWorksOrg#50 fixed
62a338b all pytests now passing
143abac tests a-d all pass
2b021fd add documentation to submodule.py
72211d2 add the new submodule class
82dd7a4 test_b_update now working
a7d8b4e all pytests passing
5322019 Bump to 0.7.9
1f07cb6 Merge pull request EarthWorksOrg#49 from ESMCI/fix/fxDONOTUSEurl
31ce71b use poetry update to update poetry.lock file
46043f0 add code of conduct (issue EarthWorksOrg#35)
0c3d683 dependabot changes and issue EarthWorksOrg#48

git-subtree-dir: .lib/git-fleximod
git-subtree-split: 8f78adcb0386cf496de377b9a92705e44f3f9a37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants