GNU bug report logs - #29301
[PATCH] substitute* hard-coded paths with paths from "out"

Previous Next

Package: guix-patches;

Reported by: Kristofer Buffington <kristoferbuffington <at> gmail.com>

Date: Wed, 15 Nov 2017 01:04:01 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Kristofer Buffington <kristoferbuffington <at> gmail.com>
To: 29301 <at> debbugs.gnu.org
Subject: [bug#29301] Fwd: [bug#29301] [PATCH] substitute* hard-coded paths with paths from "out"
Date: Sun, 26 Nov 2017 20:31:52 -0500
[Message part 1 (text/plain, inline)]
---------- Forwarded message ----------
From: Kristofer Buffington <kristoferbuffington <at> gmail.com>
Date: Sun, Nov 26, 2017 at 8:30 PM
Subject: Re: [bug#29301] [PATCH] substitute* hard-coded paths with paths
from "out"
To: Ludovic Courtès <ludo <at> gnu.org>


I will investigate. There are some hard coded paths in artanis/config.scm
looking for the configuration file in /etc/artanis/artanis.conf -- I don't
believe the --sysconfdir flag will have any impact on the hard-coded paths.
Perhaps I should send a patch upstream to eliminate the hard paths? Then
the substitute* would be unnecessary?

Thanks!
Kris

On Thu, Nov 23, 2017 at 4:59 PM, Ludovic Courtès <ludo <at> gnu.org> wrote:

> Hi Kristofer,
>
> Kristofer Buffington <kristoferbuffington <at> gmail.com> skribis:
>
> > * gnu/packages/guile.scm
> > used substitute* to correct some hardcoded paths in the artanis package
>
> [...]
>
> > +                 (substitute* "artanis/config.scm"
> > +                   (("/etc/artanis/artanis.conf")
> > +                    (string-append out "/etc/artanis.conf"))
> > +                   (("/etc/artanis/pages")
> > +                    (string-append out "/etc/artanis")))
> > +                 (substitute* "etc/artanis/artanis.conf"
> > +                   (("/etc/artanis/pages")
> > +                    (string-append out "/etc/artanis")))
>
> Shouldn’t we instead configure it with --sysconfdir=/etc?  That would
> allow users to provide custom configuration files, whereas currently
> OUTPUT/etc/artanis contains immutable configuration files.
>
> WDYT?
>
> Thanks,
> Ludo’.
>
[Message part 2 (text/html, inline)]

This bug report was last modified 3 years and 221 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.