From: Junio C Hamano <gitster@pobox.com>
To: Jeffrey Walton <noloader@gmail.com>
Cc: Jeff King <peff@peff.net>, Git List <git@vger.kernel.org>
Subject: Re: Git configure/make does not honor ARFLAGS
Date: Mon, 14 Sep 2015 21:56:00 -0700 [thread overview]
Message-ID: <xmqq1te0uvq7.fsf@gitster.mtv.corp.google.com> (raw)
In-Reply-To: <CAH8yC8kaWi6F5iKhNQeJrG451hqS-3_crtA0c+xFner78E2CAQ@mail.gmail.com> (Jeffrey Walton's message of "Mon, 14 Sep 2015 19:14:23 -0400")
Jeffrey Walton <noloader@gmail.com> writes:
>> Oh, indeed. ac179b4d9. Looks good to me.
>>
>> My follow-up question was going to be: is this something we should be
>> setting in config.mak.uname for appropriate versions of Darwin? It
>> wasn't clear to me from Eric's description if this is something that
>> particular versions need, or just something that people who want to
>> build Universal binaries would choose to use.
>>
> If you have something that needs testing, then please let me know. I'd
> be happy to test it for you.
In your copy of Git that is recent enough, please run this:
$ git fetch git://git.kernel.org/pub/scm/git/git.git pu
$ git checkout -b jw/make-arflags-customizable ac179b4d9
This gives you a rather old maintenance track of Git based on v2.0.5
plus a patch that makes $(ARFLAGS) customizable.
Presumably you are building a more recent versions of Git yourself.
I do not know which branch you are on, but you should be able to
merge this brancch to anything newer than v2.0.x maintenance track.
E.g.
$ git checkout master
$ git merge jw/make-arflags-customizable
should merge cleanly and would let you say things like:
$ make AR=libtool ARFLAGS='-static -o'
with any other build customization you usually use on the command
line (or in config.mak, if you prefer).
What Jeff King was wondering was if those on platform like you have
would benefit if they have something like the attached patch on top
of the ac179b4d (Makefile: allow $(ARFLAGS) specified from the
command line, 2015-09-10) patch. Pieces of information that is
needed to determine if that is a good idea are:
* What exact condition we should use in the "ifeq" to identify the
class of platforms that want this custom AR/ARFLAGS.
* Ask if everybody who shares that particular platform would always
want "libtool" and "-static -o" as their custom AR/ARFLAGS. If
the answer is "no", i.e. on the same platform, more than one
compiler toolchain may be available to the users and not
everybody may want to use "libtool" and "-static -o" lie you do,
then the approach to auto-determine this settings does not help
people, and we should not bother with shipping customized
config.mak.uname (instead we'd just tell people to do whatever
they want with AR and ARFLAGS from the command line).
* If it is worth shipping customized config.mak.uname, determine
what things, other than AR/ARFLAGS, would help the uses of the
class of platforms we identified above in the new "ifeq" section
(e.g. perhaps they may want CC=some-other-cc).
config.mak.uname | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/config.mak.uname b/config.mak.uname
index 943c439..247dfed 100644
--- a/config.mak.uname
+++ b/config.mak.uname
@@ -46,6 +46,10 @@ ifeq ($(uname_S),GNU/kFreeBSD)
DIR_HAS_BSD_GROUP_SEMANTICS = YesPlease
LIBC_CONTAINS_LIBINTL = YesPlease
endif
+ifeq ($(uname_S),"whatever jeffrey uses")
+ AR = libtool
+ ARFLAGS = -static -o
+endif
ifeq ($(uname_S),UnixWare)
CC = cc
NEEDS_SOCKET = YesPlease
prev parent reply other threads:[~2015-09-15 4:56 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-30 21:34 Git configure/make does not honor ARFLAGS Jeffrey Walton
2015-09-13 10:17 ` Jeff King
2015-09-13 17:11 ` Eric Sunshine
2015-09-13 18:37 ` Jeffrey Walton
2015-09-14 4:30 ` Jeff King
2015-09-14 4:57 ` Junio C Hamano
2015-09-14 4:59 ` Jeff King
2015-09-14 5:52 ` Junio C Hamano
2015-09-16 19:38 ` Eric Sunshine
2015-09-16 19:45 ` Eric Sunshine
2015-09-14 23:14 ` Jeffrey Walton
2015-09-15 4:56 ` Junio C Hamano [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqq1te0uvq7.fsf@gitster.mtv.corp.google.com \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=noloader@gmail.com \
--cc=peff@peff.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.