Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 540108 - net-firewall/firewalld-0.3.10 should install man pages
Summary: net-firewall/firewalld-0.3.10 should install man pages
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Matthias Maier
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-15 04:34 UTC by dE
Modified: 2015-05-25 20:59 UTC (History)
2 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 dE 2015-02-15 04:34:43 UTC
But the older packages do.

equery files verifies my concern.

Reproducible: Always
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2015-02-15 07:58:10 UTC
I guess with the "manual" approach it's easy to miss a few bits.

--- firewalld-0.2.12.ebuild
+++ firewalld-0.3.10.ebuild

 src_install() {
-       python_foreach_impl \
-               emake DESTDIR="${ED}" pythondir="$(python_get_sitedir)" install
+       # manually split up the installation to avoid "file already exists" errors
+       emake -C config DESTDIR="${ED}" install
+       emake -C po DESTDIR="${ED}" install
+       emake -C shell-completion DESTDIR="${ED}" install
+
+       install_python() {
+               emake -C src DESTDIR="${ED}" pythondir="$(python_get_sitedir)" install
+               python_optimize
+       }
+       python_foreach_impl install_python
+
+       python_replicate_script "${ED}"/usr/bin/firewall-{offline-cmd,cmd,applet,config}
+       python_replicate_script "${ED}/usr/sbin/firewalld"

Handling of doc/ is missing there.
Comment 2 Matthias Maier gentoo-dev 2015-05-25 20:59:42 UTC
This is now fixed. Thanks for the bug report.

*firewalld-0.3.13 (25 May 2015)

  25 May 2015; Matthias Maier <tamiko@gentoo.org> +firewalld-0.3.13.ebuild,
  firewalld-0.3.10.ebuild, metadata.xml:
  co-maintain package; version bump; also install man pages, bug #540108
  reported by 'dE', also thanks to Jeroen Roovers