Bug 730388 – net-dialup/freeradius-client calls ar directly

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description Agostino Sarubbo 2020-07-01 14:32:29 UTC]

@@This is an auto-filed bug@@
Feel free to change the summary at your convenience.

Issue: net-dialup/freeradius-client calls ar directly.
Discovered on: amd64

NOTE:
As per QA policy, toolchain tools must not be called directly because they can cause issue in cross-compiling and because is not possible use a different AR implementation (like llvm-ar). To reproduce, please use sys-devel/gcc-config[-native-symlinks], sys-devel/binutils-config[-native-symlinks] and set the AR variable to x86_64-pc-linux-gnu-ar.

[Comment 1 Agostino Sarubbo 2020-07-01 14:32:41 UTC]

Created attachment 647310 [details]
build.log

build log and emerge --info

[Comment 2 Agostino Sarubbo 2020-07-01 14:32:47 UTC]

Created attachment 647312 [details]
other.tar.bz2

other logs

[Comment 3 Alarig Le Lay 2020-07-01 15:02:09 UTC]

Hi,

Did you used the last git version of the tree? https://gitweb.gentoo.org/repo/gentoo.git/commit/net-dialup/freeradius-client/freeradius-client-1.1.7-r1.ebuild?id=6c35e0080c2fcd69c99bee94e7d96c82d555558a has been merged five hours ago, and it should fix this issue.

[Comment 4 Agostino Sarubbo 2020-07-01 15:23:01 UTC]

At the beginning of the build log you will see a link to the git head at the time of emerging.

[Comment 5 Alarig Le Lay 2020-07-08 17:40:16 UTC]

Created attachment 648442 [details]
patched version shell session

Hi,

I wasn’t able to do a full fix with my old patch, so I used a totaly opposite fix. I patched configure.ac to look for x86_64-pc-linux-gnu-ar if ar is not found.

checking for ar... no
checking for x86_64-pc-linux-gnu-ar... /usr/bin/x86_64-pc-linux-gnu-ar

See attachment for the complete shell session of the emerge.

If it’s OK, I can open a PR on github.

[Comment 6 Sam James 2020-08-30 23:39:42 UTC]

(In reply to Alarig Le Lay from comment #3)
> Hi,
> 
> Did you used the last git version of the tree?
> https://gitweb.gentoo.org/repo/gentoo.git/commit/net-dialup/freeradius-
> client/freeradius-client-1.1.7-r1.
> ebuild?id=6c35e0080c2fcd69c99bee94e7d96c82d555558a has been merged five
> hours ago, and it should fix this issue.

This should be fine? You don't want to assume /usr/bin/ar because even if it exists, it may not be the preferred implementation.

Try it out by doing:
"To reproduce, please use sys-devel/gcc-config[-native-symlinks], sys-devel/binutils-config[-native-symlinks] and set the AR variable to x86_64-pc-linux-gnu-ar."

... you may want to mv /usr/bin/ar /usr/bin/ar.bak too btw.

[Comment 7 Alarig Le Lay 2020-08-31 19:33:37 UTC]

Thanks!

I tested it on an LXC with those USE flags and it compiled (while with my first flag it didn’t).

I created the PR on github, as always don’t hesitate to make comments if you want me to change things.

[Comment 8 Larry the Git Cow 2020-09-02 20:41:10 UTC]

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=97cf65fd933a09edc3bd7282c4d8f10ab27fb227

commit 97cf65fd933a09edc3bd7282c4d8f10ab27fb227
Author:     Alarig Le Lay <[e-mail]>
AuthorDate: 2020-08-31 19:27:17 +0000
Commit:     Sam James <[e-mail]>
CommitDate: 2020-09-02 20:40:06 +0000

    net-dialup/freeradius-client: respect AR
    
    Closes: https://bugs.gentoo.org/730388
    Package-Manager: Portage-3.0.4, Repoman-2.3.23
    Signed-off-by: Alarig Le Lay <[e-mail]>
    Closes: https://github.com/gentoo/gentoo/pull/17351
    Signed-off-by: Sam James <[e-mail]>

 .../files/freeradius-client-1.1.7-ar-configure.in.patch    | 10 ++++++++++
 .../freeradius-client/freeradius-client-1.1.7-r1.ebuild    | 14 ++++++--------
 2 files changed, 16 insertions(+), 8 deletions(-)