Bug 144988

Summary:

[GTK] ar warning when linking static libraries

Product:

WebKit

Reporter:

Carlos Garcia Campos <cgarcia>

Component:

WebKitGTK

Assignee:

Michael Catanzaro <mcatanzaro>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, mcatanzaro, mrobinson, pnormand, zan

Priority:

Keywords:

Gtk

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	none

Carlos Garcia Campos

Reported 2015-05-13 23:11:34 PDT

Linking CXX static library lib/libbmalloc.a /usr/bin/ar: `u' modifier ignored since `D' is the default (see `U') It started to happen recently, probably because I upgraded binutils and this warning is new, and it happens for every static library we build. I have no idea what it means, bu in this redhat bug it's suggested to use either cr or crUu instead of cru as we currently use (and T also).

Attachments
Patch (2.83 KB, patch) 2015-07-09 18:14 PDT, Michael Catanzaro	no flags	Details Formatted Diff Diff
Patch (2.72 KB, patch) 2015-07-09 18:16 PDT, Michael Catanzaro	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Carlos Garcia Campos

Comment 1 2015-05-13 23:12:05 PDT

Forgot the link to the rh bz, https://bugzilla.redhat.com/show_bug.cgi?id=1155273

Michael Catanzaro

Comment 2 2015-07-09 17:40:40 PDT

See also: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=20082

Michael Catanzaro

Comment 3 2015-07-09 18:14:11 PDT

Created attachment 256550 [details] Patch

Michael Catanzaro

Comment 4 2015-07-09 18:16:50 PDT

Created attachment 256551 [details] Patch

Carlos Garcia Campos

Comment 5 2015-07-09 23:15:07 PDT

Comment on attachment 256551 [details] Patch Ok, sounds reasonable. Thanks!

Philippe Normand

Comment 6 2015-07-10 00:12:39 PDT

Comment on attachment 256551 [details] Patch Shouldn't we check the binutils version before removing the 'u'?

Michael Catanzaro

Comment 7 2015-07-10 05:13:39 PDT

(In reply to comment #6) > Comment on attachment 256551 [details] > Patch > > Shouldn't we check the binutils version before removing the 'u'? I don't think so, since the issue is only slightly related to binutils version. What matters is whether the distro has decided to use deterministic builds by default or not. If we could find a way to check that, then we could use it determine whether or not to use u, but I don't think anybody else plans to do that, and there's not much point; I'd rather just remove 'u'. Also, since we use 'T' for thin archives, I doubt there's much any benefit to 'u' (but I haven't measured).

WebKit Commit Bot

Comment 8 2015-07-10 05:34:25 PDT

Comment on attachment 256551 [details] Patch Clearing flags on attachment: 256551 Committed r186670: <http://trac.webkit.org/changeset/186670>

WebKit Commit Bot

Comment 9 2015-07-10 05:34:31 PDT

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.