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.

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

cmd/go: ar default option changed on Archlinux core/binutils 2.25.1-3 #12310

Closed
LiuPai opened this issue Aug 25, 2015 · 2 comments
Closed

cmd/go: ar default option changed on Archlinux core/binutils 2.25.1-3 #12310

LiuPai opened this issue Aug 25, 2015 · 2 comments
Milestone

Comments

@LiuPai
Copy link

LiuPai commented Aug 25, 2015

When compile always show warning
ar: u' modifier ignored since D' is the default (see U')`

relate file:
go/src/cmd/go/build.go

func (gccgoToolchain) pack(b *builder, p *Package, objDir, afile string, ofiles []string) error {
    var absOfiles []string
    for _, f := range ofiles {
        absOfiles = append(absOfiles, mkAbs(objDir, f))
    }
    return b.run(p.Dir, p.ImportPath, nil, "ar", "cru", mkAbs(objDir, afile), absOfiles)
}
@ianlancetaylor ianlancetaylor changed the title ar default option changed on Archlinux core/binutils 2.25.1-3 cmd/go: ar default option changed on Archlinux core/binutils 2.25.1-3 Aug 25, 2015
@ianlancetaylor ianlancetaylor added this to the Go1.6 milestone Aug 25, 2015
@ianlancetaylor ianlancetaylor self-assigned this Aug 25, 2015
@rsc
Copy link
Contributor

rsc commented Nov 5, 2015

Seems like we should just drop the 'u'. We rebuilt the objects, we might as well put them in the archive. Ian?

@gopherbot
Copy link

CL https://golang.org/cl/16664 mentions this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants