Skip to content

Commit f3fc9ea

Browse files
committed
CONTRIBUTING_GO.md: fix modules to use new monorepo path
Update the commands and examples to use the new go.podman.io paths. Signed-off-by: Paul Holzinger <pholzing@redhat.com>
1 parent 75063d4 commit f3fc9ea

File tree

1 file changed

+21
-13
lines changed

1 file changed

+21
-13
lines changed

CONTRIBUTING_GO.md

Lines changed: 21 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -29,22 +29,29 @@ The bot automatically opens new PRs with updates that should be merged by mainta
2929

3030
However sometimes, especially during development, it can be the case that you like to update a dependency.
3131

32-
To do so you can use the `go get` command, for example to update containers/storage to the a specific version use:
32+
To do so you can use the `go get` command, for example to update the storage library to the a specific version use:
3333
```
34-
$ go get github.com/containers/storage@v1.55.1
34+
$ go get go.podman.io/storage@v1.60.0
3535
```
3636

3737
Or to update it to the latest commit from main use:
3838
```
39-
$ go get github.com/containers/storage@main
39+
$ go get go.podman.io/storage@main
4040
```
4141

42-
This command will update the go.mod/go.sum files, because we use [go's vendor mechanism](https://go.dev/ref/mod#vendoring)
43-
you must also update the files in the vendor dir. To do so use
42+
This command will update the go.mod/go.sum files, in some repos we use [go's vendor mechanism](https://go.dev/ref/mod#vendoring)
43+
so there you must also update the files in the vendor dir. To do so use:
4444
```
4545
$ make vendor
4646
```
4747

48+
If you are working in the [container-libs](https://github.com/containers/container-libs) monorepo use:
49+
```
50+
make tidy
51+
```
52+
This command syncs the dependency versions across all modules in the repo.
53+
54+
4855
Then commit the changes and open a PR. If you want to add other changes it is recommended to keep the
4956
dependency updates in their own commit as this makes reviewing them much easier.
5057

@@ -55,16 +62,16 @@ dependencies to ensure all our tools use the same properly tested library versio
5562

5663
Sometimes it is helpful (or a maintainer asks for it) to test your library changes in the final binary, e.g. podman.
5764

58-
Assume we like to test a containers/common PR in Podman so that we can have the full CI tests run there.
59-
First you need to push your containers/common changes to your github fork (if not already done).
65+
Assume we like to test a container-libs/common PR in Podman so that we can have the full CI tests run there.
66+
First you need to push your container-libs/common changes to your github fork (if not already done).
6067
Now open the podman repository, create a new branch there and then use.
6168
```
62-
$ go mod edit -replace github.com/containers/common=github.com/<account name>/<fork name>@<branch name>
69+
$ go mod edit -replace go.podman.io/common=github.com/<account name>/<fork name>/common@<branch name>
6370
```
64-
Replace the variable with the correct values, in my case it the reference might be `github.com/Luap99/common@netns-dir`, where
71+
Replace the variable with the correct values, in my case it the reference might be `github.com/Luap99/container-libs/common@myfeature`, where
6572
- account name == `Luap99`
66-
- fork name == `common`
67-
- branch name that I like to test == `netns-dir`
73+
- fork name == `container-libs`
74+
- branch name that I like to test == `myfeature`
6875

6976
Then just run the vendor command again.
7077
```
@@ -86,12 +93,13 @@ If you performed a the git bisect and the resulting commit is one that updated a
8693
the problem is in that library instead. In such cases it may be needed to find the bad commit from this
8794
repository instead. Thankfully this is not much more difficult than the normal bisect usage.
8895

89-
Clone the library repository locally (for this example assume we it is github.com/containers/storage),
96+
Clone the library repository locally (for this example we assume it is github.com/containers/container-libs)
97+
which contains the storage library as module in a subdirectory,
9098
I assume it is in a directory next to the podman repo.
9199

92100
Then in podman run (where you replace the path to the storage repo with your actual one)
93101
```
94-
$ go mod edit -replace github.com/containers/storage=/path/to/storage
102+
$ go mod edit -replace go.podman.io/storage=/path/to/container-libs/storage
95103
$ make vendor
96104
```
97105

0 commit comments

Comments
 (0)