From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from lists.gentoo.org ([140.105.134.102] helo=robin.gentoo.org) by nuthatch.gentoo.org with esmtp (Exim 4.43) id 1E3gUE-0003w4-8T for garchives@archives.gentoo.org; Fri, 12 Aug 2005 20:50:42 +0000 Received: from robin.gentoo.org (localhost [127.0.0.1]) by robin.gentoo.org (8.13.4/8.13.4) with SMTP id j7CKo7pj008820; Fri, 12 Aug 2005 20:50:07 GMT Received: from zproxy.gmail.com (zproxy.gmail.com [64.233.162.199]) by robin.gentoo.org (8.13.4/8.13.4) with ESMTP id j7CKo6UJ031171 for ; Fri, 12 Aug 2005 20:50:07 GMT Received: by zproxy.gmail.com with SMTP id x7so531280nzc for ; Fri, 12 Aug 2005 13:50:07 -0700 (PDT) DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:user-agent:x-accept-language:mime-version:to:subject:references:in-reply-to:content-type:content-transfer-encoding; b=YMBzySDacpG2CZsLVTm+GyT8cKUXoCVrQM+ecFgCF5MTJZn5TWueoi5sBbY03BD30j+XS0HYcXPdOYTWNjWNv3/pjFNvZVwFGa6usRrVNLvadTVBiRfLz7AYqhkeWRBqq2qPomMqGFGPpztCC9tUqlfraFnlz7tnjU+Mygdl8xw= Received: by 10.36.19.1 with SMTP id 1mr2687110nzs; Fri, 12 Aug 2005 13:50:07 -0700 (PDT) Received: from ?192.168.1.104? ([24.30.171.44]) by mx.gmail.com with ESMTP id 15sm317041nzp.2005.08.12.13.50.06; Fri, 12 Aug 2005 13:50:07 -0700 (PDT) Message-ID: <42FD0D42.3020500@gmail.com> Date: Fri, 12 Aug 2005 13:57:38 -0700 From: Zac Medico User-Agent: Mozilla Thunderbird 1.0.6 (X11/20050804) X-Accept-Language: en-us, en Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-portage-dev@gentoo.org Reply-to: gentoo-portage-dev@lists.gentoo.org MIME-Version: 1.0 To: gentoo-portage-dev@lists.gentoo.org Subject: Re: [gentoo-portage-dev] the refactoring of emerge, continued... (was PATCH: refactor emerge spinner (#102073)) References: <42FADD3A.7020103@gmail.com> <200508112306.20986.jstubbs@gentoo.org> <42FC3163.2080709@gmail.com> <42FC97C9.7050301@egr.msu.edu> In-Reply-To: <42FC97C9.7050301@egr.msu.edu> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Archives-Salt: ee4acc24-2110-4bc4-8b26-f018ce0c5fce X-Archives-Hash: 060b7edafba5452fa3fcdd365d129950 Alec Warner wrote: > > I have emerge rewritten ( somewhere ) for HEAD once all of Brian's > config/domain/crazy stuff goes in, unless someone wants to unlease > modular emerge now, in which case I can pull the code out of the cobwebs > of my ${HOME} and work on it again. > > Basically the code allowed you to have a folder for modules that emerge > would detect at run-time, and load the proper ones depending on user > specification. The bonus to this was that some tools would be merged > into emerge instead of being seperate, which many users got upset about. > "emerge --rebuild" = revdep-rebuild -X for example. The main problem > with this is basically the same as above; emerge serves about 12 > different functions with crappy code everywhere, I managed to replicate > about half the functionality by just copying and pasting important code > everywhere, but most of it still needs to be completely rewritten ( > *stabs depgraph* ). I was going to wait for the new API to be done ( no > use rewriting emerge twice, IMHO ), but if you want it done now it > wouldn't be a big deal. Hmm, interesting. I can see how building other tools into the emerge interface would be useful if it allows more code to be shared somehow (and less duplication). Are there other reasons for combining other tools into the same interface? My main concern about "rewritten" code is that, depending on the nature of the code being rewritten, it may be likely to introduce unwanted changes or regressions. Of course, thats why I put so much emphasis on careful refactoring/reorganizing of existing code that is known to work in the desired way. I don't see a reason to keep messy code around for long periods of time when it can be so quickly reorganized. Why wait? Messy code only makes more difficult the job of maintaining and improving the code. Zac -- gentoo-portage-dev@gentoo.org mailing list