<table><tr><td style="">aacid added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D27973">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D27973#625707" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D27973#625707</a>, <a href="https://phabricator.kde.org/p/heikobecker/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@heikobecker</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D27973#625570" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D27973#625570</a>, <a href="https://phabricator.kde.org/p/aacid/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@aacid</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Heiko, added you ask audex person, i think i have not removed any of the API used by audex either</p></div>
</blockquote>

<p>It still uses KCDDB::Config(), KCDDB::Client::config() and KCDDB::Client::submit(), which are all gone.</p></div>
</blockquote>

<p>Right, submit needs to be gone, there's nowhere to submit once freedb.org is gone. As for config() there's no config now, so you would need to just remove those calls too.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>k3b also breaks due to Client::store().</p></blockquote>

<p>What k3b here is doing is saving to the "kcddb cache" which i removed because i thought noone is using it, now we could add it back, but i'd rather not and lose that "feature" for 3 reasons:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">the cache is "cross application", and that is kind of weird if you ask me.</li>
<li class="remarkup-list-item">If we keep the cache we need to keep the kcm module just to be able to say in which folder the cache is, which again is weird because it's something you configure at the "system" level and not at the app level</li>
<li class="remarkup-list-item">the cache is centered a bit around the CD "database" model, which with the actual CDDB gone, i am not sure it makes sense to keep.</li>
</ul>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>audiocd-kio doesn't build either, but it looks like it's only missing a few includes.</p>

<p>I assume you want to get this in until tomorrow night to avoid bug reports about CD lookup not working?</p></blockquote>

<p>Yeah, i mean being part of the release service it *could* wait until next week since we know the thing is going to be released which is one of the reasons for the dependency freeze to be a week before the actual release (make sure your dependencies are released and in place).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R348 KCDDB Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27973">https://phabricator.kde.org/D27973</a></div></div><br /><div><strong>To: </strong>aacid<br /><strong>Cc: </strong>sitter, heikobecker, rstephenson, kde-doc-english, gennad, fbampaloukas, skadinna<br /></div>