Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 195771

Summary: webapp-config patch to make it work with prefixed portage
Product: Gentoo/Alt Reporter: Dirk Tilger <dirk>
Component: Prefix SupportAssignee: Gentoo non-Linux Team <alt>
Status: VERIFIED LATER    
Severity: enhancement CC: dirk, lamotte85
Priority: High    
Version: unspecified   
Hardware: x86   
OS: OS X   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 195780, 206973    
Attachments: Patch to webapp-config 1.50.16-r2 ebuild to make it work with prefixed portage
Patch to webapp-config 1.50.16 itself to work with the prefixed portage
Updated patch to webapp-config 1.50.16 itself to work with the prefixed portage
Updated patch to webapp-config 1.50.16-r2 ebuild to make it work with prefixed portage
webapp.eclass patch to make it work in prefixed portage

Description Dirk Tilger 2007-10-13 21:10:17 UTC
USE flags for testing: /

objective: `emerge mapserver`

test system: x86-macos
Comment 1 Dirk Tilger 2007-10-13 21:11:21 UTC
Created attachment 133383 [details, diff]
Patch to webapp-config 1.50.16-r2 ebuild to make it work with prefixed portage

This ebuild needs a file in $FILES_DIR to be present which will be attached separately.
Comment 2 Dirk Tilger 2007-10-13 21:20:54 UTC
Created attachment 133384 [details, diff]
Patch to webapp-config 1.50.16 itself to work with the prefixed portage
Comment 3 Dirk Tilger 2007-10-13 21:26:17 UTC
There was a notification I got regarding the sandbo. I got it when adding new applications to my apache2, but I'm unable to reproduce it now. It maybe that something in webapp-config needs further tuning.
Comment 4 Elias Pipping (RETIRED) gentoo-dev 2007-10-13 23:53:24 UTC
webapp-config-1.50.16-prefixportage.patch:

while os.environ['EPREFIX'] might work, the proper way to do this is @GENTOO_PORTAGE_EPREFIX@ in the patch and eprefixify in the ebuild.
Comment 5 Dirk Tilger 2007-10-20 18:12:21 UTC
Created attachment 133975 [details, diff]
Updated patch to webapp-config 1.50.16 itself to work with the prefixed portage

- changed to use @GENTOO_PORTAGE_EPREFIX@
- appended patch for webapp-cleaner

Since the sandbox hasn't been ported to Darwin, I can't test how the sandbox related cod works.
Comment 6 Dirk Tilger 2007-10-20 18:16:53 UTC
Created attachment 133976 [details, diff]
Updated patch to webapp-config 1.50.16-r2 ebuild to make it work with prefixed portage
Comment 7 Dirk Tilger 2007-10-20 18:24:54 UTC
Created attachment 133977 [details, diff]
webapp.eclass patch to make it work in prefixed portage

webapp-config related apps rely on an eclass that I forgot to include in this bug to make the whole thing work. 

Some comments for further improver:
webapp-config is called from the command line and the eclass is called and from ebuilds. Both use a common configuration file that has the prefix already included. For that reason the functions called from the ebuilds that rely on functions like `dodir` need some special care, as for those the path needs to be altered. It was further complicated by eclass internal variables (MY_*) that are defined once and then used through successive calls during the webapp ebuilds.
Comment 8 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2008-02-25 21:38:13 UTC
And now the webapp.eclass got updated so changes worked on by Dirk may be outdated. lovely..
Comment 9 Fabian Groffen gentoo-dev 2008-02-25 21:39:08 UTC
sorry .... :(
Comment 10 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2008-05-12 18:58:13 UTC
Dirk, 
I have self motivation for getting trac into the prefix tree and that depends on webapp-config. Since webapp.eclass has been changed since your bug and comment #8, someone will have to look at it later. Thanks for your changes, if you have anything newer, please re-open this bug and I will look at it. 
Comment 11 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2008-05-20 15:08:49 UTC
webapp-config is in the tree for now. I used your patch in conjunction with a patch from bug 206973, both had about the same content. Thanks!
Comment 12 Fabian Groffen gentoo-dev 2008-05-20 15:11:26 UTC
hmmm, sorry, that was not my intention