diff mbox

[ovs-dev,v0] lib: Create $(sysconfdir)/openvswitch upon install

Message ID 1473783975-24077-1-git-send-email-flavio@flaviof.com
State Superseded
Headers show

Commit Message

Flaviof Sept. 13, 2016, 4:26 p.m. UTC
In cases where dbdir and etcdir are not the same, there is a need
for creating etcdir (i.e. $(sysconfdir)/openvswitch) explicitly.

Note that there is no attempt being made here to make the etcdir
configurable as in "--with-dbdir".

Reported-at: http://openvswitch.org/pipermail/dev/2016-September/TBD.html
Fixes: f973f2af2fd4 ("Make the location of the database separately configurable.")
Signed-off-by: Flavio Fernandes <flavio@flaviof.com>
---
 lib/automake.mk | 1 +
 1 file changed, 1 insertion(+)

Comments

Ben Pfaff Sept. 14, 2016, 5:55 p.m. UTC | #1
On Tue, Sep 13, 2016 at 11:26:15AM -0500, Flavio Fernandes wrote:
> In cases where dbdir and etcdir are not the same, there is a need
> for creating etcdir (i.e. $(sysconfdir)/openvswitch) explicitly.
> 
> Note that there is no attempt being made here to make the etcdir
> configurable as in "--with-dbdir".
> 
> Reported-at: http://openvswitch.org/pipermail/dev/2016-September/TBD.html
> Fixes: f973f2af2fd4 ("Make the location of the database separately configurable.")
> Signed-off-by: Flavio Fernandes <flavio@flaviof.com>

Seems reasonable, thanks.  I applied this to master and branch-2.6.
diff mbox

Patch

diff --git a/lib/automake.mk b/lib/automake.mk
index b00e90f..e9d508a 100644
--- a/lib/automake.mk
+++ b/lib/automake.mk
@@ -528,3 +528,4 @@  lib-install-data-local:
 	$(MKDIR_P) $(DESTDIR)$(PKIDIR)
 	$(MKDIR_P) $(DESTDIR)$(LOGDIR)
 	$(MKDIR_P) $(DESTDIR)$(DBDIR)
+	$(MKDIR_P) $(DESTDIR)$(sysconfdir)/openvswitch