Rockbox.org home
Downloads
Release release
Dev builds dev builds
Extras extras
themes themes
Documentation
Manual manual
Wiki wiki
Device Status device status
Support
Forums forums
Mailing lists mailing lists
IRC IRC
Development
Bugs bugs
Patches patches
Dev Guide dev guide
Search



Donate

Rockbox Technical Forums


Login with username, password and session length
Home Help Search Staff List Login Register
News:

Rockbox Ports are now being developed for various digital audio players!

+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  Plugins/Viewers
| | |-+  SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« previous next »
  • Print
Pages: [1] 2

Author Topic: SNCViewer Lyric Viewer and Sansa c200 c240 c250?  (Read 5427 times)

Offline OldiesButGoodies

  • Member
  • *
  • Posts: 15
SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« on: September 30, 2008, 03:39:34 PM »
I searched and searched the forums last night for hours. Then I googled for any mention of Rockbox that include the lyric viewer plugin for my Sansa c250 player. I came up empty. Like all others that try to get it to work that I've read in various other Rockbox forums, all I get is that message like "this plugin is not supported" or something like that.

Does anyone know of any recent build for the c250 that has or supports this plugin? I don't have the smarts to make my own Rockbox or even begin to know how, even after reading the "how to make your own" pages.

By the way, just who is it that determines what features get supported or not in the main releases of Rockbox? Is it just one or two people that make all the decisions? There seem to be so many wonderful things that others have added in their own builds, like an FM signal strength meter, expanded FM frequency ranges (so you can listen to weather channels and things like that, now crippled in the all main releases), the lyrics viewer, etc. Why are these left out from being supported by all players in the main release of Rockbox? It seems nothing short of a sin to cripple Rockbox like the manufacturers do with their own firmware. Isn't that the whole purpose of Rockbox? To uncripple everyone's players? There should be a place where everyone who writes and uses Rockbox can vote on this stuff.

Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #1 on: September 30, 2008, 03:45:29 PM »
Why should the users get to vote, exactly? How many of them are going to look at the code underneath the feature and discuss it on a "how it affects the overall project" basis rather than a "Oh, I think this feature is neat" basis?

Most features aren't included because there are significant flaws that need to be fixed first. We often want to include them, but they're done wrong, and the author has no interest in fixing them because he or she just wanted something he himself could use, and didn't really care about making sure it wouldn't interfere with future development.

Many of these features have negative side effects we don't want to introduce, or the author simply hasn't come forward and tried to get it included by asking "What else needs to be done?"

There's a group of over 50 people who decides what goes into Rockbox. In many cases only a small number need to be confident of a patch for it to get in, it's not a total group vote.

We aren't crippling Rockbox. We're making sure that we don't break it entirely by adding everything as soon as each thing works on its own. This is just a way toward significantly buggier software, and we value "it works" over "it can do everything, but crashes every 5 minutes" which is what will happen if we accept features without significant testing and code oversight.

I guess the difference is, we have the patience to wait for these patches to go through testing, and get improved to the point where we're satisfied, while you just want something that works now. Since you don't have the responsibility of being the one people come to when the software locks up, crashes, corrupts data, or otherwise, you don't have to worry about whether things actually work in the long run.

I assure you, if a feature is good, it gets in, in time. Just take a look at how far Rockbox has come over the last two years.
« Last Edit: September 30, 2008, 03:57:46 PM by Llorean »
Logged

Offline OldiesButGoodies

  • Member
  • *
  • Posts: 15
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #2 on: September 30, 2008, 04:10:51 PM »
Quote from: Llorean on September 30, 2008, 03:45:29 PM
Why should the users get to vote, exactly?

Because developers of other open-source projects listen to the needs of the end-users and like to make everyone happy. Too often the programmers are into programming and forget what the end-user really needs, wants, and likes. I used to program 6502 machine language 28 years ago, to make others happy (I've since forgotten all I knew back then, life took a 180-degree course). I listened to what other people wanted and tried to provide it for them whenever I programmed my lowly C64. Without the feedback from the end-users then what's the point of open-source projects? You're back to a totalitarian regime that dictates what everyone else wants, no different than a corporation headed by a few greedy and tunnel-vision CEOs.

It was interesting reading your over-the-top defensive reply to my semi-related ponderings (touched a nerve did I?), but I really just want to find a build that supports the lyric viewer.

Can you answer that one? This is afterall the section for discussing plugins, is it not? You tried to reply to everything but the question about the plugin. Can't you stay on topic?

Logged

Offline MarcGuay

  • Rockbox Expert
  • Member
  • *
  • Posts: 1065
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #3 on: September 30, 2008, 04:17:10 PM »
If you can't find a build that has that particular feature patched in (and it sounds like your searching would have turned it up if it existed), you can try and find the person who coded it in the first place and encourage them, or help them, to get it up to snuff so that it can be included in the main builds.

The patch is here in case you hadn't found it already:
http://www.rockbox.org/tracker/task/7432

And the comments generally will explain what still needs to be done.
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #4 on: September 30, 2008, 04:17:24 PM »
That's because Unsupported Builds are off topic in these forums, which you probably would've gotten if you read the guidelines, so I responded to the section that was actually a question about Rockbox.

Rockbox is coded for the programmers, not the users. We don't depend on donations, and the target audience is "the people working on it" with considerations for "everyone else we give it away to, for free."

So, frankly, while users considerations are taken into mind, I don't see why that should give them a place in the actual decision making process.

Your defense is "Hey guys, we use your software, so even though we're not the ones writing it, we, being the majority, should get to dictate how you use your spare time."

Seriously, when the programmers are in it for fun (as all of ours are), telling them what they can or cannot work on by way of voting is just a way to discourage them.
« Last Edit: September 30, 2008, 04:21:00 PM by Llorean »
Logged

Offline bluebrother

  • Developer
  • Member
  • *
  • Posts: 3421
  • creature
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #5 on: September 30, 2008, 04:34:46 PM »
Quote from: OldiesButGoodies on September 30, 2008, 04:10:51 PM
Without the feedback from the end-users then what's the point of open-source projects?
Fun. It's the only point in actively doing open source (oh, and before you mention it, there's in fact another point: to educate yourself. But one doesn't want to educate himself all the time, so fun is much more important).

Quote
You're back to a totalitarian regime that dictates what everyone else wants, no different than a corporation headed by a few greedy and tunnel-vision CEOs.
Nonsense. You can always start changing things -- it's just that you need to do the work. The people who actually did the work on open source projects just share their result with you, even without charge. Isn't that a great thing?

It's always the same with open source: take it, leave it or change it. But you have to do the work yourself.
Logged
Rockbox Utility development binaries (updated infrequently) · How to ask questions the smart way · We do not estimate timeframes.

Offline OldiesButGoodies

  • Member
  • *
  • Posts: 15
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #6 on: September 30, 2008, 04:37:29 PM »
Quote from: MarcGuay on September 30, 2008, 04:17:10 PM
The patch is here in case you hadn't found it already:
http://www.rockbox.org/tracker/task/7432

And the comments generally will explain what still needs to be done.

Thanks, I read all that. It appears that it was working just fine up until recent build of Rockbox. This is why I don't understand why the main developers aren't supporting it. Looks like they changed something simple without taking this plugin into consideration nor taking into consideration all the people that enjoyed using this plugin. (How would they know? They don't want to hear votes on what anyone else feels is important, apparently.)

Quote from: Llorean on September 30, 2008, 04:17:24 PM
...telling them what they can or cannot work on by way of voting is just a way to discourage them.

And comments like yours discourage the end-users from encouraging, applauding, and supporting the developers to let them know how great we might think someone is. If there was a way for everyone to vote on what everyone likes, then they'd know if what they are doing is something fantastic, or just an off-the-wall personal tweak that fills no useful purpose for everyone else on the planet. Think about how that all works. I see that you have the time to.

Still want to discuss off-topic? I'm just looking for a way to get the lyric plugin/viewer working on my Sansa.
Logged

Offline AlexP

  • Global Moderator
  • Member
  • *
  • Posts: 3688
  • ex-BigBambi
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #7 on: September 30, 2008, 04:42:46 PM »
This is getting silly.

It appears it is working fine is very different to it is written in the correct way to get into Rockbox without compromising the code base and therefore stability or future development.

And no, what users would like is not the most important thing - the way it is written is the most important thing.  A patch might be the best thing ever, but it will not get in if it is not done correctly, until someone fixes it.

Note I have no idea if this is true for this particular plugin, I'm just trying to explain why it isn't as simple as users want this.

It has been pointed out multiple times in this thread that pretty much everything that is done here is by volunteers *exactly* to scratch their own personal itch.  That is the difference between doing something for free and for fun, and being paid.

As a final note, slagging off a project on their forums is not the best way to persuade people to help you.
« Last Edit: September 30, 2008, 04:44:28 PM by BigBambi »
Logged
H140, F60, S120, e260, c240, Clip, Fuze v2, Connect, MP170, Meizu M3, Nano 1G, Android

Offline MarcGuay

  • Rockbox Expert
  • Member
  • *
  • Posts: 1065
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #8 on: September 30, 2008, 04:44:11 PM »
Quote from: OldiesButGoodies on September 30, 2008, 04:37:29 PM
Thanks, I read all that. It appears that it was working just fine up until recent build of Rockbox.

The last comment mentions specifically that the patch doesn't follow the house coding style.
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #9 on: September 30, 2008, 04:45:41 PM »
To get it working, compile a build with it, and fix any problems the patch has that prevent you from using it. You've already been told this.

In fact, you could take a little more time to improve the patch, and then it could get included. Since you seem to feel so strongly about it, I'm sure you're willing to take some time to work on it.
Logged

Offline OldiesButGoodies

  • Member
  • *
  • Posts: 15
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #10 on: September 30, 2008, 04:46:37 PM »
Quote from: MarcGuay on September 30, 2008, 04:44:11 PM
Quote from: OldiesButGoodies on September 30, 2008, 04:37:29 PM
Thanks, I read all that. It appears that it was working just fine up until recent build of Rockbox.

The last comment mentions specifically that the patch doesn't follow the house coding style.

OMG! That's the only reason? How foolishly petty.
Logged

Offline Bagder

  • Member
  • *
  • Posts: 1452
    • Daniel's site
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #11 on: September 30, 2008, 04:48:10 PM »
Quote from: OldiesButGoodies on September 30, 2008, 04:37:29 PM
Looks like they changed something simple without taking this plugin into consideration nor taking into consideration all the people that enjoyed using this plugin.

There is currently more than 300 open patches in the tracker. Of course we do changes without considering if one or more patch are affected.

Instead of wasting your energy bashing hundreds of volunteers that did all this for free and whose work you get to enjoy, I suggest you help out to get the patch in shape.
« Last Edit: September 30, 2008, 04:51:55 PM by Bagder »
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #12 on: September 30, 2008, 04:48:36 PM »
Nobody said that not matching the coding style was the only reason. But it's one of them. You could spend a few hours, fix it, and then it'd be that much closer. This would be a rather constructive alternative to complaining about it.

If you have any specific questions for help, ask them. If your next post is just more whining, I'm going to lock the thread, and you can come back when you have some clear request for help beyond "It doesn't work, and I don't want to work on it to make it work."
Logged

Offline MarcGuay

  • Rockbox Expert
  • Member
  • *
  • Posts: 1065
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #13 on: September 30, 2008, 04:49:00 PM »
Quote from: OldiesButGoodies on September 30, 2008, 04:46:37 PM
OMG! That's the only reason? How foolishly petty.

I didn't say it was the only reason.  You're the one who read all the comments, you tell me.
Logged

Offline bluebrother

  • Developer
  • Member
  • *
  • Posts: 3421
  • creature
Re: SNCViewer Lyric Viewer and Sansa c200 c240 c250?
« Reply #14 on: September 30, 2008, 05:08:49 PM »
Quote from: Bagder on September 30, 2008, 04:48:10 PM
There is currently more than 300 open patches in the tracker. Of course we do changes without considering if one or more patch are affected.
And while at it, don't forget that none of the patches is supported. Everything that is supported is in svn, and people actively working on patches usually get help from the devs when asking.

By the way, it can even happen that one developer unintentionally breaks what another dev is currently working on -- that's nothing special in collaborative development.
Logged
Rockbox Utility development binaries (updated infrequently) · How to ask questions the smart way · We do not estimate timeframes.

  • Print
Pages: [1] 2
« previous next »
+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  Plugins/Viewers
| | |-+  SNCViewer Lyric Viewer and Sansa c200 c240 c250?
 

  • SMF 2.0.17 | SMF © 2019, Simple Machines
  • Rockbox Privacy Policy
  • XHTML
  • RSS
  • WAP2

Page created in 0.115 seconds with 14 queries.