Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 128960 - euse from gentoolkit uses readlink -f
Summary: euse from gentoolkit uses readlink -f
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Tools (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Portage Tools Team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2006-04-05 13:54 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2007-01-08 21:17 UTC (History)
1 user (show)

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


Attachments
gentoolkit-0.2.2_pre4-realpath.patch (gentoolkit-0.2.2_pre4-realpath.patch,1.42 KB, patch)
2006-04-19 16:58 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 2006-04-05 13:54:04 UTC
Currently the euse command uses the "readlink -f" command that does not work on non-GNU systems.

An alternative is to use app-admin/realpath as done by eselect which works on other systems too.
Comment 1 solar (RETIRED) gentoo-dev 2006-04-05 14:05:28 UTC
readlink -f rocks++; Please don't add new deps to gentoolkit unless we 
are actually one of these systems that lacks gnu-coreutils 
Comment 2 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-04-05 14:10:53 UTC
I suppose it can be a || ( app-admin/realpath sys-apps/coreutils ) dep like for eselect and selected the right one either at runtime or at installation time.
Comment 3 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-04-19 16:58:46 UTC
Created attachment 84988 [details, diff]
gentoolkit-0.2.2_pre4-realpath.patch

Proposed patch that might work as intended.

Also, the dependency can add sys-freebsd/freebsd-bin to the list as that package provide a /bin/realpath already.
Comment 4 Benigno B. Junior (RETIRED) gentoo-dev 2006-04-19 18:08:53 UTC
Hmm, it just made me remember the portage's quickpkg, which has the same issue for us:
quickpkg:           x=$(readlink -f $x)
Comment 5 Paul Varner (RETIRED) gentoo-dev 2006-07-31 20:55:58 UTC
Fixed in subversion revision 307
Comment 6 Paul Varner (RETIRED) gentoo-dev 2007-01-08 21:17:19 UTC
This was fixed with the release of gentoolkit-0.2.3_pre2.