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

move get_*.sh scripts to top-level subdirectory "wmo_scripts" #210

Closed
edwardhartnett opened this issue Jul 22, 2024 · 4 comments · Fixed by #236
Closed

move get_*.sh scripts to top-level subdirectory "wmo_scripts" #210

edwardhartnett opened this issue Jul 22, 2024 · 4 comments · Fixed by #236
Assignees
Labels
build Relates to the build system

Comments

@edwardhartnett
Copy link
Contributor

We can move these scripts out of our source code directory.

The scripts need to be modified to work from the new location.

@webisu
Copy link
Collaborator

webisu commented Jul 22, 2024 via email

@edwardhartnett
Copy link
Contributor Author

I'm afraid that method of using plugins will no longer work.

If you add a file to a CMake built directory and run make, it will make no difference. The file will be ignored if it is not in the CMakeLists.txt file. Nor will the user be able to turn off options by deleting code files. All build options have to be specifically handled in cmake.

Many programs and libraries have plugins, including netCDF and HDF5. The way they do it is to require .so files which they dynamically link to and then they call functions of predefined API. So to use a plugin you do not have to (re)compile the whole library, just the plugin.

We can transition to this if you want to keep your plugin architecture. It would not be hard. Then the plugins can also be in a separate directory, or just exist in user-space, if it's a user-built plugin.

@webisu
Copy link
Collaborator

webisu commented Jul 22, 2024 via email

@edwardhartnett
Copy link
Contributor Author

Let's discuss that under another issue, which I will start.

The point of this issue is to move the get_*.sh scripts to their own directory away from the source code. THe scripts will have to be modified to find the files from the new location, but this should be easy.

The options scripts can be handled separately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Relates to the build system
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants