Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

proposal: x/tools: tag and delete refactor/rename, refactor/importgraph, go/buildutil, cmd/gomvpkg #69538

Open
adonovan opened this issue Sep 19, 2024 · 2 comments
Open

proposal: x/tools: tag and delete refactor/rename, refactor/importgraph, go/buildutil, cmd/gomvpkg #69538

adonovan opened this issue Sep 19, 2024 · 2 comments
Labels
Proposal Tools This label describes issues relating to any tools in the x/tools repository.
Milestone
Proposal

Comments

@adonovan
Copy link
Member

adonovan commented Sep 19, 2024 •
edited
Loading

While implementing this proposal:

I immediately realized that most of the logic is not in the cmd package but in the refactor/rename package, which has never worked with Go modules. Similarly, refactor/importgraph and go/buildutil have the same limitation. And cmd/gomvpkg is in a similar state to the (now deleted) cmd/gorename.

I propose to tag and delete all of them following a similar process.

(FWIW, the only seemingly valuable part of go/buildutil is TagsFlag, which doesn't actually work in conjunction with go/packages; one must use the syntax gopackages -buildflag=-tags=... patterns....)
@gopherbot gopherbot added the Tools This label describes issues relating to any tools in the x/tools repository. label Sep 19, 2024
@gopherbot gopherbot added this to the Unreleased milestone Sep 19, 2024
@ianlancetaylor ianlancetaylor changed the title x/tools/refactor/{rename,importgraph}: tag and delete proposal: x/tools/refactor/{rename,importgraph}: tag and delete Sep 19, 2024
@ianlancetaylor ianlancetaylor modified the milestones: Unreleased, Proposal Sep 19, 2024
@adonovan adonovan changed the title proposal: x/tools/refactor/{rename,importgraph}: tag and delete proposal: x/tools: tag and delete refactor/rename, refactor/importgraph, go/buildutil, cmd/gomvpkg Sep 22, 2024
@gabyhelp
Copy link

Related Issues and Documentation

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/614895 mentions this issue: cmd/{callgraph,ssadump}, gopackages: make -tags flags work

gopherbot pushed a commit to golang/tools that referenced this issue Sep 25, 2024
@adonovan
cmd/{callgraph,ssadump}, gopackages: make -tags flags work
34638fc 
Previously, the -tags flag used the obsolete buildutil.TagsFlag,
which only affects the behavior of the obsolete go/loader.
This change causes the -tags flag to affect go/packages
in these three tools, as it should.

Updates golang/go#69556
Updates golang/go#69538

Change-Id: Ie45c51c7fe04863dba00bc2f81b0a78f1c9bd0e3
Reviewed-on: https://go-review.googlesource.com/c/tools/+/614895
TryBot-Bypass: Alan Donovan <[email protected]>
Reviewed-by: Michael Matloob <[email protected]>
Auto-Submit: Alan Donovan <[email protected]>
Commit-Queue: Alan Donovan <[email protected]>
@adonovan adonovan mentioned this issue Sep 25, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Proposal Tools This label describes issues relating to any tools in the x/tools repository.
Projects
Proposals
Status: Incoming
Milestone
Proposal
Development

No branches or pull requests
4 participants
@ianlancetaylor @adonovan @gopherbot @gabyhelp