How do I remove the test_codec?

[lee321987]

   I followed the instructions at http://www.rockbox.org/twiki/bin/view/Main/CodecPerformanceComparison, and now am able to use the test_codec.

   I have two questions:

1)  How do I remove it from my compiles?  (I already tried removing the extra line the instructions said to put into the SOURCES file.)

2) Does including the test_codec make a build considered modified? (as in, "Don't report bugs on modified builds.")

[Llorean]

1) Removing that line removes it from the build. If you don't delete the .rock file from your player, though, you will obviously still see it in the list.

2) No. If that is the one, single change you've made (to the SOURCE file) it does not make your build considered modified, assuming your development environment is right. Specifically test_codec is official code, and changing the SOURCES file just enables that code. And since it's "working code that we just don't put in the release build" rather than "not done yet" code (like the USB code) it's valid to report bugs on.

[lee321987]

   About answer 1) -- The line is removed, and I deleted the .zip file from the last build, but test_codec.rock is still showing up in the new .zip.

[Llorean]

If it's actually IN the .zip file (and you're not just referring to seeing it on the player) you should probably re-run configure and do a clean build.

[cool_walking_]

It looks like buildzip.pl is just grabbing everything matching *.rock or *.ovl (what is this?), so if you just delete builddir/apps/plugins/test_codec.rock, which was built previously, it shouldn't appear in the .zip anymore.

[lee321987]

   All is well.
   Llorean, I mixed up the order of what you suggested...  I ran "make veryclean" first, then started at configure, but I did end up with a correct build.  And yes, I was talking about the .zip -- not the player.
   Then I got curious, and I made another build including the test_codec, so I could try your solution Cool_Walking_ (I deleted builddir/apps/plugins/test_codec.MAP at the same time). This worked too.

   Thanks guys,

Will