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

Add compilation CI #5

Closed
wants to merge 2 commits into from
Closed

Add compilation CI #5

wants to merge 2 commits into from

Conversation

echus
Copy link

@echus echus commented Jun 9, 2023

See #3

@echus echus marked this pull request as draft June 9, 2023 02:28
@echus echus marked this pull request as ready for review June 9, 2023 04:42
@echus
Copy link
Author

echus commented Jun 14, 2023

@harshula looks like the build is failing atm:

==> [2023-06-14-02:08:34.336193] Error: ProcessError: ./MOM_compile.csh: No such file or directory

Any ideas?

@harshula
Copy link

Hi, MOM5's source code is not modified by me. MOM5's SPD contains:

        with working_dir(join_path(self.stage.source_path, "exp")):
            build = Executable("./MOM_compile.csh")

self.stage.source_path would have to be incorrect for ./MOM_compile.csh not to be found.

Information about working_dir: https://spack.readthedocs.io/en/latest/packaging_guide.html#working-dir

Is it possible to get a directory listing at the location where ./MOM_compile.csh was executed? e.g.

==> [2023-06-14-02:08:34.243730] Created stage in /tmp/root/spack-stage/spack-stage-mom5-3-build-ci-4wuq7nrywonbel7opuztsfnmzo32qcfh

@echus
Copy link
Author

echus commented Jun 22, 2023

@harshula looks like the build is failing atm:

==> [2023-06-14-02:08:34.336193] Error: ProcessError: ./MOM_compile.csh: No such file or directory

Any ideas?

Solved in ACCESS-NRI/build-ci@f88b50c by installing csh in the base Docker image.

@echus
Copy link
Author

echus commented Jun 22, 2023

@harshula looks like we're getting another build failure on Github Actions. Doesn't occur locally. See: https://github.com/ACCESS-NRI/MOM5/actions/runs/5262225975/jobs/9680881999

I think this is the relevant error:

  >> 701    /tmp/root/spack-stage/spack-stage-mom5-3-build-ci-4wuq7nrywonbel7op
            uztsfnmzo32qcfh/spack-src/src/mom5/ocean_core/ocean_model.F90(337):
             error #7002: Error in opening the compiled module file.  Check INC
            LUDE paths.   [VERSION_MOD]
     702    use version_mod,                  only: MOM_COMMIT_HASH
     703    ----^
  >> 704    /tmp/root/spack-stage/spack-stage-mom5-3-build-ci-4wuq7nrywonbel7op
            uztsfnmzo32qcfh/spack-src/src/mom5/ocean_core/ocean_model.F90(337):
             error #6580: Name in only-list does not exist or is not accessible
            .   [MOM_COMMIT_HASH]
     705    use version_mod,                  only: MOM_COMMIT_HASH
     706    ----------------------------------------^
  >> 707    /tmp/root/spack-stage/spack-stage-mom5-3-build-ci-4wuq7nrywonbel7op
            uztsfnmzo32qcfh/spack-src/src/mom5/ocean_core/ocean_model.F90(700):
             error #6404: This name does not have a type, and must have an expl
            icit type.   [MOM_COMMIT_HASH]
     708            write(stdoutunit,*) MOM_COMMIT_HASH
     709    ----------------------------^
     710    compilation aborted for /tmp/root/spack-stage/spack-stage-mom5-3-bu
            ild-ci-4wuq7nrywonbel7opuztsfnmzo32qcfh/spack-src/src/mom5/ocean_co
            re/ocean_model.F90 (code 1)
  >> 711    make: *** [Makefile:241: ocean_model.o] Error 1
     712    Make failed to create lib_ocean.a

@harshula
Copy link

Hi, Follow the same process as last time. See if the version_mod file exits. Also remember: #3 (comment) and Issue #4.

@echus echus marked this pull request as draft July 5, 2023 06:31
@echus
Copy link
Author

echus commented Jul 5, 2023

Closing this pull request to open a new one as Github is failing to register the latest commit to 3-build-ci.

@echus echus closed this Jul 5, 2023
@echus echus mentioned this pull request Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants