[Buildroot] [PATCH 1/1] package/xfsprogs: install in the same location as e2fsprogs

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Thu Jun 9 20:09:05 UTC 2016


Hello,

On Wed, 25 May 2016 23:11:10 +0200, Thomas Petazzoni wrote:

> On Tue, 24 May 2016 06:31:16 -0700, Charles Hardin wrote:
> > The configure script hard codes "/sbin" for installation which
> > installs the XFS tools in a different location then the e2fsprogs.
> > 
> > This is just a cosmetic patch so all the tools end up in "/usr/sbin"
> > for consistency.
> > 
> > Signed-off-by: Charles Hardin <ckhardin at exablox.com>  
> 
> What is the motivation for this change? If the upstream xfsprogs
> developers think it should be installed in /sbin, why should we change
> it?
> 
> In any case, the patch itself needs a description + your Signed-off-by
> line. Also, it should be submitted to the upstream xfsprogs developers
> to see if they might accept it. Otherwise I think we would prefer to
> keep the existing behavior.

Since there was no reply to my questions, I've marked this patch as
"Rejected" in our patch tracking system. Do not hesitate to resend an
updated version that takes into account the comments and that provides
a more detailed justification.

Thanks for your contribution!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the buildroot mailing list