Re: [Nautilus-list] Remove Eazel from nautilus news



On Tue, 10 Jul 2001, Darin Adler wrote:

> On Tuesday, July 10, 2001, at 06:29  AM, Jens Finke wrote:
>
> > attached is a patch which removes explicitly the Eazel news entry from the
> > nautilus sidebar. The changes in nautilus-news.c are only neccessary
> > because there exists local copies of the xml file for every user, so a
> > simple substitution of the system wide file isn't sufficient. Although
> > the hardcoded thing isn't a nice solution.
>
> Removing the entry from the news_channels.xml file is a no brainer. Please
> commit that change (with a change log entry) if you would.

Done.

> What to do about people who already have an XML file is a separate issue.
> I think that hard-coding the fact that this channel is gone is not a good
> solution. Other channels may disappear or change in the future too.
>
> So lets see if we can think of a better solution for that half of the
> problem that is not just a one-time hack for the Eazel channel.

Ok. What about a news_channels_remove.xml file, which is only installed
systemwide? The format is the same as for the news_channels.xml but only
with the name attribute for every channel (the other
attributes are not needed). We load the channels in the remove-file and
compare them during loading with the channels in the users
news_channels.xml file. If there is a hit then skip this channel and go on
with loading the users file.

Well, if you think this is too much effort for a nearly unused feature or
don't like the idea at all, just tell me. I just found the Eazel entry in
the news, while playing with nautilus and thought this should be updated.
Maybe its totally sufficient to remove it from the systemwide file.

Regards,

   Jens





[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]