Re: Patch required to avoid breakage

[Bruce Korb]

On Wed, Aug 8, 2018 at 3:09 PM Bruno Haible <address@hidden> wrote:
> 1) The error occurs in directory libopts/, which contains a hand-made
> use of gnulib modules - without use of gnulib-tool. This is not
> supported, because internals of modules can change without notice.
> Instead, please take a look at this documentation section:
> https://www.gnu.org/software/gnulib/manual/html_node/Multiple-instances.html#Multiple-instances

Thank you. Maybe. I use gnulib-tool to pull the sources into autogen
and then spin off a libopts tarball. That tarball includes the gnulib stuff.
My intention was to be able to unroll it, add in ACLOCAL_AMFLAGS
and have it work. It needs work.

> 2) You hand-imported module 'stdnoreturn'. But it depends on module
> 'snippets/_Noreturn'. You didn't hand-import this module correctly.
> Namely, the 'Files: lib/_Noreturn.h' should translate into an action
> of yours that copies gnulib/lib/_Noreturn.h to libopts/_Noreturn.h.

I guess this is where my confusion lives. I see that when I specify
a mess of modules, those modules pull in their "Depends-on:"
modules. Mostly. I'll add snippet/_Noreturn to the autogen stuff
and see where that goes.

Again, thank you so very much for your help. I had gotten to
aimless wandering now knowing how far South I had gotten.

> 3) The EXTRA_DIST addition in libopts/Makefile.am is correct.
> gnulib-tool would do it for you, but you chose to do it by hand.

It is actually a computed list, based on what gets pulled in via gnulib-tool.

> 4) Later on,

I need to distribute the man page, despite the fact it is rebuldable.

You have been a great help. Thank you so much. Regards, Bruce