By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 750

Summary:	Akregator default feeds all point to KDE sites
Product:	TDE	Reporter:	Darrell <darrella>
Component:	tdepim	Assignee:	Timothy Pearson <kb9vqf>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bugwatch, darrella
Priority:	P1
Version:	R14.0.0 [Trinity]
Hardware:	Other
OS:	Other
Compiler Version:		TDE Version String:
Application Version:		Application Name:	Akregator
Attachments:	Update the default feeds Updated patch for latest git Updated patch for latest git.

Description Darrell 2011-12-17 12:41:44 CST

The built-in default RSS feeds for Akregator all point to KDE sites.

Akregator News all date from 2005 or earlier.

Akregator Blog is broken and no longer exists.

KDE Dot News, Planet KDE are current feeds but all relate to KDE4 news.

KDE Apps and KDE Look is current but mostly relates to KDE4.

This is a paper cut candidate and a somewhat serious public relations bug. All of these feeds should be removed.

To show new users that Akregator is working correctly, at least one Trinity feed should exist.

Comment 1 Darrell 2012-01-02 22:02:46 CST

Created attachment 247 [details]
Update the default feeds

Patch to update the default feeds to the following:

LXer Linux News (http://lxer.com/module/newswire/headlines.rss)
Tuxmachines (http://www.tuxmachines.org/node/feed)
lwn.net (http://lwn.net/headlines/rss)
H-Online (http://www.h-online.com/grand-atom.xml)

Comment 2 Darrell 2012-01-02 22:04:01 CST

The patch changes the feed category/heading from "KDE" to "Free/Libre Software News."

Comment 3 Timothy Pearson 2012-01-02 22:16:55 CST

I have updated the RSS feed to point to TDE's news channel in GIT hash d409ae59.  Should these other feeds be included as well?

Comment 4 Darrell 2012-01-03 00:21:48 CST

Created attachment 249 [details]
Updated patch for latest git

We have a news feed? I missed that memo.... :) Nonetheless, cool!

Per the limited discussion in the mailing list, I proposed those four sites as more current and relative feeds than the ones currently hard-coded in the sources. (I see in git you deleted the old feeds.) Nobody responded in any negative manner, hence my first patch.

I merged my changes with the latest git. New patch attached.

I haven't built tdepim and tested, but the patch looks sane to me. :)

I took the liberty of changing the "ak" designator to "tde." Looks more "appropriate" to anybody who looks at the code. ;)

Comment 5 Darrell 2012-01-03 01:41:25 CST

Created attachment 250 [details]
Updated patch for latest git.

Oops. Next time don't submit a patch without testing. Shame on me. :)

Attached is an updated patch for git.

I named the main folder "Free/Libre Software News." The TDE feed is the first one under the new folder.

Another option would have been to create two folders, say, firstFolder and secondFolder, with TDE being firstFolder and Free/Libre Software News being secondFolder. I won't be upset if you want to do that. :)

I deleted the previous obsolete patches.

Comment 6 Timothy Pearson 2012-01-03 01:57:10 CST

No problem, and thanks for updating the patches! :)

I had looked around for this bug report before updating the code in GIT, but could not find it that day for whatever reason.  I didn't want to ship outdated RSS feeds yet again, so I pushed in the update anyway.

I silently rolled out the RSS feature, so you didn't miss any memos. :)

Comment 7 Timothy Pearson 2012-01-11 13:31:21 CST

Committed to GIT in hash 156eeed.

Thanks for reporting, and for the (updated) patch!