[Mono-docs-list] [PATCH] Monodoc. Workaround for a bug and index improvements

[Rafael Ferreira]

Mario, 

Looks good to me. I'm glad to see that bug resolved. As far as the index
fix, I have to say... why don't you just create the index once as part
of the "make install" target? That way you don't have to worry about
doing the async index building... and the user does not have to do
anything. I apologize if that is a silly question but it just seems a
bit "over engineered".

You might also want to change this:

 Console.WriteLine ("You don't have permissions to wirte on " + dir);
to this:
 Console.WriteLine ("You don't have permissions to write to " + dir);


As always, just my 2 cents. :-)



On Mon, 2005-09-19 at 16:56 +0200, Mario Sopena wrote:
> Hey,
> 
> 2005/9/19, Rafael Ferreira <lists at ophion.org>:
> > Mario, no patch attached?
> > 
> 
>   I did resent the patch, look that your filters didn't put my message
> in the mono-devel list, as I sent also the patch there.
> 
> Sorry, I would like to post a link to my message, but the web mailing
> list archive seems to be down for the moment.
> 
> Mario
> 
> 
> > On Fri, 2005-09-16 at 16:36 +0200, Mario Sopena wrote:
> > > Hello,
> > >
> > >     We have a problem in Monodoc when showing big html code with
> > > gecko. Monodoc hangs and do nothing (try to load the Gtk Namespace).
> > > This is due to a bug in gtkmozembed
> > > (https://bugzilla.mozilla.org/show_bug.cgi?id=245960). The workaround
> > > I've implemented writes the html on disk and loads the file from it,
> > > when the html code is big enough. The file is being writen to a temp
> > > directory.
> > > The only thing I'm not sure is whether I should myself delete what I
> > > write there or if I can just leave that for the system (for the
> > > moment, nothing is removed).
> > >
> > > The other thing that comes with this patch is a user-feature requested
> > > by miguel. The index and the search_index require those index to be
> > > create prior to use them. Right now, if they don't exist, a label is
> > > showed telling you that you have to run a command in root to create
> > > them.
> > > With this patch, monodoc looks for the index also in the user dir, and
> > > when it doesn't find them, it shows a panel with a progress bar that
> > > lets you make them at that time. The index are created in another
> > > Thread, so you can use monodoc while making them (specially the search
> > > index is slow, well, around a minute in my machine).
> > > To try the patch, remove your monodoc.index file and search_index dir
> > > from the monodoc directory and run the patched monodoc.
> > >
> > > Hope you all enjoy it. Comments please?!?
> > >
> > > Mario
> > > _______________________________________________
> > > Mono-docs-list maillist  -  Mono-docs-list at lists.ximian.com
> > > http://lists.ximian.com/mailman/listinfo/mono-docs-list
> > >
> > 
> >
>