552424 – app-emulation/xen: install /boot/xen-syms optionally

Bug 552424 - app-emulation/xen: install /boot/xen-syms optionally

Summary: app-emulation/xen: install /boot/xen-syms optionally

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Gentoo Xen Devs

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2015-06-17 23:53 UTC by Jonas Jelten
Modified:	2015-12-27 01:23 UTC (History)
CC List:	1 user (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Jonas Jelten 2015-06-17 23:53:48 UTC

Please make the install of /boot/xen-syms-$version optional, best via a useflag. It is pretty large compared to the other contents of /boot, currently mine has 16M.

On my device space on /boot is limited, and I don't want that file to be created on every xen installation. Making this file optional should satisfy everyone and save space on /boot.

Reproducible: Always

Comment 1 Ian Delaney (RETIRED) gentoo-dev

2015-06-25 02:34:25 UTC

Let's let dlan comment on the merits here too.

Comment 2 Yixun Lan archtester

2015-06-25 09:26:18 UTC

I think xen-syms is useful for debugging purpose, so how about make it controlled under USE=debug?

or another idea, handle this from user side by using INSTALL_MASK, see "man 5 make.conf"

Comment 3 Yixun Lan archtester

2015-06-25 09:28:04 UTC

by handling from user side, I mean

# cat /etc/portage/env/xen.conf 
INSTALL_MASK=/boot/xen-syms*
# cat /etc/portage/package.env/general 
app-emulation/xen xen.conf

Comment 4 Doug Goldstein (RETIRED) gentoo-dev

2015-12-06 05:36:51 UTC

(In reply to Yixun Lan from comment #3)
> by handling from user side, I mean
> 
> # cat /etc/portage/env/xen.conf 
> INSTALL_MASK=/boot/xen-syms*
> # cat /etc/portage/package.env/general 
> app-emulation/xen xen.conf

This is a much better solution. That's exactly what we have INSTALL_MASK for.

Comment 5 Ian Delaney (RETIRED) gentoo-dev

2015-12-13 09:22:12 UTC

This means no adjustment to ebuilds of xen, rather a prompt to user via elog via pkg_postinst(). If no objection to this I will commit in a week or so.  The option of use of INSTALL_MASK effectively provides a fix which need be effected manually by a user post install.

Comment 6 Ian Delaney (RETIRED) gentoo-dev

2015-12-27 01:23:48 UTC

commit 4e6cfc899a24f2d6a6ed9af72abede0973f87053
Author: Ian Delaney <idella4@gentoo.org>
Date:   Sat Dec 26 15:06:33 2015 +0800

    app-emulation/xen: add msg in pkg_postinst to all vns. 4.5 4.5 re xen-syms
    
    Prompt from user in bug #552424 to make install of /boot/xen-syms optional,
    done by informing user via msg in pkg_postinst to do this using INSTALL_MASK
    
    Gentoo bug: #552424