All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: linux-hotplug@vger.kernel.org
Subject: Re: [patch] automatically install correct initscript
Date: Wed, 03 Mar 2004 19:36:32 +0000	[thread overview]
Message-ID: <20040303193630.GA28198@kroah.com> (raw)
In-Reply-To: <1078188416.10733.44.camel@localhost>

On Wed, Mar 03, 2004 at 01:21:34PM -0600, Chris Larson wrote:
> * Greg KH (greg@kroah.com) wrote:
> > On Wed, Mar 03, 2004 at 12:46:08PM -0600, Chris Larson wrote:
> > > 	Speaking as a package and distribution maintainer and all around CM and
> > > build guy, there are two problems I feel the need to point out.  First,
> > > you're installing into $(etcdir) directly.  Bad, Bad, Bad.  -always-
> > > obey DESTDIR.  -always-.  Second, the distro I'm installing into isn't
> > > necessarily the distro I'm building on.  Any opposition to making the
> > > tests obey DESTDIR as well?
> > 
> > I already made this change in the 021 release.
> 
> Oh good, glad to hear it.
> 
> > As for the cross-distro install issue, do you have any suggestions?
> 
> Well, how about something as simple as using a makefile variable that
> contains the name of the initscript to be installed, so that it can be
> easily overridden by those building it?

Sounds good to me, care to write a patch?  :)

thanks,

greg k-h


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id\x1470&alloc_id638&op=click
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

      parent reply	other threads:[~2004-03-03 19:36 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-03-02  0:46 [patch] automatically install correct initscript Robert Love
2004-03-02  1:04 ` Kay Sievers
2004-03-02  1:08 ` Robert Love
2004-03-02  1:14 ` Kay Sievers
2004-03-02  1:23 ` Robert Love
2004-03-02 20:28 ` Greg KH
2004-03-03 18:46 ` Chris Larson
2004-03-03 19:10 ` Greg KH
2004-03-03 19:21 ` Chris Larson
2004-03-03 19:36 ` Greg KH [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20040303193630.GA28198@kroah.com \
    --to=greg@kroah.com \
    --cc=linux-hotplug@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.