On Tue, 13 Jul 2010, Michael Jennings wrote:

On Tuesday, 13 July 2010, at 16:18:53 (-0300),
Iv?n Briano (Sachiel) wrote:

And we really want that? Does any other program do things like this?

No.  Other programs properly reference $sysconfdir.

 pamdir = /etc/pam.d/
 pam_DATA = elsa

 confdir = /etc
 conf_DATA = elsa.conf

Hard-coding /etc will make it impossible to do proper safe building of
packages as a non-root user.  Always use $sysconfdir or @sysconf...@.

like that ? :

Index: data/Makefile.am
===================================================================
--- data/Makefile.am    (revision 50230)
+++ data/Makefile.am    (working copy)
@@ -3,10 +3,10 @@

 SUBDIRS = themes

-pamdir = /etc/pam.d/
+pamdir = $(sysconfdir)/pam.d/
 pam_DATA = elsa

-confdir = /etc
+confdir = $(sysconfdir)
 conf_DATA = elsa.conf

 EXTRA_DIST = elsa elsa.conf
------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to