Re: [PATCH] mig: Drop the Perl runtime dependency

[Thomas Schwinge]

Hi!

On Tue, 22 Jul 2014 09:58:51 -0400, David Michael <fedora.dm0@gmail.com> wrote:
> On Tue, Jul 22, 2014 at 9:12 AM, Thomas Schwinge
> <thomas@codesourcery.com> wrote:
> > On Fri, 18 Jul 2014 19:29:07 -0400, David Michael <fedora.dm0@gmail.com> 
> > wrote:
> >> I tried running mig on a system without Perl ealier with bad results.
> >> Perl seems to be used only for functionality that is also provided by
> >> GNU coreutils (which is already required, e.g. for dirname).  Does it
> >> make sense to drop the Perl command in favor of using realpath here?
> >
> >> -libexecdir_rel=$(perl -MFile::Spec -e 'print 
> >> File::Spec->abs2rel("'"$libexecdir"'","'"$bindir"'")')
> >> +libexecdir_rel=$(realpath --relative-to="$bindir" "$libexecdir")
> >
> > Thanks for the patch, and I'd be happy to apply that, however, on four
> > arbitrary (Debian, Ubuntu, Trisquel) systems that I just tried, I either
> > see: »realpath: unrecognized option '--relative-to'«, or »bash: realpath:
> > command not found«, whereas the Perl variant works on all of these.
> 
> Okay, I didn't realize that it had only been two or three years since
> a coreutils release first included this functionality.  Sorry for the
> noise.

Not noise at all -- we always welcome such cleanup/simplification
patches, so keep these coming.  Heh, and I'll set a note in my calendar
for 2020, to revisit this.  :-) (In the Hurd project, we're never in a
hurry.)


Grüße,
 Thomas

pgpLAJlBYkVVh.pgp
Description: PGP signature