sysconfdir not respected/implemented

[sagetrac-felixs]

Configuration files should be placed into 'sysconfdir' [1]. Files such as gprc.expect in $(SAGE_LOCAL)/etc or default.qepcadrc in $(SAGE_LOCAL) will break future sagelib packaging.
SAGE_LOCAL will supposedly be set to 'prefix' for Sage-the-library packages.

It might make sense to add SAGE_ETC=sysconfdir to the environment (and use that whereever appropriate). sysconfdir defaults to prefix/etc (=SAGE_LOCAL/etc) but can be overridden for custom installations.

[1] www.gnu.org/prep/standards/html_node/Directory-Variables.html‎

Component: distribution

Keywords: sysconfdir, sage-the-library

Author: Felix Salfelder

Branch/Commit: u/felixs/14750 @ 573b23e

Reviewer: Nathann Cohen

Issue created by migration from https://trac.sagemath.org/ticket/14750

[nexttime]

Replying to @sagetrac-felixs:

It might make sense to add SAGE_ETC=sysconfdir to the environment (and use that whereever appropriate).

Go ahead (letting SAGE_ETC default to $SAGE_LOCAL/etc)...

(It's a bit funny to say Sage doesn't repect that Makefile variable when it doesn't even have a configure; Sage's top-level Makefile certainly doesn't follow any of those conventions, and the Sage library currently doesn't use make at all.)

[nexttime]

comment:2

P.S.: Other config files end up below $DOT_SAGE (which defaults to $HOME/.sage/).

[sagetrac-felixs]

comment:3

Replying to @nexttime:

(It's a bit funny to say Sage doesn't repect that Makefile variable when it doesn't even have a configure; Sage's top-level Makefile certainly doesn't follow any of those conventions, and the Sage library currently doesn't use make at all.)

My copy of Sage has a configure. I must have repressed the previous state. ;)

Replying to @nexttime:

P.S.: Other config files end up below $DOT_SAGE (which defaults to $HOME/.sage/).

While I'm at it: How about use something like the following to access config files in general?

def configfile(name):
  if os.exists(HOME + "/.sage/" + name)
    return open(HOME + "/.sage/" + name)
  return open("SAGE_ETC"+name)

This might increase readability and enable overrides.

[sagetrac-felixs]

Commit: 0f74370

[sagetrac-felixs]

comment:5

Here's a patch...

[nathanncohen]

Changed commit from 0f74370 to none

[nathanncohen]

comment:7

Could we have a branch name too ? :-P

Nathann

[sagetrac-felixs]

comment:8

Replying to @nathanncohen:

Could we have a branch name too ? :-P

yes.

[sagetrac-felixs]

Commit: 573b23e

[sagetrac-felixs]

Branch: u/felixs/14750

[nathanncohen]

Reviewer: Nathann Cohen

[nathanncohen]

comment:9

Goooooooood to go !

Nathann