Bug 366671 – sys-apps/attr-2.4.46 fails to build if /bin/sh is not bash

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Alec Moskvin 2011-05-09 23:26:09 UTC]

attr fails to build with /bin/sh set to a different POSIX shell.


Reproducible: Always

Steps to Reproduce:
1. set /bin/sh to point to /bin/dash
2. emerge =sys-apps/attr-2.4.46


Actual Results:  
...
eval: 1: base_compile+= x86_64-pc-linux-gnu-gcc: not found
eval: 1: base_compile+= -O2: not found
eval: 1: base_compile+= -march=native: not found
eval: 1: base_compile+= -pipe: not found
...


Expected Results:  
/bin/sh should not be used on bash scripts.

[Comment 1 Alec Moskvin 2011-05-09 23:26:30 UTC]

Created attachment 272683 [details]
build.log

[Comment 2 Alec Moskvin 2011-05-09 23:27:27 UTC]

Created attachment 272685 [details, diff]
Fix for attr-2.4.46.ebuild

This patch replaces SHELL=/bin/sh with SHELL=/bin/bash in include/buildmacros,
which causes the build to complete successfully.

[Comment 3 Alec Moskvin 2011-05-09 23:28:37 UTC]

Created attachment 272687 [details]
Patched ebuild (for convenience)

[Comment 4 Otamay 2011-05-12 07:26:01 UTC]

Confirmed this and Bug 366393 on Gentoo AMD64 with /bin/dash. Patch works, too.

¡Thank you!

[Comment 5 Alec Moskvin 2011-05-15 19:13:15 UTC]

Created attachment 273317 [details, diff]
Fix for attr-2.4.46.ebuild, by passing SHELL to emake

A better fix as suggested by mgorny in bug 366393.

[Comment 6 SpanKY 2011-05-16 05:52:13 UTC]

Comment on attachment 273317 [details, diff]
Fix for attr-2.4.46.ebuild, by passing SHELL to emake

this ignores the actual problem

[Comment 7 SpanKY 2011-05-16 06:06:10 UTC]

probably fixed now

http://sources.gentoo.org/sys-apps/attr/attr-2.4.46.ebuild?r1=1.1&r2=1.2
http://sources.gentoo.org/sys-apps/attr/files/attr-2.4.46-config-shell.patch?rev=1.1

[Comment 8 Alec Meyers 2011-05-16 18:16:59 UTC]

Thanks, it works.