From: Anton Lundin <glance@ac2.se>
To: lvm-devel@redhat.com
Subject: [PATCH] Fix header include for out of tree builds
Date: Fri, 2 Jun 2023 19:30:35 +0200 [thread overview]
Message-ID: <20230602173035.GJ18946@accum.se> (raw)
In-Reply-To: <9d6b1dc2-fe01-c80a-d5f6-ba359f0683ff@redhat.com>
On 17 May, 2023 - Zdenek Kabelac wrote:
> Dne 17. 05. 23 v 14:09 Zdenek Kabelac napsal(a):
> > Dne 17. 05. 23 v 13:50 Anton Lundin napsal(a):
> > > On 17 May, 2023 - Zdenek Kabelac wrote:
> > >
> > > > Dne 17. 05. 23 v 13:35 Anton Lundin napsal(a):
> > > > > On 17 May, 2023 - Zdenek Kabelac wrote:
> > > > >
> > > > > > Dne 16. 05. 23 v 18:05 Anton Lundin napsal(a):
> > > > > > > The relative include didn't match up for out of tree builds. The
> > > > > > > build tree include directory is in the include paths already.
> > > > > > >
> > > > > > > Signed-off-by: Anton Lundin <glance@ac2.se>
> > > > > > > ---
> > > > > > >
> > > > > > > Now the following works:
> > > > > > > $ mkdir lvm2-build
> > > > > > > $ cd lvm2-build
> > > > > > > $ ../lvm2/configure
> > > > > > Hi
> > > > > >
> > > > > > Your bug is actually in this command:
> > > > > >
> > > > > > configure --srcdir=XXXXX
> > > > > >
> > > > > > > $ make
> > > > > > >
> > > > > > Otherwise srcdir is located in your builddir and you
> > > > > > would actually get more
> > > > > > errors in various parts of lvmb building - as we tend to use $(srcdir)
> > > > > > inside makefiles..
> > > > > I might be misunderstanding you. Are your comment about the fix or the
> > > > > example on now to trigger the bug?
> > > > >
> > > > > Just running configure from another directory will set srcdir
> > > > > automatically. At least in any kind of modern auto-tools.
> > > >
> > > > While we certainly can push this minor change? (and I'll likely push this
> > > > change upstream with my next changeset) - the usage should be based on
> > > > srcdir settings - which I'm setting myself for a long time? - as there were
> > > > always some problems if this was not set properly.
> > > >
> > > > So I could try if this got eventually better over time? - but lvm2 is using
> > > > somewhat hybrid approach - as we only use 'autoconf' but not 'automake'
> > > > historically.
> > > With this fix out of tree builds works with or without --srcdir= , and
> > > if I revert this fix on my local tree the builds fails with or without
> > > --srcdir= like:
> > >
> > > In file included from ../../lvm2/lib/commands/toolcontext.h:21,
> > > ????????????????? from ../../lvm2/lib/integrity/integrity.c:17:
> > > ???????????????? ../../lvm2/lib/commands/cmd_enum.h:15:10: fatal error:
> > > ???????????????? ../../include/cmds.h: No such file or directory
> > > ??????????????????? 15 | #include "../../include/cmds.h"
> > >
> > > //Anton
> >
> >
> > Can you please provide? corresponding??? 'make V=1' log ?
>
>
> Actuallly not needed - reproduced myself.
I haven't seen this patch pused out, or anything like it. Is there
anything I can help out with?
//Anton
next prev parent reply other threads:[~2023-06-02 17:30 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-16 16:05 [PATCH] Fix header include for out of tree builds Anton Lundin
2023-05-17 10:29 ` Zdenek Kabelac
2023-05-17 11:35 ` Anton Lundin
2023-05-17 11:41 ` Zdenek Kabelac
2023-05-17 11:50 ` Anton Lundin
2023-05-17 12:09 ` Zdenek Kabelac
2023-05-17 14:33 ` Zdenek Kabelac
2023-06-02 17:30 ` Anton Lundin [this message]
2023-06-05 13:58 ` Zdenek Kabelac
-- strict thread matches above, loose matches on Subject: below --
2023-05-16 16:09 Anton Lundin
2023-05-16 16:20 Anton Lundin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230602173035.GJ18946@accum.se \
--to=glance@ac2.se \
--cc=lvm-devel@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).