Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Added target directory creation for configuration file. #52

Closed
wants to merge 1 commit into from

1 participant

Thorsten Fischer
Thorsten Fischer

The installation process of the barnyard2.conf configuration file assumes that
the target directory exists. This breaks 'make install' calls for prefixes
which do not exist at the time of installation, resulting in a destination
file name of 'etc'. We therefore explicitly create the destination
folder now.

Signed-off-by: Thorsten Fischer thorsten@froschi.org

Thorsten Fischer froschi Added target directory creation for configuration file.
The installation process of the barnyard2.conf configuration file assumes that
the target directory exists. This breaks 'make install' calls for prefixes
which do not exist at the time of installation, resulting in a destination
file name of 'etc'. We therefore explicitly create the destination
folder now.

Signed-off-by: Thorsten Fischer <thorsten@froschi.org>
562f796
Thorsten Fischer froschi closed this
Thorsten Fischer

Closing this pull request; I just saw that there is another one which addresses the same issue, but better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 25, 2012
  1. Thorsten Fischer

    Added target directory creation for configuration file.

    froschi authored
    The installation process of the barnyard2.conf configuration file assumes that
    the target directory exists. This breaks 'make install' calls for prefixes
    which do not exist at the time of installation, resulting in a destination
    file name of 'etc'. We therefore explicitly create the destination
    folder now.
    
    Signed-off-by: Thorsten Fischer <thorsten@froschi.org>
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 1 deletion.
  1. +2 −1  etc/Makefile.am
3  etc/Makefile.am
View
@@ -4,4 +4,5 @@ AUTOMAKE_OPTIONS=foreign no-dependencies
EXTRA_DIST = barnyard2.conf
install-data-am:
- test -e $(sysconfdir)/barnyard2.conf || install -m 600 $(top_srcdir)/etc/barnyard2.conf $(sysconfdir)
+ test -d $(sysconfdir) || install -m 755 -d $(sysconfdir)
+ test -e $(sysconfdir)/barnyard2.conf || install -m 600 $(top_srcdir)/etc/barnyard2.conf $(sysconfdir)/barnyard2.conf
Something went wrong with that request. Please try again.