On Wed, 2007-09-05 at 15:54 -0500, Douglas McClendon wrote:
Jeremy Katz wrote:
> On Tue, 2007-09-04 at 16:47 -0500, Douglas McClendon wrote:
>> Mark McLoughlin wrote:
>>> On Mon, 2007-09-03 at 15:33 -0500, Douglas McClendon wrote:
>>>> Mark McLoughlin wrote:
>>> What you have is mostly fine IMHO, it was the tarball I had a problem
>>> with.
>> Ok, in that case, it should be no sweat (relatively) to put together a
>> new version of the patch, updated against the 30+ commits seen since the
>> last version, which uses dumpe2fs in anaconda, no tarball, osmin.img
>> truncation, and setting loops and dms to readonly where possible.
>>
>> But, before I do that, can I get feedback from one other person,
>> preferably who has commit authority, to tell me that this will be
>> committed? Jeremy, you mentioned objections in the past? (all my
>> goading of said objections, was because I was so sure that there is no
>> remotely more correct way to accomplish the task, and I would be
>> genuinely entertained to be corrected on that fact)
>
> I think that some of the above will go a long way towards making things
> look nicer which is going to make me more amenable to it. I still don't
> necessarily _like_ it because I still think that it makes anaconda
> depend a bit too much on a lot of the details; but I guess as long as it
> can fall back cleanly in the absence of the bits, that just means a
> larger testing matrix.
I'm going to assume that that was a "yes I like what the code does, yes
it will be applied"
I've been largely persuaded at least between your mails and markmc
twisting my arm a little yesterday ;)
As far as the larger testing matrix- If you go by my suggestion,
which
markmc echoed, and make it the default path (along with removing the
--ignore-deleted option in favor of it being default), then the testing
matrix will be the same.
Well, the anaconda path still has to handle both cases as people could
be using an older livecd-creator. That's really where the code-path
explosion is.
It definitely makes sense to make the new path the default in
livecd-creator and probably not to really make it a separate option to
enable/disable. As has probably been noticed, there are a lot fewer
options now, and a good chunk are marked as "for debugging/development
use only" with comments. I'm going to poke at optparse and hide them
from --help too.
Jeremy