All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] build: propagate runstatedir
@ 2017-09-13  8:52 Wei Liu
  2017-09-13  9:07 ` Juergen Gross
  0 siblings, 1 reply; 4+ messages in thread
From: Wei Liu @ 2017-09-13  8:52 UTC (permalink / raw)
  To: Xen-devel; +Cc: Wei Liu, Ian Jackson

Debian Stretch's autoconf has a new variable called runstatedir. Add
that to Paths.mk.in.

Signed-off-by: Wei Liu <wei.liu2@citrix.com>
---
 config/Paths.mk.in | 1 +
 1 file changed, 1 insertion(+)

diff --git a/config/Paths.mk.in b/config/Paths.mk.in
index a603295146..660b515121 100644
--- a/config/Paths.mk.in
+++ b/config/Paths.mk.in
@@ -28,6 +28,7 @@ psdir                    := @psdir@
 includedir               := @includedir@
 localstatedir            := @localstatedir@
 sysconfdir               := @sysconfdir@
+runstatedir              := @runstatedir@
 
 LIBEXEC                  := @LIBEXEC@
 LIBEXEC_BIN              := @LIBEXEC_BIN@
-- 
2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] build: propagate runstatedir
  2017-09-13  8:52 [PATCH] build: propagate runstatedir Wei Liu
@ 2017-09-13  9:07 ` Juergen Gross
  2017-09-13  9:11   ` Wei Liu
  0 siblings, 1 reply; 4+ messages in thread
From: Juergen Gross @ 2017-09-13  9:07 UTC (permalink / raw)
  To: Wei Liu, Xen-devel; +Cc: Ian Jackson

On 13/09/17 10:52, Wei Liu wrote:
> Debian Stretch's autoconf has a new variable called runstatedir. Add
> that to Paths.mk.in.
> 
> Signed-off-by: Wei Liu <wei.liu2@citrix.com>

You did think of

https://lists.xen.org/archives/html/xen-devel/2017-02/msg01883.html

I suppose?


Juergen

> ---
>  config/Paths.mk.in | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/config/Paths.mk.in b/config/Paths.mk.in
> index a603295146..660b515121 100644
> --- a/config/Paths.mk.in
> +++ b/config/Paths.mk.in
> @@ -28,6 +28,7 @@ psdir                    := @psdir@
>  includedir               := @includedir@
>  localstatedir            := @localstatedir@
>  sysconfdir               := @sysconfdir@
> +runstatedir              := @runstatedir@
>  
>  LIBEXEC                  := @LIBEXEC@
>  LIBEXEC_BIN              := @LIBEXEC_BIN@
> 


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] build: propagate runstatedir
  2017-09-13  9:07 ` Juergen Gross
@ 2017-09-13  9:11   ` Wei Liu
  2017-09-20 15:09     ` Wei Liu
  0 siblings, 1 reply; 4+ messages in thread
From: Wei Liu @ 2017-09-13  9:11 UTC (permalink / raw)
  To: Juergen Gross; +Cc: Xen-devel, Wei Liu, Ian Jackson

On Wed, Sep 13, 2017 at 11:07:15AM +0200, Juergen Gross wrote:
> On 13/09/17 10:52, Wei Liu wrote:
> > Debian Stretch's autoconf has a new variable called runstatedir. Add
> > that to Paths.mk.in.
> > 
> > Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> 
> You did think of
> 
> https://lists.xen.org/archives/html/xen-devel/2017-02/msg01883.html
> 
> I suppose?
> 

Yes. That was added at a time when autoconf didn't have runstatedir
iirc. Now autoconf has that. There is no reason not to propagate it.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] build: propagate runstatedir
  2017-09-13  9:11   ` Wei Liu
@ 2017-09-20 15:09     ` Wei Liu
  0 siblings, 0 replies; 4+ messages in thread
From: Wei Liu @ 2017-09-20 15:09 UTC (permalink / raw)
  To: Juergen Gross; +Cc: Xen-devel, Wei Liu, Ian Jackson

On Wed, Sep 13, 2017 at 10:11:25AM +0100, Wei Liu wrote:
> On Wed, Sep 13, 2017 at 11:07:15AM +0200, Juergen Gross wrote:
> > On 13/09/17 10:52, Wei Liu wrote:
> > > Debian Stretch's autoconf has a new variable called runstatedir. Add
> > > that to Paths.mk.in.
> > > 
> > > Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> > 
> > You did think of
> > 
> > https://lists.xen.org/archives/html/xen-devel/2017-02/msg01883.html
> > 
> > I suppose?
> > 
> 
> Yes. That was added at a time when autoconf didn't have runstatedir
> iirc. Now autoconf has that. There is no reason not to propagate it.

Thinking about this a bit more, exposing runstatedir like this doesn't
have much value. I want to make it work with the existing option that we
already have. 

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-09-20 15:10 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-13  8:52 [PATCH] build: propagate runstatedir Wei Liu
2017-09-13  9:07 ` Juergen Gross
2017-09-13  9:11   ` Wei Liu
2017-09-20 15:09     ` Wei Liu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.