On Tuesday, February 19, 2019 7:18:57 PM CET Bruno Haible wrote:
> Hi Pavel,
>
> > > makes the IFS handling a bit more robust.
>
> What I meant is that
>
>save_IFS="$IFS"
>for fpf_dir in $fpf_dirs
>do
> IFS="$save_IFS"
> [Some more code]
>done
>IFS="$save_IFS"
>
>
Paul Smith noted on gnu-prog-discuss that other systems than glibc have support
for $ORIGIN in rpath. This patch changes the 'relocatable-prog' module to
make use of this feature, thus removing the need for a "wrapper"/"trampoline"
executable on these platforms.
Tested on
- FreeBSD 11,
- NetBS
This set of patches improve the platform specific code in progreloc.c.
Most BSD systems and Solaris nowadays have a way to retrieve the name
of the current executable, in a way similar to /proc/self/exe on Linux.
2019-02-19 Bruno Haible
progreloc: Simplify code for Android.
*
On Tue, Feb 19, 2019 at 10:02 AM Bruno Haible wrote:
>
> Hi Pavel,
>
> > > [...]
> > > This patch fixes both issues, and makes the IFS handling a bit more
> > > robust.
> > > [...]
> >
> > > -case $_fpf_arg in
> > > +case "$_fpf_arg" in
> > > [...]
> > > - fpf_dirs=$1 ; shift
> > > - fp
Hi Pavel,
> > makes the IFS handling a bit more robust.
What I meant is that
save_IFS="$IFS"
for fpf_dir in $fpf_dirs
do
IFS="$save_IFS"
[Some more code]
done
IFS="$save_IFS"
is more robust than
save_IFS="$IFS"
for fpf_dir in $fpf_dirs
do
[Some more code]
Hi Pavel,
> > [...]
> > This patch fixes both issues, and makes the IFS handling a bit more robust.
> > [...]
>
> > -case $_fpf_arg in
> > +case "$_fpf_arg" in
> > [...]
> > - fpf_dirs=$1 ; shift
> > - fpf_cb=$1 ; shift
> > + fpf_dirs="$1"; shift
> > + fpf_cb="$1"; shift
> > [...]
>
> From: Bruno Haible
> Cc: Eli Zaretskii , Paul Eggert
> Date: Tue, 19 Feb 2019 00:32:24 +0100
>
> Hi Eli,
>
> > ./alloca.h:40:18: error: inlining failed in call to always_inline
> > '__builtin_alloca': function not considered for inlining
> > # define alloca __builtin_alloca
> >
Thanks for working on this, Bruno! Only nits..
On Thursday, February 14, 2019 8:53:33 PM CET Bruno Haible wrote:
> [...]
> This patch fixes both issues, and makes the IFS handling a bit more robust.
> [...]
> -case $_fpf_arg in
> +case "$_fpf_arg" in
> [...]
> - fpf_dirs=$1 ; shift
> -
1. Accept dates in the ISO 8601 basic date and time format. This
is a format allowing the expression of point in time to the resolution
of a second with only alphanumeric characters -- no spaces, no
punctuation -- suitable for use in fields that cannot contain spaces,
hyphens, or colons.
2. Accept
