Re: [gentoo-portage-dev] rewriting the ldconfig logic

2006-04-04 Thread Zac Medico
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Mike Frysinger wrote:
 On Saturday 01 April 2006 17:17, Zac Medico wrote:
 Mike Frysinger wrote:
 i thought about this some more ... why do we even care about the mtimes
 during package merge ?  if a package doesnt install any files into a
 libdir, why should we bother running ldconfig ?  if a user updated the
 dirs, then they can either run ldconfig or env-update ...

 so imo, the logic should be:
 if srcroot is None:
 mtime checks
 else:
 libdir existence checks
 Yeah, I think that's correct.  Also, even though we don't need to check the
 mtimes when srcroot is defined, we can still update the mtimes in the
 mtimedb to the current values in order to avoid an unnecessary ldconfig run
 on the next env_update call.
 
 feel like implementing this ?  that way we dont stop on each others feet 
 again 
 and i dont have to waste time when you rewrite my work ;)
 -mike

While investigating this I've found that srcroot will actually be emptied of 
all files in some cases.  For example, this happens when PORTAGE_TMPDIR is on 
the same root filesystem that is being merged to, and the files are merged via 
os.rename.  In these cases, srcroot becomes useless (passing in the CONTENTS 
would do the trick though).  Because of this, the srcroot parameter to 
env_update should be removed.

Zac

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.2.2 (GNU/Linux)

iD8DBQFEMxxD/ejvha5XGaMRAih0AJ9cIl6peHsEN2/MmYQeRpNOvAEQXwCfVMhz
QgJko+J7xgTgfXzzgGIYNsE=
=EdVn
-END PGP SIGNATURE-
-- 
gentoo-portage-dev@gentoo.org mailing list



Re: [gentoo-portage-dev] rewriting the ldconfig logic

2006-04-01 Thread Mike Frysinger
On Saturday 01 April 2006 17:17, Zac Medico wrote:
 Mike Frysinger wrote:
  i thought about this some more ... why do we even care about the mtimes
  during package merge ?  if a package doesnt install any files into a
  libdir, why should we bother running ldconfig ?  if a user updated the
  dirs, then they can either run ldconfig or env-update ...
 
  so imo, the logic should be:
  if srcroot is None:
  mtime checks
  else:
  libdir existence checks

 Yeah, I think that's correct.  Also, even though we don't need to check the
 mtimes when srcroot is defined, we can still update the mtimes in the
 mtimedb to the current values in order to avoid an unnecessary ldconfig run
 on the next env_update call.

feel like implementing this ?  that way we dont stop on each others feet again 
and i dont have to waste time when you rewrite my work ;)
-mike
-- 
gentoo-portage-dev@gentoo.org mailing list