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

Support repos mounted with trailing @ #58

Open
tclune opened this issue Feb 11, 2020 · 2 comments
Open

Support repos mounted with trailing @ #58

tclune opened this issue Feb 11, 2020 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@tclune
Copy link
Collaborator

tclune commented Feb 11, 2020

Currently our convention is to mount repos with a leading @ character. There is some pressure to change this to a trailing @. We should generalize the few points that currently strip @ to strip off leading and trailing @. Then, if someone really prefers the alternative convention, they can always modify their own mepo config file.

@tclune tclune added the enhancement New feature or request label Feb 11, 2020
@mathomp4
Copy link
Member

This is probably not too hard. Something like:

string(REPLACE '@' '' foo ${foo})

could do it (assuming no future components will have an @-symbol in the middle).

The problem is changing CMakeLists.txt files throughout the repositories in every add_subdirectory() and esma_add_library() call as well, e.g.:

esma_add_subdirectories (
  @MAPL
  @GMAO_Shared
  @NCEP_Shared
  @GSW
  )

# Special case - FMS is built twice with two
# different precisions.
add_subdirectory (@FMS fms_r4)
add_subdirectory (@FMS fms_r8)

@tclune
Copy link
Collaborator Author

tclune commented Feb 12, 2020

I think the idea in the short term will be to modify the esma_add_subdirectories() implementation. It should strip any @ chars from the argument "foo" and then check for directories named "foo" "@foo" and "foo@" and add the first of those that it finds.

We can then at our leisure remove the @'s in the various components.

Note: Ricardo wants to eliminate them entirely, though mostly because tab complete is not working for him. He may not be opposed to trailing ones. I really don't care so long as there is a consensus and we don't go back and forth. Existing was a proposed solution. Pushback is the change. And then we hold the line.

@tclune tclune changed the title Support repos mounted with trailing @ Support repos mounted with trailing & Apr 9, 2020
@tclune tclune changed the title Support repos mounted with trailing & Support repos mounted with trailing @ Apr 9, 2020
mathomp4 added a commit that referenced this issue Apr 15, 2020
…nt-placement-of-@

Fixes #78 support for varying @ placement
mathomp4 added a commit that referenced this issue Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants