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

Bug 434914

Summary: app-admin/webapp-config: Manage recursive server-owned dirs independently
Product: Gentoo Linux Reporter: Vincent Brillault <gentoo>
Component: Current packagesAssignee: Gentoo Web Application Packages Maintainers <web-apps>
Status: UNCONFIRMED ---    
Severity: enhancement CC: web-apps
Priority: Normal Keywords: PATCH
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 434916    
Attachments: webapp-config patch: use a third configuration file
webapp.eclass patch: put the recursive server own dirs in a new file
webapp.eclass patch: put the recursive server own dirs in a new file
Put the recursive server own dir references in a new file

Description Vincent Brillault 2012-09-13 16:04:51 UTC
Created attachment 323676 [details, diff]
webapp-config patch: use a third configuration file

The "server-owned" files are currently marked individually. This makes the eclass use a 'find' to check all them and then create a big list of thoses. This patch proposes to use a third configuration file, 'server-owned-dirs', which directly contains the folders that need to be recursively server-owned.

This enhancement also needs a modification of the eclass but the new files are retrocompatible with the previous one (it still uses them)
Comment 1 Vincent Brillault 2012-09-13 16:05:47 UTC
Created attachment 323678 [details, diff]
webapp.eclass patch: put the recursive server own dirs in a new file
Comment 2 Vincent Brillault 2012-09-13 16:10:45 UTC
Created attachment 323680 [details, diff]
webapp.eclass patch: put the recursive server own dirs in a new file

Remove patch part that wasn't supposed to be here
Comment 3 Vincent Brillault 2012-11-01 08:18:58 UTC
Created attachment 327914 [details, diff]
Put the recursive server own dir references in a new file

Eclass patch update: check for directories instead of files

I made a mistake when I tried to factorize the helper function. This patch is a non-factorized version. I don't know which one is the more readable.