musl - Re: [PATCH] Build process uses script to add CFI directives to x86 asm

Follow @Openwall on Twitter for new release announcements and other news

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150520170819.GA24879@alex-ThinkPad-L530>
Date: Wed, 20 May 2015 19:08:19 +0200
From: Alex Dowad <alexinbeijing@...il.com>
To: musl@...ts.openwall.com
Subject: Re: [PATCH] Build process uses script to add CFI directives to x86
 asm

> Writing portable shell script is a little hard, but I've found a good
> approach is to make something ancient like pdksh a symlink for /bin/sh and
> even /bin/bash - typically in a small VM. This makes it much more likely
> that your code will work without big changes on AIX, BSDs, etc. The other
> way is to use dash, not BusyBox ash, as BusyBox ash includes a few bashisms
> depending on the configure options at build time.

Just tried swapping all references to /bin/sh to /bin/pdksh instead -- the
config script ran just fine and musl was built correctly (with the added CFI
data). Same with /bin/dash.

Thanks for the idea -- I will keep it in mind next time I am writing shell.

Maintainers: is there still anything blocking this patch from getting merged
in? Please let me know so I can fix any remaining problems.

Thanks,
Alex Dowad

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.