Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 93210 - fluxbox-styles-fluxmod-20050128 arch testing
Summary: fluxbox-styles-fluxmod-20050128 arch testing
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Other
: High normal
Assignee: Jonathan Smith (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-19 06:37 UTC by Jonathan Smith (RETIRED)
Modified: 2005-07-21 13:07 UTC (History)
3 users (show)

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


Attachments
patch made to the ebuild to get sed quiet (fluxbox-styles-fluxmod-20050128.ebuild.patch,1.31 KB, patch)
2005-07-12 13:46 UTC, Fabian Groffen
Details | Diff
a platform independent fix for the sed issue (fluxbox-styles-fluxmod-20050128.ebuild.patch2,1.66 KB, patch)
2005-07-17 12:19 UTC, Fabian Groffen
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Smith (RETIRED) gentoo-dev 2005-05-19 06:37:15 UTC
aplha, mips, amd64, hppa please mark as stable

osx, please test and ~arch if appropiate

thanks
Comment 1 René Nussbaumer (RETIRED) gentoo-dev 2005-05-19 11:17:17 UTC
stable on hppa
Comment 2 Jan Brinkmann (RETIRED) gentoo-dev 2005-05-19 14:32:14 UTC
stable on amd64
Comment 3 Fernando J. Pereda (RETIRED) gentoo-dev 2005-06-14 03:34:19 UTC
We came, we tested, we alpha'd

Cheers,
Ferdy
Comment 4 Jonathan Smith (RETIRED) gentoo-dev 2005-06-16 09:34:35 UTC
closing due to timeout... thanks
Comment 5 Fabian Groffen gentoo-dev 2005-07-12 13:46:00 UTC
Created attachment 63271 [details, diff]
patch made to the ebuild to get sed quiet

BSD sed on OSX is not able to do a case insensitive match.  I solved it using
some other script-foo, which seems to have the same effect on OSX.
Comment 6 Lina Pezzella (RETIRED) gentoo-dev 2005-07-17 11:10:29 UTC
This shouldn't be a problem. On OSX we have "gsed" which is automatically aliased to "sed" by portage. 
This should've been installed on both Panther and Tiger as a part of system.
Comment 7 Fabian Groffen gentoo-dev 2005-07-17 12:19:08 UTC
Created attachment 63635 [details, diff]
a platform independent fix for the sed issue

It seemed that on OSX by default "gsed" is used upon invocation of "sed".  When
being called through another process, such as find or xargs, this alias is not
being used resulting in the reported errors by sed.

This proposed patch makes sure the ebuild calls sed directly by storing find's
output first in a variable.
Comment 8 Fabian Groffen gentoo-dev 2005-07-21 13:07:48 UTC
Comment on attachment 63635 [details, diff]
a platform independent fix for the sed issue

no patch should be necessary to run this ebuild on OSX.  The issue why sed
causes trouble on OSX is because there sed is actually an alias to gsed, and
for aliases you need a shell.  The fork done by find to execute sed does not
spawn a shell first (why should it) and as such the normal BSD sed is executed.
 To solve this	for OSX, a symlink to gsed as sed should be placed somewhere
and put as first in the PATH environment, such that the exec() call after the
fork() in find, finds the 'right' (g)sed due to the symlink and the PATH.  I've
tested this setup and that works find with the original ebuild, hence I
obsolete this patch.