Bug 263775

I am running Amarok 2.4 on KDE 4.6 with the same problem.  I have tried encoding with ogg, mp3 and flac all with the same result.  I should not as well, that Amarok can't even play the cd.  It recognizes it and reads the track names, but thats it.

Setting status to confirmed.

Could you please test with latest amarok 2.4-git? CD playing and ripping should be fixed now.

Any news on this? Without feedback within 2 weeks I will close this as fixed.

*** Bug 276035 has been marked as a duplicate of this bug. ***

Created attachment 61153 [details]
log during error in amarok 2.4-GIT as of today

seems still a problem in KDE 4.6.4 and Amarok 2.4-GIT (gst backend).

I still get

amarok:   "" 
amarok:   NAY, transcoding fail! 

Please see attached 'amarok --debug' output for details.

reopening since its still reproducible in current 2.4-GIT

It was never closed, just "waitingforinfo"

I don't know if this is at all relevant, but this occurs even when ffmpeg clearly isn't installed or supported by the set of gstreamer plugins. I run on a Gentoo system so I can switch on and off ffmpeg support fairly simply. I did this as a test when I first saw this message.

Also interesting in the message is that it's asking ffmpeg (by all appearances) to access a URL. I'm pretty sure ffmpeg doesn't understand URL's, or at least definitely not KDE URL's.

There's a part of the latest log that reads like this:
"ffmpeg", "-i", "/MP3/Track 01.mp3"

ffmpeg is going to expect an /MP3 folder, when in fact KDE means cdda://MP3, neither of which ffmpeg can read.

I'm not a developer, though, so I could be wrong. It may be that ffmpeg used to be able to understand, or KDE is doing something it's not reporting in logs. I've just switched from GNOME to KDE, so I'm not used to all these library layers that don't have any intermediate executables to allow for debugging and testing, at least from a power user's point of view.

From a discussion on IRC, it appears that if you choose the "copy" option when inserting a CD, this works and produces an output transcoded to ogg vorbis format (I assume using KDE's built-in audio CD handling). This certainly isn't clear from the interface. The "transcode" option, with choices of output format etc, doesn't work.

Created attachment 63319 [details]
Output of 'amarok --debug', starting at CD insert

Also seeing complete CD-Rip failure.

Using 

amarok -v
 Qt: 4.7.3
 KDE Development Platform: 4.7.00 (4.7.0) "release 10"
 Amarok: 2.4.3

On disc insert, I see the CD appear in Amarok.

@ Rt-click, Copy Colletion -> Local Collection -> Transcode (either default or custom), simply creates the ./(Artist Name)/(Album Name) directory/subdirectroy in the target folder, but no transcoding occurs.

Output of 'amarok --debug', starting at CD insert, is attached

building from git,

git log | head -n 7
 commit a501993bb7efa3a4d52e04a9fb206a54b0ff0df4
 Author: Christophe Giboudeaux <cgiboudeaux@gmx.com>
 Date:   Fri Sep 2 21:16:49 2011 +0200
 
     more build fixes
     CCBUG: 281131

installs,

amarok -v
 Qt: 4.7.3
 KDE Development Platform: 4.7.00 (4.7.0) "release 10"
 Amarok: 2.4-GIT

repeating the procedures above, I get a different result.

Amarok now displays in 'Local Collection' under "Various Artists" the "Album Name" (not the Artist Name, as before), and displays a "Scanning Music" progressbar, which unfortunately, never progresses.

output from

 amarok --debug

simply cycles, repeating endlessly,

 ...
 amarok:   BEGIN: virtual bool Meta::AudioCdAlbum::isCompilation() const
 amarok:   END__: virtual bool Meta::AudioCdAlbum::isCompilation() const [Took: 0s]
 ...

with an occassional,

 ...
 amarok:   BEGIN: void ScanManager::checkForDirectoryChanges()
 amarok:     BEGIN: void DirWatchJob::setPaused(bool)
 amarok:     END__: void DirWatchJob::setPaused(bool) [Took: 0s]
 amarok:   END__: void ScanManager::checkForDirectoryChanges() [Took: 0s]
 amarok:   BEGIN: void ScanManager::checkForDirectoryChanges()
 amarok:     BEGIN: void DirWatchJob::setPaused(bool)
 amarok:     END__: void DirWatchJob::setPaused(bool) [Took: 0s]
 ...

thrown in.

Please do not Cc me, I am already subscribed to the bugs for Amarok, else I get those twice.

(In reply to comment #13)
> Please do not Cc me, I am already subscribed to the bugs for Amarok, else I get
> those twice.

Sorry.

You'd promised to previously close the bug if no feedback within two weeks,

 "Without feedback within 2 weeks I will close this as fixed."

but there hasn't been any comment, from anyone, since 2011-06-22.

It simply wasn't clear what the situation was, whether you'd received any of those responses you'd asked for, and I was trying to be helpful.

Read comment #7, there was feedback after my request.

yes, that's clear.  i didn't suggest there wasn't.

Closing, this is fixed in 2.4.3 for sure. Please make sure you have all necessary encoders installed.

Git commit 1a0287f7925d92a05a530ab83fffbc9afae67e86 by Matěj Laitl.
Committed on 21/01/2012 at 01:13.
Pushed by laitl into branch 'master'.

Rework transcoding: remember encoder, transcode on move, cleaner code

This is a major rework of transcoding feature that brings following
user-visible changes to Amarok:
 * Amarok can remember preferred transcoding configuration per each
   collection that supports transcoding. Therefore, the "Use default
   configuration" work-around can go away and the "Transcode or copy?"
   dialog can (and is) be one-step now. This preference can be changed
   in configuration.
 * Transcoding is now supported even during the move operation. No
   worries, only successfully transcoded tracks are removed from their
   original location.
 * Only formats playable on the target collection are offered. Already
   used & tested in yet-to-be-merged iPod collection rewrite.
 * The "Organize Tracks" dialog title and progress bar operation name
   now more verbosely describe actual operation to prevent user
   mistakes.
 * Double-transcode when ripping audio CDs that caused failures is
   avoided. (ChangeLog entry for this was miscredited to my earilier
   commit)

Technically, following changes are made:
 * many methods that accepted optional TranscodingConfiguration now
   either have it mandatory or not at all.
 * TranscodingConfiguration's NULL_CODEC was splitted to JUST_COPY and
   INVALID along with convenience methods isValid() and isJustCopy().
   This simplifies logic in many methods.
 * CollectionLocation::prepare{Copy,Move}() now don't have optional
   TranscodingConfiguration parameter. Depending on target collection,
   CollectionLocation determines it automatically or asks user in
   showSourceDialog() (overridable). AudioCdCollectionLocation already
   overrides it.
 * Collections that support transcoding now should expose
   TranscodeCapability which is used to a) indicate that transcoding
   is supported; b) query which file formats are playable on target
   collection; c) read & save & unset preferred transcoding parameters.

Why the hell the new Capability?
================================

Many Amarok devs dislike the concept of capabilities[1]. Why the hell I
introduced the new one? In ideal world Amarok would be able to transcode
everything regardless of the target collection. This is however not
doable witch current copyUrlToCollection() design - target collection
needs to do non-trivial things such as re-reading file tags, accounting
for different file name and space requirements etc. See my comments in
[1]. We therefore need a way for target collection to indicate it
supports transcoding (in order not to fool user). Some collection
locations such as TrashCollectionLocation should even intentionally
disallow transcoding. Additionally, we want to be able to query
supported destination file formats, to save preferred transcoding
paremeters etc.

I simply didn't want to pollute already over-crowded CollectionLocation
with three more methods used by only a few subclasses. On the other
hand, TranscodeCapability is not the central idea of this patch and I
can factor it into CollectionLocation should there be a voice supporting
it.

v2 patch version: gui string changes as suggested by Bart & Teo

[1] https://git.reviewboard.kde.org/r/103752/
Related: bug 280526, bug 264681, bug 291722
FIXED-IN: 2.6
REVIEW: 104213
DIGEST: Feature: much improved transcoding

M  +4    -1    ChangeLog
M  +1    -0    src/CMakeLists.txt
M  +10   -33   src/browsers/CollectionTreeView.cpp
M  +1    -2    src/browsers/CollectionTreeView.h
M  +1    -38   src/browsers/filebrowser/FileView.cpp
M  +0    -1    src/browsers/filebrowser/FileView.h
M  +21   -3    src/configdialog/dialogs/CollectionConfig.cpp
M  +33   -0    src/configdialog/dialogs/CollectionConfig.ui
M  +48   -12   src/core-impl/collections/db/sql/SqlCollection.cpp
M  +14   -1    src/core-impl/collections/db/sql/SqlCollection.h
M  +39   -18   src/core-impl/collections/db/sql/SqlCollectionLocation.cpp
M  +3    -1    src/core-impl/collections/db/sql/SqlCollectionLocation.h
M  +1    -1    src/core-impl/collections/mediadevicecollection/MediaDeviceCollectionLocation.h
M  +17   -0    src/core-impl/collections/support/CollectionLocationDelegateImpl.cpp
M  +3    -0    src/core-impl/collections/support/CollectionLocationDelegateImpl.h
M  +1    -1    src/core-impl/collections/support/PlaylistCollectionLocation.h
M  +2    -2    src/core-impl/collections/support/TrashCollectionLocation.h
M  +1    -1    src/core-impl/collections/umscollection/UmsCollectionLocation.h
M  +1    -0    src/core/CMakeLists.txt
M  +1    -0    src/core/capabilities/Capability.h
C  +5    -22   src/core/capabilities/TranscodeCapability.cpp [from: src/core/transcoding/TranscodingConfiguration.cpp - 073% similarity]
A  +84   -0    src/core/capabilities/TranscodeCapability.h     [License: GPL (v2+)]
M  +64   -21   src/core/collections/CollectionLocation.cpp
M  +19   -8    src/core/collections/CollectionLocation.h
M  +22   -0    src/core/collections/CollectionLocationDelegate.h
M  +112  -3    src/core/transcoding/TranscodingConfiguration.cpp
M  +41   -4    src/core/transcoding/TranscodingConfiguration.h
M  +17   -16   src/core/transcoding/TranscodingController.cpp
M  +21   -9    src/core/transcoding/TranscodingController.h
M  +11   -12   src/core/transcoding/TranscodingDefines.h
M  +3    -1    src/core/transcoding/TranscodingFormat.h
M  +4    -6    src/core/transcoding/formats/TranscodingNullFormat.cpp
M  +1    -1    src/core/transcoding/formats/TranscodingNullFormat.h
M  +2    -6    src/services/magnatune/MagnatuneCollectionLocation.cpp
M  +1    -1    src/services/mp3tunes/Mp3tunesServiceCollectionLocation.h
M  +1    -0    src/transcoding/CMakeLists.txt
M  +83   -75   src/transcoding/TranscodingAssistantDialog.cpp
M  +25   -9    src/transcoding/TranscodingAssistantDialog.h
M  +157  -296  src/transcoding/TranscodingAssistantDialog.ui
M  +3    -2    src/transcoding/TranscodingOptionsStackedWidget.cpp
A  +55   -0    src/transcoding/TranscodingSelectConfigWidget.cpp     [License: GPL (v2+)]
C  +42   -39   src/transcoding/TranscodingSelectConfigWidget.h [from: src/transcoding/TranscodingAssistantDialog.h - 050% similarity]
M  +1    -0    tests/core-impl/collections/db/sql/TestSqlCollectionLocation.cpp
M  +3    -0    tests/core/collections/MockCollectionLocationDelegate.h

http://commits.kde.org/amarok/1a0287f7925d92a05a530ab83fffbc9afae67e86