Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 499078 - suggestion: set USE="-acl -ads" on net-fs/cifs-utils for minimal installcd iso to drop samba and other deps
Summary: suggestion: set USE="-acl -ads" on net-fs/cifs-utils for minimal installcd is...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Release Media
Classification: Unclassified
Component: InstallCD (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Release Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-23 21:46 UTC by Ben Kohler
Modified: 2014-08-08 14:11 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 Ben Kohler gentoo-dev 2014-01-23 21:46:20 UTC
Currently the minimal installcd iso ships a full copy of samba.  This is the largest package shipped on the whole cd, at over 100MB (uncompressed).  This is pulled in due to USE=acl on cifs-utils.  USE=acl also forces USE=ads which brings keyutils, talloc, and mit-krb5.

I don't really think that acl/ads support for cifs on our minimal iso is important enough to justify the added size.  With "net-fs/cifs-utils -acl -ads" in package.use, the minimal installcd iso shrinks by ~35MB.

If there's some vital use-case for acl/ads on cifs, I'd be interested to hear it =)

Thanks!
Comment 1 Rick Farina (Zero_Chaos) gentoo-dev 2014-01-30 01:23:18 UTC
At the present time, releng uses a stock gentoo profile and I don't think we modify it at all.  I'm fine with changing that, but do we really want to do that?

Personally for the livecd I think it is perfectly safe to do so.
Comment 2 Jorge Manuel B. S. Vicetto (RETIRED) Gentoo Infrastructure gentoo-dev 2014-01-30 12:38:56 UTC
We are already using portage_confdir on the install-cd, but iirc, I added it to be able to workaround some blocks caused by mod-utils / module-init-tools.

$ grep portage_confdir releases/weekly/specs/amd64/installcd-stage*
releases/weekly/specs/amd64/installcd-stage1.spec:portage_confdir: /release/releng/releases/weekly/portage/installcd
releases/weekly/specs/amd64/installcd-stage2-minimal.spec:portage_confdir: /release/releng/releases/weekly/portage/installcd

I've been thinking about this bug for the past 2 or 3 days. I guess we can just use a full-featured confdir and start picking which use flags to use.
I do feel a bit bad about doing it after I poked Patrick so many times about the cifs-utils / samba blocks and he was so helpful about fixing the blocks / use flags deps.

@releng:
opinions?
Comment 3 Rick Farina (Zero_Chaos) gentoo-dev 2014-02-03 15:03:48 UTC
(In reply to Jorge Manuel B. S. Vicetto from comment #2)
> We are already using portage_confdir on the install-cd, but iirc, I added it
> to be able to workaround some blocks caused by mod-utils / module-init-tools.
> 
> $ grep portage_confdir releases/weekly/specs/amd64/installcd-stage*
> releases/weekly/specs/amd64/installcd-stage1.spec:portage_confdir:
> /release/releng/releases/weekly/portage/installcd
> releases/weekly/specs/amd64/installcd-stage2-minimal.spec:portage_confdir:
> /release/releng/releases/weekly/portage/installcd
> 
> I've been thinking about this bug for the past 2 or 3 days. I guess we can
> just use a full-featured confdir and start picking which use flags to use.
> I do feel a bit bad about doing it after I poked Patrick so many times about
> the cifs-utils / samba blocks and he was so helpful about fixing the blocks
> / use flags deps.
> 
Don't feel bad, samba is installable by default now because of us, rather than being entirely broken and making the user change flags to install. Patrick would be proud.

> @releng:
> opinions?

Drop it like it's hot. Use portage_confdir.
Comment 4 Jorge Manuel B. S. Vicetto (RETIRED) Gentoo Infrastructure gentoo-dev 2014-06-22 11:43:44 UTC
11:38 < irker316> proj/releng: jmbsvicetto releases/weekly/portage/installcd/: Signed-off-by: Jorge Manuel B. S. Vicetto (jmbsvicetto) <jmbsvicetto <AT> gentoo.org>
Comment 6 Ben Kohler gentoo-dev 2014-06-25 15:29:55 UTC
> releases/weekly/portage/installcd/package.cifs-utils	[new file with mode: 0644]	

I believe this file was supposed to be put at releases/weekly/portage/installcd/package.use/cifs-utils
Comment 7 Jorge Manuel B. S. Vicetto (RETIRED) Gentoo Infrastructure gentoo-dev 2014-06-25 19:45:04 UTC
Fixed[1]. Thanks for noticing the mistake.

 [1] - http://git.overlays.gentoo.org/gitweb/?p=proj/releng.git;a=commit;h=2546599186f0968cb8fc8cd253731f71bc528053
Comment 8 Ben Kohler gentoo-dev 2014-08-08 14:11:32 UTC
We've got several releases out now with this change, looks good.  Thanks!