Re: [PATCH] tools: link libintl if discovered by AM_GNU_GETTEXT

[Arsen Arsenović]

Hi Mike,

Mike Frysinger <vapier@gentoo.org> writes:
> this patch seems to be based on a tree that isn't ours, or requires some
> other unrelated patches.  can you please use our tree only with no other
> custom changes ?  as is, it doesn't apply.
> -mike

Not sure what happened there, to be honest.  Either way, I prepared a
new patch.  This one should also be more correct.

Tested on x86_64-pc-linux-gnu, via the Gentoo ebuild, and on
x86_64-linux-mlibc, via the usual, simple autotools build.  On mlibc
(where the libc does not provide gettext), libintl is properly linked
into libacl.so:

  [i] ~/.../acl/usr/lib$ eu-readelf --dynamic libacl.so | grep NEEDED
    NEEDED            Shared library: [libintl.so.9]
    NEEDED            Shared library: [libattr.so.1]
    NEEDED            Shared library: [libc.so]

... and getfacl does run:
  bash-5.1# getfacl /
  mlibc: uselocale() is a no-op
  mlibc: uselocale() is a no-op
  getfacl: Removing leading '/' from absolute path names
  # file: .
  # owner: 1001
  # group: 1001
  user::rwx
  group::r-x
  other::r-x
  getfacl: .: No such process (ESRCH)

Bar libc flaws, naturally.

v2-0001-Add-LTLIBINTL-to-libacl_LIBADD-tools_ldadd.patch
Description: [PATCH v2] *: Add $(LTLIBINTL) to libacl_LIBADD tools_ldadd

Thanks, and have a good day.
-- 
Arsen Arsenović

signature.asc
Description: PGP signature