seichejs / sylverant

Automatically exported from code.google.com/p/sylverant
1 stars 1 forks source link

Don't hard-code "/usr/local/share/sylverant" path #10

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I've noticed that in libsylverant/include/sylverant/config.h, 
SYLVERANT_DIRECTORY is hard-coded to /usr/local/share/sylverant, and the 
various executables use this to look up their configuration files.  This makes 
it impossible to install the server in a directory other than /usr/local .  
Aside from being annoying to people who want to install 3rd-party programs in a 
different directory  (/opt, for example), this also means it's impossible to 
install and run the server on a computer where you don't have root access.

Instead, it would probably make sense to have the autotools build process 
define SYLVERANT_DIRECTORY as "$(datarootdir)/sylverant".  That should preserve 
the current behavior for users who don't specify a directory but also work for 
users who specify different directories by passing --prefix or --datarootdir to 
./configure .

Original issue reported on code.google.com by minne...@gmail.com on 10 Oct 2010 at 4:11

GoogleCodeExporter commented 9 years ago
I've been meaning to do that for a long while actually, and just never got 
around to it. In all honesty, its mostly due to the fact that my experience 
with autotools outside of this project is almost 0. I'll add this to my list of 
stuff to do soon.

Original comment by bluecrab on 11 Oct 2010 at 12:29

GoogleCodeExporter commented 9 years ago
Resolved in r245.

Original comment by bluecrab on 25 Oct 2010 at 1:55