Hello again,

* Szabolcs Szakacsits <[EMAIL PROTECTED]> [2008-05-17 19:49]:
> On Sat, 17 May 2008, Georg Lukas wrote:
> > Actually I did not intend to do so, $(rootsbindir) is automatically set
> > to /usr/sbin somewhere deep in the OpenEmbedded build system.
> /usr/sbin can break mountability in certain circumstances, e.g. if /usr is 
> a separate partition.

That might be true, especially for people having their /usr on NTFS.
However, I think that having a $(rootsbindir) should not do any harm,
when the user knows what she is doing.

> > The DESTDIR is usually set to an empty directory for creating an ipkg
> > package, lets say /tmp/ipk/
> > 
> > My problem was, that the install -d line creates a directory
> > $(DESTDIR)$(rootsbindir), which is /tmp/ipk/usr/sbin then.
> > After that, a symlink to /sbin/ntfs-3g is created in /tmp/ipk/sbin, but
> > /tmp/ipk/sbin does not exist. 
> I see. So we should create that directory too if it's missing.

We should only create the directory we are using. In current stable, we
create $(rootsbindir) and use /sbin. Therefore I submitted the patch to
use $(rootsbindir) instead of hard coded /sbin, making the install
process self-consistent.

If you think that it is bad to give the user (or rather package
maintainer) the choice of overriding that directory, you should hard
code both lines to /sbin and not use the $(rootsbindir) variable at all.

> The below should work:
> 
>       ./configure --sbindir=/sbin 

What is the point in having a config variable with only one working
value?


Kind regards,

Georg
-- 
|| http://op-co.de ++  GCS/CM d? s: a-- C+++ UL+++ !P L+++ E--- W++  ++
|| gpg: 0x962FD2DE ||  N++ o? K- w---() O M V? PS+ PE-- Y+ PGP++ t*  ||
|| Ge0rG: euIRCnet ||  5 X+ R tv b+(+++) DI+(+++) D+ G e* h! r* !y+  ||
++ IRCnet OFTC OPN ||________________________________________________||

Attachment: signature.asc
Description: Digital signature

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
ntfs-3g-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ntfs-3g-devel

Reply via email to