[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[STINNER Victor]

Change by STINNER Victor :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.10, Python 3.11

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[STINNER Victor]

STINNER Victor  added the comment:

Thanks to the bug report Andrew C. Morrow, it's now fixed.

Commits in main:

commit 59acfd4a09df1c141dac7845eed008af8970fce7
Author: Ned Deily 
Date:   Mon May 24 23:00:04 2021 -0400

bpo-43109: Fix --with-lto configure option on macOS (GH-26341)

Allow --with-lto configure option to work with Apple-supplied Xcode or
Command Line Tools which do not provide llvm-ar.

commit 63f17c252ab1b9357c5f022296213c42d9629458 (HEAD -> main, upstream/main)
Author: Victor Stinner 
Date:   Tue May 25 12:25:22 2021 +0200

bpo-43109: configure doc: LTO requires "ar" on macOS (GH-26349)


Commits in 3.10 branch:

commit ddc503c8d20c4893e69b5bf466ad85da6a3f0be6 (upstream/3.10, 3.10)
Author: Miss Islington (bot) <31488909+miss-isling...@users.noreply.github.com>
Date:   Tue May 25 03:56:37 2021 -0700

bpo-43109: configure doc: LTO requires "ar" on macOS (GH-26349)

(cherry picked from commit 63f17c252ab1b9357c5f022296213c42d9629458)

Co-authored-by: Victor Stinner 

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[miss-islington]

miss-islington  added the comment:


New changeset ddc503c8d20c4893e69b5bf466ad85da6a3f0be6 by Miss Islington (bot) 
in branch '3.10':
bpo-43109: configure doc: LTO requires "ar" on macOS (GH-26349)
https://github.com/python/cpython/commit/ddc503c8d20c4893e69b5bf466ad85da6a3f0be6


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[miss-islington]

Change by miss-islington :


--
pull_requests: +24940
pull_request: https://github.com/python/cpython/pull/26350

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +24939
pull_request: https://github.com/python/cpython/pull/26349

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[Ned Deily]

Ned Deily  added the comment:


New changeset 3af61a7176a68bfeb349eeed314b9c3d7ebc3ad5 by Miss Islington (bot) 
in branch '3.9':
bpo-43109: Fix --with-lto configure option on macOS (GH-26341) (GH-26343)
https://github.com/python/cpython/commit/3af61a7176a68bfeb349eeed314b9c3d7ebc3ad5


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[miss-islington]

miss-islington  added the comment:


New changeset 25a9cf197ea0d77abd49992a7751efa0046bb1e6 by Miss Islington (bot) 
in branch '3.10':
bpo-43109: Fix --with-lto configure option on macOS (GH-26341)
https://github.com/python/cpython/commit/25a9cf197ea0d77abd49992a7751efa0046bb1e6


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[miss-islington]

Change by miss-islington :


--
pull_requests: +24935
pull_request: https://github.com/python/cpython/pull/26343

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[miss-islington]

Change by miss-islington :


--
nosy: +miss-islington
nosy_count: 5.0 -> 6.0
pull_requests: +24934
pull_request: https://github.com/python/cpython/pull/26342

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[Ned Deily]

Change by Ned Deily :


--
pull_requests: +24931
pull_request: https://github.com/python/cpython/pull/26341

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[Ned Deily]

Ned Deily  added the comment:

Thanks for the PR. I am looking at this area currently.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[Ned Deily]

Change by Ned Deily :


--
assignee:  -> ned.deily
components: +macOS
nosy: +ned.deily, ronaldoussoren

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[Rishav Kundu]

Change by Rishav Kundu :


--
keywords: +patch
nosy: +xrisk
nosy_count: 2.0 -> 3.0
pull_requests: +24652
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/25999

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[STINNER Victor]

Change by STINNER Victor :


--
nosy: +vstinner

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue43109] When using Apple Clang, --with-lto builds should not check for llvm-ar

[Andrew C. Morrow]

New submission from Andrew C. Morrow :

When building with `--with-lto`, an explicit check for `llvm-ar` is performed 
as part of the configure step. However, Apple does not ship `llvm-ar` as part 
of Xcode, so when building with Apple Clang this check fails and prevents using 
`--with-lto`.

However, using the builtin `ar` seems to work just fine, and it can be selected 
by passing `LLVM_AR=/path/to/ar` to `configure`. It would be nice to see the 
`configure` script updated to know that system `ar` is sufficient when using 
Apple Clang on macOS.

Please see https://github.com/Homebrew/homebrew-core/pull/70177 for additional 
background.

--
components: Build
messages: 386174
nosy: acmorrow
priority: normal
severity: normal
status: open
title: When using Apple Clang, --with-lto builds should not check for llvm-ar
type: enhancement
versions: Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com