Skip to content

Commit a0c839a

Browse files
authored
Merge pull request #93 from nmarticorena/feat-docs-contributing
Add step regarding dependencies.yaml
2 parents b0755a3 + 24515ba commit a0c839a

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

docs/Contributing.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,9 @@ Sometimes, it may be required to patch the packages. An example of how to do so
3030
6. Create a patch file with `git diff > changes.patch`
3131
7. Check that the patch contains the intended changes
3232
8. Move the file into the `patches` directory of this repository, renaming it according to the naming convention. If the changes are portable across all supported operating system, the file should be called `<package name>.patch`; otherwise, `<package name>.<win/linux/osx>.patch`
33-
9. Rerun `pixi run build` to ensure that the patching succeeds and the package builds without errors
34-
10. Commit the new file, push to your fork and create a PR
33+
9. In some cases the `recipe.yaml` generated needs also a patch (eg. add a conda dependency), in those scenarios a `add_host` key needs to be added to `patch/dependencies.yaml` [example](https://github.com/RoboStack/ros-noetic/blob/bd1d2f44fcbfb02a31c464e51a761fb0fdc32ec0/patch/dependencies.yaml#L15-L17)
34+
10. Rerun `pixi run build` to ensure that the patching succeeds and the package builds without errors
35+
11. Commit the new file, push to your fork and create a PR
3536

3637
## Extending an existing patch file
3738

0 commit comments

Comments
 (0)