Skip to content

Conversation

@nilason
Copy link
Contributor

@nilason nilason commented Feb 9, 2026

Fix FOLDER structure for IDE's that support it (e.g. Xcode and MSVS).

This is a low-risk, coarse clean up of the folder structure, generating a tree like:

image

There is plenty of work to do on this end, but that will be for future pull requests.

Would be good to have this for 8.5.

@nilason nilason added this to the 8.5.0 milestone Feb 9, 2026
@nilason nilason added the backport to 8.5 PR needs to be backported to release branch 8.5 label Feb 9, 2026
petrasovaa
petrasovaa previously approved these changes Feb 9, 2026
echoix
echoix previously approved these changes Feb 9, 2026
@wenzeslaus
Copy link
Member

This split Python/Tools (scripts) vs Tools/Temporal (temporal) does not make sense. The situation in the source code is not ideal (temporal scripts versus all the other scripts), but this adds another layer of unclear.

A quick fix perhaps would be Python/Other Tools (scripts) and Python/Temporal Tools (temporal).

@nilason
Copy link
Contributor Author

nilason commented Feb 9, 2026

This split Python/Tools (scripts) vs Tools/Temporal (temporal) does not make sense. The situation in the source code is not ideal (temporal scripts versus all the other scripts), but this adds another layer of unclear.

A quick fix perhaps would be Python/Other Tools (scripts) and Python/Temporal Tools (temporal).

Python tools are divided into subfolders (Python/Tools/Raster etc), just as compiled tools. We may eventually merge the two (just one Tools folder), but currently the Python files does not show in IDE (just as header files are missing), so I’d say it is better to separate them for now.
How about putting Python tools in one and the same Tools folder but still separate sub-folders: Tools/Raster (Scripts) or Tools/Raster (Python)?

@github-actions github-actions bot added GUI wxGUI related vector Related to vector data processing raster Related to raster data processing temporal Related to temporal data processing translation Message translation related database Related to database management libraries module docs display imagery raster3d CMake labels Feb 10, 2026
@nilason
Copy link
Contributor Author

nilason commented Feb 10, 2026

Could this be a better (interim) solution?:

image

As I mentioned above, there's a lot to do with regards to IDE support, but this is a tremendous improvement to the current state.

@nilason nilason dismissed stale reviews from petrasovaa and echoix via c52ab79 February 10, 2026 10:03
Copy link
Member

@wenzeslaus wenzeslaus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All under Tools makes sense to me. I would go with Raster Script rather than Raster (Script), but then I don't know what to do with the Binary (not that I would be so sure about "Binary (Scripts)").

@nilason
Copy link
Contributor Author

nilason commented Feb 10, 2026

All under Tools makes sense to me. I would go with Raster Script rather than Raster (Script), but then I don't know what to do with the Binary (not that I would be so sure about "Binary (Scripts)").

Well the content of "Binary" and "Binary (Script)" is all the binary targets, which are not Tools:

image

It will take some work to sort that out, together with "the rest". Let us not dwell on details in an iteration of a WIP that does not affect GRASS as such.

@nilason nilason enabled auto-merge (squash) February 12, 2026 07:47
@nilason nilason merged commit 48172e2 into OSGeo:main Feb 12, 2026
27 checks passed
@nilason nilason removed the backport to 8.5 PR needs to be backported to release branch 8.5 label Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CMake database Related to database management display docs GUI wxGUI related imagery libraries module raster Related to raster data processing raster3d temporal Related to temporal data processing translation Message translation related vector Related to vector data processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants