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

cmd/internal/xcoff: move to x/debug/xcoff #29038

Open
Helflym opened this issue Nov 30, 2018 · 4 comments
Open

cmd/internal/xcoff: move to x/debug/xcoff #29038

Helflym opened this issue Nov 30, 2018 · 4 comments
Labels
NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Milestone
Backlog

Comments

@Helflym
Copy link
Contributor

Helflym commented Nov 30, 2018

Hi,

Since CL 149957, the tests inside gccgoimporter as long as the gccgo command is available, which is the case on most of AIX machines. However, it's only aware of ELF gccgo files, it doesn't recognize XCOFF files. To add the features, cmd/internal/xcoff must be moved to x/debug/xcoff as it was suggested by issue 28037.
Is that possible ?

Morever, big archive format is only available on AIX, as far as I know. Therefore, is that okay if I implement functions to read them in x/debug/xcoff ? Or I'd rather do a x/debug/ar as it was suggested here:
https://github.com/golang/go/blob/master/src/cmd/internal/buildid/buildid.go#L121

/cc @ianlancetaylor @bradfitz
@ianlancetaylor ianlancetaylor added the NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. label Nov 30, 2018
@ianlancetaylor ianlancetaylor added this to the Go1.12 milestone Nov 30, 2018
@ianlancetaylor
Copy link
Contributor

CC @rsc

I don't think we've really decided what is happening with x/debug. Or if we did, I've forgotten.

Note that we could keep postponing that and address this issue by moving cmd/internal/xcoff to internal/xcoff.

@Helflym
Copy link
Contributor Author

Helflym commented Nov 30, 2018

I haven't thought about internal/xcoff. I'd rather have x/debug/xcoff as it could be used by anyone as debug/elf or debug/macho currently are. But it's as you wish.

@bradfitz
Copy link
Contributor

bradfitz commented Nov 30, 2018 •
edited

I want to see the aix/ppc64 column all black and "ok" on https://build.golang.org/ before we start doing nice-to-have changes and entering the cherry-picking/branch-management world.

@bradfitz bradfitz modified the milestones: Go1.12, Go1.13 Nov 30, 2018
@bradfitz
Copy link
Contributor

Moving to Go 1.13 for now. If AIX is in good shape soon then we could probably do this during the freeze, but it's not important.

@andybons andybons modified the milestones: Go1.13, Go1.14 Jul 8, 2019
@rsc rsc modified the milestones: Go1.14, Backlog Oct 9, 2019
@bcmills bcmills mentioned this issue Nov 8, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
5 participants
@bradfitz @rsc @andybons @ianlancetaylor @Helflym