Skip to content

Commit 2c14176

Browse files
authored
Revert "[chore] Use crosslink tidylist in make gotidy (#37142)" (#37173)
Reverts #37142 as something with this change is not working as expected. Reverting for now to unblock PRs. See #37170 and #37172
1 parent bc20ac9 commit 2c14176

File tree

6 files changed

+7
-332
lines changed

6 files changed

+7
-332
lines changed

.github/workflows/build-and-test.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -221,10 +221,6 @@ jobs:
221221
run: |
222222
make crosslink
223223
git diff --exit-code || (echo 'Replace statements are out of date, please run "make crosslink" and commit the changes in this PR.' && exit 1)
224-
- name: tidylist
225-
run: |
226-
make tidylist
227-
git diff --exit-code || (echo 'Tidylist is out of date, please run "make tidylist" and commit the changes in this PR.' && exit 1)
228224
- name: Check for go mod dependency changes
229225
run: |
230226
make gotidy

Makefile

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -115,22 +115,9 @@ stability-tests: otelcontribcol
115115
gogci:
116116
$(MAKE) $(FOR_GROUP_TARGET) TARGET="gci"
117117

118-
# `internal/tidylist/tidylist.txt` lists modules in topological order, to ensure `go mod tidy` converges.
119-
# `make tidylist` will update this list.
120-
# We exclude otelcontribcol and oteltestbedcol as those modules are not gitted and may not be present.
121-
.PHONY: tidylist
122-
tidylist: $(CROSSLINK)
123-
cd internal/tidylist && \
124-
$(CROSSLINK) tidylist --validate --allow-circular allow-circular.txt tidylist.txt && \
125-
sed -i.bak -E '/cmd\/otel(contrib|testbed)col/d' tidylist.txt && \
126-
rm tidylist.txt.bak
127-
128118
.PHONY: gotidy
129119
gotidy:
130-
@for mod in $$(cat internal/tidylist/tidylist.txt); do \
131-
echo "Tidying $$mod"; \
132-
(cd $$mod && rm -rf go.sum && $(GOCMD) mod tidy -compat=1.22.0) || exit $?; \
133-
done
120+
$(MAKE) $(FOR_GROUP_TARGET) TARGET="tidy"
134121

135122
.PHONY: remove-toolchain
136123
remove-toolchain:

internal/tidylist/allow-circular.txt

Lines changed: 0 additions & 12 deletions
This file was deleted.

internal/tidylist/tidylist.txt

Lines changed: 0 additions & 296 deletions
This file was deleted.

internal/tools/go.mod

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ require (
1313
github.com/jstemmer/go-junit-report v1.0.0
1414
go.opentelemetry.io/build-tools/checkfile v0.15.0
1515
go.opentelemetry.io/build-tools/chloggen v0.15.0
16-
go.opentelemetry.io/build-tools/crosslink v0.16.0
16+
go.opentelemetry.io/build-tools/crosslink v0.15.0
1717
go.opentelemetry.io/build-tools/issuegenerator v0.15.0
1818
go.opentelemetry.io/build-tools/multimod v0.15.0
1919
go.opentelemetry.io/collector/cmd/builder v0.117.0
@@ -233,7 +233,7 @@ require (
233233
gitlab.com/bosi/decorder v0.4.2 // indirect
234234
go-simpler.org/musttag v0.13.0 // indirect
235235
go-simpler.org/sloglint v0.7.2 // indirect
236-
go.opentelemetry.io/build-tools v0.16.0 // indirect
236+
go.opentelemetry.io/build-tools v0.15.0 // indirect
237237
go.opentelemetry.io/collector/component v0.117.0 // indirect
238238
go.opentelemetry.io/collector/config/configtelemetry v0.117.0 // indirect
239239
go.opentelemetry.io/collector/confmap v1.23.0 // indirect

internal/tools/go.sum

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)