From: "Anthony G. Basile" <basile@opensource.dyc.edu>
To: gentoo-catalyst@lists.gentoo.org
Subject: Re: [gentoo-catalyst] [PATCH v2] stage1-controller.sh: Remove some old poor cleaning code
Date: Wed, 10 Sep 2014 17:00:15 -0400 [thread overview]
Message-ID: <5410BBDF.8050408@opensource.dyc.edu> (raw)
In-Reply-To: <5410876F.8000505@gentoo.org>
On 09/10/14 13:16, Rick "Zero_Chaos" Farina wrote:
> On 09/09/2014 02:38 PM, Brian Dolbec wrote:
>> On Mon, 1 Sep 2014 22:05:45 -0700
>> Brian Dolbec <dolsen@gentoo.org> wrote:
>>
>>> This code had portage bin path hard coded. That path needed to be
>>> changed for a new portage ebuild and install system.
>>> After testing the origianl code and comparing it with some updated
>>> code supplied by Douglas Freed. It turned out both code chunks
>>> resulted in nothing being cleaned.
>>>
>>> Tested and confirmed by zero_chaos.
>>
>> I have gone over things more and tested the new find command. It does
>> work on my host system. However, the question remains... DOES this
>> particular cleaning operation NEED to be performed?
>>
>> With current the tree snapshot for my testing 20140829. It does not
>> find anything to clean. BUT, will that remain the same in the future
>> as pkgs are bumped?
>>
>> For safety, I'd be inclined to keep the find command (v1 of the
>> patch) and clean any it does find just in case.
>>
>
> If we truly need to remove these files, I don't think catalyst was ever
> the place to do this. We have USE=static and USE=static-libs for a
> reason. Randomly removing files from the filesystem was a hack then,
> and if we are cleaning this up let's just remove it. If I want to build
> things with USE=static or USE=static-libs then catalyst shouldn't be
> pointlessly crippling my builds.
>
> v2 means one less horrible hack in catalyst, and one at a time is the
> only way will remove all the horror.
>
> -Zero
>
I just did some testing here. You have to be very careful not to remove
the wrong .a's else you break the toolchain. So I agree, remove the
.a's in the ebuilds switched on USE=statlic-libs, and not in catalyst.
The biggest .a's in stage1 are libpythonXX.a which are 3 to 4 MB, but
the others are like a few hundred K's at most. The gain is a smaller
stage, but the downside is headaches making sure we don't break something.
So, to slim down stage1, see if you can get USE=static-libs in python's
ebuilds and build USE=-static-libs. That itself will reduce that stage
by about 8 MBs which is more than all the "expendable" .a's put together.
--
Anthony G. Basile, Ph. D.
Chair of Information Technology
D'Youville College
Buffalo, NY 14201
(716) 829-8197
next prev parent reply other threads:[~2014-09-10 20:57 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-02 2:31 [gentoo-catalyst] [PATCH 0/7] Pending branch patches (various) Brian Dolbec
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 1/7] Remove unused urllib import Brian Dolbec
2014-09-02 5:22 ` W. Trevor King
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 2/7] Remove unused variable new and an undefined variable s Brian Dolbec
2014-09-02 5:43 ` W. Trevor King
2014-09-05 18:16 ` W. Trevor King
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 3/7] stage1-controller.sh: Fix portage bin path hard coding Brian Dolbec
2014-09-02 3:35 ` Rick "Zero_Chaos" Farina
2014-09-02 5:05 ` [gentoo-catalyst] [PATCH v2] stage1-controller.sh: Remove some old poor cleaning code Brian Dolbec
2014-09-02 5:19 ` [gentoo-catalyst] [PATCH v2 3/7] " Brian Dolbec
2014-09-02 5:31 ` [gentoo-catalyst] [PATCH v2] " W. Trevor King
2014-09-09 18:38 ` Brian Dolbec
2014-09-10 17:16 ` Rick "Zero_Chaos" Farina
2014-09-10 21:00 ` Anthony G. Basile [this message]
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 4/7] chroot-functions.sh: Remove --nodeps option from portage update Brian Dolbec
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 5/7] chroot-functions.sh: Fix a mis-worded comment Brian Dolbec
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 6/7] setup_pkgmgr(): WIP Make the 'build' use flag optional Brian Dolbec
2014-09-09 18:27 ` Brian Dolbec
2014-09-02 2:31 ` [gentoo-catalyst] [PATCH 7/7] Fix a relative path bug Brian Dolbec
2014-09-02 5:06 ` W. Trevor King
2014-09-02 5:26 ` Brian Dolbec
2014-09-09 17:20 ` Brian Dolbec
2014-09-11 3:30 ` [gentoo-catalyst] [PATCH 0/7] Pending branch patches (various) Brian Dolbec
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5410BBDF.8050408@opensource.dyc.edu \
--to=basile@opensource.dyc.edu \
--cc=gentoo-catalyst@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox