Bug 381399 – app-portage/pfl-2.2: installs bash script with /bin/sh shebang

• Status: RESOLVED FIXED
• Product: Portage Development

[Description Michał Górny 2011-09-01 08:15:01 UTC]

Created attachment 285227 [details, diff]
Patch to fix the issue

Like the summary states. Patch attached.

[Comment 1 bugs 2011-09-01 20:19:01 UTC]

Hi,
thanks for the patch.

The patch is not changing the shebang. I think it would be the best to apply your patch and changing the shebang from /bin/sh to /bin/bash, right?

cheers
Daniel

[Comment 2 bugs 2011-09-01 20:20:57 UTC]

(In reply to comment #1)
> The patch is not changing the shebang. I think it would be the best to apply
> your patch and changing the shebang from /bin/sh to /bin/bash, right?

Ups, my fault. The patch is converting the bash syntax into sh.

[Comment 3 Michał Górny 2011-09-01 20:28:57 UTC]

(In reply to comment #2)
> (In reply to comment #1)
> > The patch is not changing the shebang. I think it would be the best to apply
> > your patch and changing the shebang from /bin/sh to /bin/bash, right?
> 
> Ups, my fault. The patch is converting the bash syntax into sh.

Yep, it is. The script is simple enough not to actually use bash; it's all basically awk.

[Comment 4 bugs 2011-09-06 21:03:05 UTC]

Hi,
updated e-file http://files.portagefilelist.de/e-file-20110906

I think we should wait on #381403 until we create a new ebuild.

cheers
Daniel

[Comment 5 Daniel Pielmeier 2011-10-19 17:47:25 UTC]

@Michał: Thank you for report this is fixed in pfl-2.3

@Daniel: Thanks for the new version.

Sorry for it taking a bit longer, I am still catching up after switching home/work.