Skip to content

contributing: Add notes on sandbox and addon builds to AGENTS.md - #7609

Merged
echoix merged 2 commits into
OSGeo:mainfrom
wenzeslaus:add-sandbox-and-addon-to-agents
Jul 1, 2026
Merged

echoix merged 2 commits into
OSGeo:mainfrom
wenzeslaus:add-sandbox-and-addon-to-agents

Conversation

@wenzeslaus

Copy link
Copy Markdown
Member

Using a use case of a standalone addon tool repo (specifically, r.pops.spread is published in addons, but the developed in a standalone repo), this expands instructions in AGENTS.md on how to compile and run with CMake in a sandbox (developed with Claude Code sandbox) and how to compile an addon tool. While grass-addon and cookie cutter repo instructions would be helpful as well, the addons instruction part covers the case when you use the source code of GRASS, but don't deal with the grass-addon repo.

I'm also slighltly changing how the grass.gunittest is presented, focusing on a single file, rather than a directory run. This simplifies the project-related instructions, replacing the currently confusing version.

The addons piece is something which may possible confuse an agent (if you don't work with addons or g.extension), but I would just wait to see how it behaves.

@nilason or @HuidaeCho if you can check if the CMake prefix instructions are good, that would be great. This is a result based on how it behaved and what Claude came up with as an evaluation, but it is intended? I'm still not 100% comfortable with CMake, so it is hard for me to be sure this right and does have some red flags for me, mainly the use of -D vs --prefix.

Using a use case of a standalone addon tool repo (specifically, r.pops.spread is published in addons, but the developed in a standalone repo), this expands instructions in AGENTS.md on how to compile and run with CMake in a sandbox (developed with Claude Code sandbox) and how to compile an addon tool. While grass-addon and cookie cutter repo instructions would be helpful as well, the addons instruction part covers the case when you use the source code of GRASS, but don't deal with the grass-addon repo.

I'm also slighltly changing how the grass.gunittest is presented, focusing on a single file, rather than a directory run. This simplifies the project-related instructions, replacing the currently confusing version.
@github-actions github-actions Bot added docs markdown Related to markdown, markdown files labels Jun 26, 2026
@echoix
echoix merged commit 08c3e8a into OSGeo:main Jul 1, 2026
22 checks passed
@github-actions github-actions Bot added this to the 8.6.0 milestone Jul 1, 2026
@nilason

nilason commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Some of the code hard codes CMAKE_INSTALL_PREFIX e.g. during configuration of resource_paths.py.

cmake --install build --prefix ./tmp_install_dir is on the other hand, used for e.g. staging installation. This may be used in combination with setting CMAKE_INSTALL_PREFIX, but redirected to --prefix (essentially a copy operation), with the final package usable only if unpacked/copied to CMAKE_INSTALL_PREFIX.

ninsbl pushed a commit to ninsbl/grass that referenced this pull request Jul 7, 2026
…eo#7609)

* contributing: Add notes on sandbox and addon builds to AGENTS.md

Using a use case of a standalone addon tool repo (specifically, r.pops.spread is published in addons, but the developed in a standalone repo), this expands instructions in AGENTS.md on how to compile and run with CMake in a sandbox (developed with Claude Code sandbox) and how to compile an addon tool. While grass-addon and cookie cutter repo instructions would be helpful as well, the addons instruction part covers the case when you use the source code of GRASS, but don't deal with the grass-addon repo.

I'm also slighltly changing how the grass.gunittest is presented, focusing on a single file, rather than a directory run. This simplifies the project-related instructions, replacing the currently confusing version.

* Wrap lines correctly
neteler pushed a commit that referenced this pull request Jul 14, 2026
* contributing: Add notes on sandbox and addon builds to AGENTS.md

Using a use case of a standalone addon tool repo (specifically, r.pops.spread is published in addons, but the developed in a standalone repo), this expands instructions in AGENTS.md on how to compile and run with CMake in a sandbox (developed with Claude Code sandbox) and how to compile an addon tool. While grass-addon and cookie cutter repo instructions would be helpful as well, the addons instruction part covers the case when you use the source code of GRASS, but don't deal with the grass-addon repo.

I'm also slighltly changing how the grass.gunittest is presented, focusing on a single file, rather than a directory run. This simplifies the project-related instructions, replacing the currently confusing version.

* Wrap lines correctly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs markdown Related to markdown, markdown files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants