public inbox for gentoo-catalyst@lists.gentoo.org
 help / color / mirror / Atom feed
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


  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