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

Minor issue: Status of sub-externals shown incorrectly during checkout #87

Open
billsacks opened this issue Apr 4, 2018 · 1 comment
Labels

Comments

@billsacks
Copy link
Member

Summary of Issue:

When running checkout_externals -v, the status of sub-externals is shown incorrectly. I have determined that this problem came in with 1ae8c84 (Merge bugfix branch for stale subexternals into master).

The end result is correct, so this isn't a big deal. This just makes checkout_externals -v less useful than it otherwise would be.

Expected behavior and actual behavior:

Expected behavior (and the behavior prior to the above commit):

[roo2:~/temporary/cism-wrapper-temp2]$ ./manage_externals/checkout_externals -v
Processing externals description file : Externals.cfg
Processing externals description file : Externals_CISM.cfg
Processing externals description file : Externals_CLM.cfg
Checking status of externals: cism, source_cism, mosart, ctsm, fates, ptclm, cime,
Checking out externals:


    ./components/mosart
        clean sandbox, on mosart1_0_30

    ./components/clm
        clean sandbox, on clm4_5_18_r274

    ./cime
        clean sandbox, on cime5.4.0-alpha.24

Checking out externals:

    ./glimmer-cism
        clean sandbox, on cism/branch_tags/longer_var_list_tags/longer_var_list_n01_cism2_1_develop_n02

Checking out externals:

    ./components/clm/src/fates
        clean sandbox, on fates_s1.4.1_a3.0.0_rev2

    ./components/clm/tools/PTCLM
        clean sandbox, on PTCLM2_171216c

Current, incorrect behavior:

[roo2:~/temporary/cism-wrapper-temp2]$ ./manage_externals/checkout_externals -v
Processing externals description file : Externals.cfg
Processing externals description file : Externals_CISM.cfg
Processing externals description file : Externals_CLM.cfg
Checking status of externals: cism, source_cism, mosart, ctsm, fates, ptclm, cime,
Checking out externals:


    ./components/mosart
        clean sandbox, on mosart1_0_30

    ./components/clm
        clean sandbox, on clm4_5_18_r274

    ./cime
        clean sandbox, on cime5.4.0-alpha.24

Processing externals description file : Externals_CISM.cfg
Checking out externals:

---
        -,  -->
    svn switch https://svn-ccsm-models.cgd.ucar.edu/cism/branch_tags/longer_var_list_tags/longer_var_list_n01_cism2_1_develop_n02

Processing externals description file : Externals_CLM.cfg
Checking out externals:

---
        -,  -->
    git checkout fates_s1.4.1_a3.0.0_rev2

---
        -,  -->
    git checkout PTCLM2_171216c

Steps to reproduce the problem (should include model description file(s) or link to publi c repository):

git clone [email protected]:ESCOMP/cism-wrapper.git
cd cism-wrapper
git checkout cism2_1_50
rm -rf manage_externals
git clone [email protected]:NCAR/manage_externals.git
./manage_externals/checkout_externals
./manage_externals/checkout_externals -v

What is the changeset ID of the code, and the machine you are using:

See above for notes on changeset ID

have you modified the code? If so, it must be committed and available for testing:

no

Screen output or log file showing the error message and context:

see above

@billsacks billsacks added the bug label Apr 4, 2018
@billsacks
Copy link
Member Author

@bandre-ucar Here's another which, if it would be fairly quick for you, it would be great to get your help on. But I see this as low priority, so only worth doing if it would be quick for you to fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant