Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 53806 - webapp-config-1.10 config files are installed as virtual and not config-owned
Summary: webapp-config-1.10 config files are installed as virtual and not config-owned
Status: VERIFIED TEST-REQUEST
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Stuart Herbert (RETIRED)
URL:
Whiteboard:
Keywords:
: 54012 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-06-13 06:33 UTC by Sven Wegener
Modified: 2004-06-15 12:11 UTC (History)
1 user (show)

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


Attachments
webapp-config.patch (webapp-config.patch,1.34 KB, patch)
2004-06-13 07:22 UTC, Sven Wegener
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sven Wegener gentoo-dev 2004-06-13 06:33:11 UTC
Stu, I'm going to comment bugs

>>> file (virtual) config.inc.php
Comment 1 Sven Wegener gentoo-dev 2004-06-13 06:38:09 UTC
*grml* sry, about that one. accidently clicked the submit button.

another weird error, when installing mantisbt into /var/www the file
config_inc.php which is marked as a config file in the ebuild will get
installed as virtual and not config-owned.

>>> file (virtual) config_inc.php

same happens for phpmyadmin:

>>> file (virtual) config.inc.php

the old installation from phpmyadmin in /usr/share/webapps worked, after
re-emerging i get all config-files marked as virtual.

the differnce i found is, that now /usr/share/webapps/${PN}/${PVR}/config-files
contains quotes around the filenames.
Comment 2 Sven Wegener gentoo-dev 2004-06-13 07:21:53 UTC
Yep, the quotes are leading to this.

I modified fn_determine_filetype() to print out the values it checks:

"htdocs/config.inc.php" htdocs/config.inc.php
>>> file (virtual) config.inc.php

And these values don't match each other. The attached patch applied to
webapp-config makes webapp-config to correctly mark config-files. I suppose
the same applies to fn_determine_dirtype() as well. Another way would be to
strip off the quotes with some sedfu.
Comment 3 Sven Wegener gentoo-dev 2004-06-13 07:22:41 UTC
Created attachment 33173 [details, diff]
webapp-config.patch

This WFM but stripping the quotes maybe a better alternative.
Comment 4 Stuart Herbert (RETIRED) gentoo-dev 2004-06-13 10:17:47 UTC
The best way to fix this is to stop the webapp.eclass from adding the quotes in the first place, and then adding some IFS trickery when we read those files.

This'll be fixed in -r1.

Making bash scripts work with files containing spaces sure is hard work :(

Best regards,
Stu
Comment 5 Stuart Herbert (RETIRED) gentoo-dev 2004-06-14 16:46:32 UTC
webapp-config 1.10-r1 is now out, and contains a fix for this bug.  Please test.

Best regards,
Stu
Comment 6 Sven Wegener gentoo-dev 2004-06-15 08:05:33 UTC
Works in -r1 as expected.
Comment 7 Stuart Herbert (RETIRED) gentoo-dev 2004-06-15 12:11:34 UTC
*** Bug 54012 has been marked as a duplicate of this bug. ***