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 - Akregator default feeds all point to KDE sites
Summary: Akregator default feeds all point to KDE sites
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdepim (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Other
: P1 normal
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2011-12-17 12:41 CST by Darrell
Modified: 2012-10-19 15:22 CDT (History)
2 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name: Akregator

Attachments
Update the default feeds (2.64 KB, patch)
2012-01-02 22:02 CST, Darrell 		Details | Diff
Updated patch for latest git (1.67 KB, patch)
2012-01-03 00:21 CST, Darrell 		Details | Diff
Updated patch for latest git. (1.67 KB, patch)
2012-01-03 01:41 CST, Darrell 		Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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!