Bug 195771 – webapp-config patch to make it work with prefixed portage

• Status: VERIFIED LATER
• Product: Gentoo/Alt

[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) 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) 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 2008-02-25 21:39:08 UTC]

sorry .... :(

[Comment 10 Jeremy Olexa (darkside) (RETIRED) 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) 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 2008-05-20 15:11:26 UTC]

hmmm, sorry, that was not my intention