Re: [PATCH 1/2] Group architecture Documentation under Documentation/arch.

From: Rob Landley
Date: Sat Aug 04 2007 - 15:33:51 EST


On Saturday 04 August 2007 2:24:46 pm Jesper Juhl wrote:
> On 04/08/07, Rob Landley <rob@xxxxxxxxxxx> wrote:
> > On Saturday 04 August 2007 2:07:31 pm Jesper Juhl wrote:
> > > On 04/08/07, Rob Landley <rob@xxxxxxxxxxx> wrote:
> > > > Signed-off-by: Rob Landley <rob@xxxxxxxxxxx>
> > > > Amiga part Acked-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> > > >
> > > > Move architecture-specific Documentation into a common subdirectory.
> > >
> > > ...
> > >
> > > I have only one complaint; I don't see an update to
> > > Documentation/00-INDEX
> >
> > I now have at least three. (I missed blackfin, which wasn't there last
> > time I did this.)
>
> I'm not talking about Documentation/<arch>/00-INDEX, but the 00-INDEX
> file in the top-level Documentation directory.
>
> Or am I misunderstanding you?

I now have at least three complaints about the patch I just submitted: Your
complaint is valid, the empty directories all the files were moved out of
don't seem to be deleted when I use git to apply this patch, and I didn't
move the blackfin directory into arch because it wasn't there when I
initially did this patch.

I'll resubmit with 00-INDEX updates and blackfin moved, as soon as I figure
out how to tell git to DELETE THE DARN EMPTY DIRECTORIES ALREADY...

(I am seriously suspecting "edit the patch in vi" is going to be part of the
solution here. I _like_ mercurial. I know where to hit it and what with to
make it behave. Git, not so much...)

Rob
--
"One of my most productive days was throwing away 1000 lines of code."
- Ken Thompson.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/