Lyrics / lyrics-api

Dynamic website for the lyrics database
GNU Affero General Public License v3.0
2 stars 0 forks source link

Fastcgi XSLT_DIR #8

Open C0rn3j opened 4 years ago

C0rn3j commented 4 years ago

Shouldn't this point to

fastcgi_param XSLT_DIR "/var/www/html/lyrics-api/dist/format";

Instead of

fastcgi_param XSLT_DIR "/var/www/html/lyrics-api/format";

? Since the install commands generated it in ../dist/format

defanor commented 4 years ago

It indeed should point to dist/format when setting it up with git clone (I forgot to edit it on the dist directory introduction). While when setting with make install, it's not necessary to set XSLT_DIR explicitly, unless one wants to extend/edit it in a different place, and it would be something like /usr/local/share/perl5/auto/share/dist/LyricsDB/format/ by default.

The example nginx configuration was added before there was Makefile.PL, but now it can be simplified, removing that environment variable altogether (assuming installation with make install). Although setting with plain git clone may still be desirable.

Feel free to edit it, perhaps documenting what you use and see as a more convenient way.