[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Patch required to avoid breakage
From: |
Bruce Korb |
Subject: |
Re: Patch required to avoid breakage |
Date: |
Wed, 8 Aug 2018 15:32:19 -0700 |
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