Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 93646 - env-update to support FreeBSD's ldconfig
Summary: env-update to support FreeBSD's ldconfig
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Core (show other bugs)
Hardware: All FreeBSD
: High major (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2005-05-23 00:30 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2005-07-14 06:58 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
portage-2.0.51.22-ldconfig-fbsd.patch (portage-2.0.51.22-ldconfig-fbsd.patch,1.71 KB, patch)
2005-05-23 00:31 UTC, Diego Elio Pettenò (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Elio Pettenò (RETIRED) gentoo-dev 2005-05-23 00:30:41 UTC
ldconfig in FreeBSD systems has a complete different syntax, which isn't supported by current portage.env_update function.

The attached patch to portage make it recognize FreeBSD systems and uses the right syntax to regenerate the hints file, fixing its behaviour for G/FBSD.

Thanks in advance, Diego
Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-05-23 00:31:01 UTC
Created attachment 59609 [details, diff]
portage-2.0.51.22-ldconfig-fbsd.patch
Comment 2 Brian Harring (RETIRED) gentoo-dev 2005-05-25 00:06:14 UTC
Your change to default behaviour, I'd wonder about how it behaves with osx...
Comment 3 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-05-25 03:41:55 UTC
OSX has no /sbin/ldconfig at all... 
 
Comment 4 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-05-25 10:42:28 UTC
It breaks with ${ROOT} != "/" with that patch 
 
Changing the command to that: 
 
commands.getstatusoutput("cd / ; /sbin/ldconfig -elf 
"+str(root)+"etc/ld.so.conf -f "+str(root)+"var/run/ld-elf.so.hints") 
 
should fix. 
Comment 5 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-05-25 16:23:15 UTC
Sorry the correct one is 
 
commands.getstatusoutput("cd / ; /sbin/ldconfig -elf -f 
"+str(root)+"var/run/ld-elf.so.hints"+str(root)+"etc/ld.so.conf") 
 
Comment 6 Jason Stubbs (RETIRED) gentoo-dev 2005-05-28 00:30:18 UTC
Just to confirm, should that be: 
 
commands.getstatusoutput("cd / ; /sbin/ldconfig -elf -f  
"+str(root)+"var/run/ld-elf.so.hints "+str(root)+"etc/ld.so.conf") 
 
? 
 
There appears to be a space missing after ld-elf.so.hints. 
Comment 7 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-05-28 01:49:37 UTC
Yes sorry, the space is needed :) 
 
Comment 8 Jason Stubbs (RETIRED) gentoo-dev 2005-05-28 03:43:06 UTC
Okay.. InCVS  
Comment 9 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-06-19 17:21:55 UTC
Can you please add a -i option right after "ldconfig" ? Is needed when using 
ROOT= because without it, if the directories doesn't exists, it fails. 
 
Comment 10 Jason Stubbs (RETIRED) gentoo-dev 2005-07-14 05:47:30 UTC
Fixed on or before 2.0.51.22-r1 
Comment 11 Jason Stubbs (RETIRED) gentoo-dev 2005-07-14 06:58:20 UTC
Looking through the batch of bugs, I'm not sure that some of these are 
actually fixed in stable. Others, the requirements have possibly changed after 
the initial fix was committed. 
 
If you think this bug has been closed incorrectly, please reopen or ask that 
it be reopened.