Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 590388 - webapp-config: Paludis-support broken in many ways
Summary: webapp-config: Paludis-support broken in many ways
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Web Application Packages Maintainers
URL:
Whiteboard:
Keywords:
: 653698 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-08-03 08:39 UTC by Tiziano Müller (RETIRED)
Modified: 2023-01-28 20:30 UTC (History)
3 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tiziano Müller (RETIRED) gentoo-dev 2016-08-03 08:39:40 UTC
The current support for paludis in webapp-config is broken in many ways.
The most obvious breakage is due to the usage of the deprecated popen2/3 calls which have been removed in Python 3. After fixing them and have the commands raise proper exceptions, it becomes obvious that the current support for Paludis using Python 2 works only because errors are silently ignored.

I have a patchset here which fixes most of the issues:

  https://github.com/dev-zero/webapp-config/compare/paludis-support

There is a question about the webapp-db: should the directory of the package have the category or not? Guessing from some comments in the code it should, so, it should be /var/db/webapps/www-apps/phabricator/9999/ instead of just /var/db/webapps/phabricator/9999/ , right?

Btw, shouldn't there be a `webapp-config` component in the `Gentoo Hosted Projects` product in our bugzilla?
Comment 1 Anthony Basile gentoo-dev 2016-08-03 13:32:09 UTC
I assume you've tested on a system with paludis, but have you tested your changes on a system with portage?  I don't want to introduce any regressions.
Comment 2 Arfrever Frehtes Taifersar Arahesis 2018-04-22 18:57:41 UTC
*** Bug 653698 has been marked as a duplicate of this bug. ***