Bug 21465 - K3b uses transcode to rip Video DVDs. Please make sure it is installed.
Summary: K3b uses transcode to rip Video DVDs. Please make sure it is installed.
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 6
Hardware: All Linux
Priority: Normal normal
Target Milestone: Mageia 6
Assignee: QA Team
QA Contact:
URL:
Whiteboard: MGA6-32-OK MGA6-64-OK advisory
Keywords: validated_update
Depends on:
Blocks:
 
Reported: 2017-08-07 13:29 CEST by Martin Volf
Modified: 2017-09-03 16:32 CEST (History)
7 users (show)

See Also:
Source RPM: k3b-2.10.0-9.mga6.tainted.src.rpm
CVE:
Status comment:


Attachments
K3B output (4.86 KB, text/plain)
2017-08-14 18:24 CEST, Martin Volf
Details
K3b with transcode implementation - screenshot (78.81 KB, image/png)
2017-08-29 19:26 CEST, Martin Volf
Details
Session output when transcoding a DVD (5.62 KB, text/plain)
2017-08-30 20:32 CEST, Len Lawrence
Details

Description Martin Volf 2017-08-07 13:29:19 CEST
Description of problem:
K3b can't rip DVD in case of missing transcode, even though transcode is installed. Then the button "Start Ripping" isn't active.

Version-Release number of selected component (if applicable):
k3b-2.10.0-9.mga6.tainted
transcode-1.1.7-17.mga6.tainted

How reproducible:
always

Steps to Reproduce:
1. Open K3b
2. insert original video DVD
3. go to Tools > Rip Video DVD... > warning "K3b uses transcode to rip Video DVDs. Please make sure it is installed."

Additional info:
- patch aplicateble (? - diff ver.): https://cgit.kde.org/k3b.git/commit/?id=f7d385a40cc062fa4db46c165f82f437dea82c58
- https://bugs.launchpad.net/ubuntu/+source/k3b/+bug/1686089
Martin Volf 2017-08-07 13:35:34 CEST

Target Milestone: --- => Mageia 6
CC: (none) => luigiwalser

Comment 1 David GEIGER 2017-08-07 15:15:37 CEST
Thanks for the report Martin Volf!

Should be fixed for mga6 with k3b-2.10.0-9.1.mga6:

http://svnweb.mageia.org/packages?view=revision&revision=1138486

CC: (none) => geiger.david68210

Comment 2 David GEIGER 2017-08-14 12:50:59 CEST
Assigning to QA,


Advisory:
========================

K3b can't rip DVD in case of missing transcode package, even though it is installed. Then the button "Start Ripping" isn't active.
So upstream decides to keep transcode until a replacement is implemented.
This update fixes this transcode issue.


========================

Packages in 6/core/updates_testing:
========================
k3b-2.10.0-9.1.mga6.i586.rpm
k3b-2.10.0-9.1.mga6.x86_64.rpm


Source RPM: 
========================
k3b-2.10.0-9.1.mga6.src.rpm

Assignee: pkg-bugs => qa-bugs

Comment 3 Martin Volf 2017-08-14 18:19:51 CEST
David, thank you for the update.

Unfortunately, after update no fix. Still a warning window "K3b uses transcode to rip Video DVDs. Please make sure it is installed." is appeared.
Comment 4 Martin Volf 2017-08-14 18:24:35 CEST
Created attachment 9599 [details]
K3B output
Comment 5 Frank Griffin 2017-08-14 18:33:30 CEST
Odd, cauldron has no tainted k3b.  If 6 does, it must be created when the final ISOs are made.  In any case, I suspect the problem is that David fixed the core/release k3b and Martin is using the tainted ones.

CC: (none) => ftg

Comment 6 Len Lawrence 2017-08-28 12:22:43 CEST
mga6  x86_64

Confirmed that k3b issued warning about transcode.

Updated k3b which pulled in transcode from tainted.  k3b tainted was removed.

Invoking 'rip video DVD' raised the same warning but as transcode was now installed it was safe to proceed.  Not sure at this point how to proceed but chose the copy route.  An encrypted DVD was detected, CSS keys retrieved and data written to an iso in/tmp.  Unfortunately, having no intention of burning the image to a blank DVD, as soon as I pressed cancel, the temporary file was deleted, so no chance to check its integrity.  I guess that measure is in place to discourage factory type operations.  It works in principle.

CC: (none) => tarazed25

Comment 7 Len Lawrence 2017-08-28 13:15:49 CEST
Started agai and loaded a blank DVD after the rip.  Preformatting issued an alarming series of 'unfinished line' messages and finished with a popup announcing  'Erasing failed'.

Terminal output:
...............................
(K3b::Process) found unfinished line: ' "/" '
(K3b::Process)             last char: ' "/" '
(K3b::Process)           joined line: ' "/" '
starting command:  "(CommandEject)"
Resetting medium in "/dev/sr0"
finished command:  "(CommandEject)"
no medium found
 unsupported medium found
"/dev/sr0"
MEDIA CHANGED in "/org/freedesktop/UDisks2/block_devices/sr0" ; size is: 0
"/org/freedesktop/UDisks2/block_devices/sr0"

The disk had been scratched during insertion which must have had something to do with the failure.  At least there was an opportunity to copy the iso and mount it. 
$ sudo mkdir /mnt/iso
$ sudo mount -t iso9660 -o ro,loop /data/lions.iso /mnt/iso
$ totem /mnt/iso
"Walking With Lions" started playing - no problem.

OK for 64-bits.
Len Lawrence 2017-08-28 13:16:10 CEST

Whiteboard: (none) => MGA6-64-OK

Comment 8 Lewis Smith 2017-08-28 22:26:28 CEST
Testing M6/64 to clarify things. With 

BEFORE the update: k3b-2.10.0-9.mga6

*without* transcode installed:
More actions-Rip video DVD:
"K3b uses transcode to rip Video DVDs. Please make sure it is installed"

*with* transcode installed: transcode-1.1.7-17.mga6.tainted
More actions-Rip video DVD:
"K3b uses transcode to rip Video DVDs. Please make sure it is installed."

AFTER update:

CC: (none) => lewyssmith

Comment 9 Lewis Smith 2017-08-28 22:47:15 CEST
Damo!
The tests above & below are with a video DVD inserted. K3b knew it was there because it powered up. Note my specific use of 'Rip video DVD' rather than 'Copy medium' which I think Len used.

AFTER update: k3b-2.10.0-9.1.mga6
*UN-installed* transcode:
K3b-More actions-Rip video DVD:
"K3b uses transcode to rip Video DVDs. Please make sure it is installed."
Legitimate;

*RE-installed* transcode: transcode-1.1.7-17.mga6.tainted
K3b-More actions-Rip video DVD:
"K3b uses transcode to rip Video DVDs. Please make sure it is installed."
NOT legitimate. And the 'Start Ripping' button is indeed de-activated.
Same as Comment 3.

Not OK in my book, despite Len's perseverence. So undoing the 64-bit OK and asking for feedback. Witholding the advisory for the moment.

Whiteboard: MGA6-64-OK => feedback

Comment 10 Len Lawrence 2017-08-29 01:48:08 CEST
Arrgh!  Had not noticed 'more actions'.  Sorry for the false trail.
Comment 11 Martin Volf 2017-08-29 19:26:04 CEST
Created attachment 9648 [details]
K3b with transcode implementation - screenshot

Omg, I did not relizate that I have to do more package updates. The update of the package k3b and transcode is not enough! 

For all MGA6 testers:
*** 32bit
# urpme k3b transcode libk3bdevice7 libk3blib7
# drakrpm-edit-media     ← add testing medias
# urpmi.update -a
# urpmi k3b transcode libk3bdevice7 libk3blib7

*** 64bit
# urpme k3b transcode lib64k3bdevice7 lib64k3blib7
# drakrpm-edit-media     ← add testing medias
# urpmi.update -a
# urpmi k3b transcode lib64k3bdevice7 lib64k3blib7

Those packages are needed:
.../media/core/updates_testing/k3b-devel-2.10.0-9.1.mga6.x86_64.rpm
.../media/core/updates_testing/k3b-2.10.0-9.1.mga6.x86_64.rpm
.../media/core/updates_testing/lib64k3blib7-2.10.0-9.1.mga6.x86_64.rpm
.../media/core/updates_testing/lib64k3bdevice7-2.10.0-9.1.mga6.x86_64.rpm
.../media/core/updates_testing/libk3blib7-2.10.0-9.1.mga6.i586.rpm
.../media/core/updates_testing/libk3bdevice7-2.10.0-9.1.mga6.i586.rpm
.../media/tainted/release/transcode-1.1.7-17.mga6.tainted.$ARCH.rpm

Then run k3b → Tools → Rip Video DVD ... → Start Ripping

To check implemented transcode see Settings → Configure K3b → Programs → Tab Programs → Transcode available. See attached screenshot.

Results:
========
MGA6 32bit real HW: it works for me
MGA6 64bit real HW: it works for me


Please test it too, then change a status to pull updates.

Thanks David!
Comment 12 Len Lawrence 2017-08-30 19:54:50 CEST
mga6  x86_64

Tried this on two machines and DVD rip has failed on both.  Checked k3b settings/programs and confirmed that transcode was there.  The first pass fails on the three chapters found and then k3b gives up.  The log fills up but disappears as soon as k3b closes.
Comment 13 Len Lawrence 2017-08-30 20:27:27 CEST
The man page for transcode gives an example command, so to see if transcode is working I ran that; see attached session output.  It looks as if the program has hung.  Nothing happening after more than ten minutes.

Two small files have been written to the user directory.
$ ls -l *.avi
-rw-r--r-- 1 lcl lcl 2464 Aug 30 19:08 my_movie-ch01.avi
-rw-r--r-- 1 lcl lcl 2048 Aug 30 19:08 my_movie-ch02.avi
Comment 14 Len Lawrence 2017-08-30 20:32:39 CEST
Created attachment 9650 [details]
Session output when transcoding a DVD
Comment 15 Len Lawrence 2017-08-31 12:38:34 CEST
One final try on yet another 64-bit machine.  Started with a clean slate, no k3b installed.  Installed k3b, k3b-devel and libraries from updates testing.  Mate desktop.  k3b started up; checked that transcode was available and selected all 8 titles of a commercial DVD.  That required lib64dvdcss2.  k3b attempted to rip the 8 titles but failed on each one; "Execution of transcode failed".  That whole process took about 30 minutes with eight cores working hard.

Typical extract from the terminal log:

started
"Two-pass Encoding: First Pass"
"Transcoding title 6 from Video DVD Chicago"
K3bQProcess::QProcess(0x0)
***** transcode parameters:

"/usr/bin/transcode --nice 19 --log_no_color --progress_meter 2 --progress_rate 4 -i /dev/sr0 -x dvd -T 6,-1,1 -a 0 -j 0,0,0,0 -R 1,/data/isos/k3b_0.log -y ffmpeg,null -o /dev/null -F mpeg4 -w 1200 -Z 768x576"
started
"Two-pass Encoding: First Pass"
"Transcoding title 7 from Video DVD Chicago"
K3bQProcess::QProcess(0x0)
Comment 16 Lewis Smith 2017-09-02 11:41:00 CEST
(In reply to Martin Volf from comment #11)
> For all MGA6 testers:
> *** 32bit
> # urpme k3b transcode libk3bdevice7 libk3blib7
> # drakrpm-edit-media     ← add testing medias
> # urpmi.update -a
> # urpmi k3b transcode libk3bdevice7 libk3blib7
> 
> *** 64bit
> # urpme k3b transcode lib64k3bdevice7 lib64k3blib7
> # drakrpm-edit-media     ← add testing medias
> # urpmi.update -a
> # urpmi k3b transcode lib64k3bdevice7 lib64k3blib7
> 
> Those packages are needed:
> .../media/core/updates_testing/k3b-devel-2.10.0-9.1.mga6.x86_64.rpm
> .../media/core/updates_testing/k3b-2.10.0-9.1.mga6.x86_64.rpm
> .../media/core/updates_testing/lib64k3blib7-2.10.0-9.1.mga6.x86_64.rpm
> .../media/core/updates_testing/lib64k3bdevice7-2.10.0-9.1.mga6.x86_64.rpm
> .../media/core/updates_testing/libk3blib7-2.10.0-9.1.mga6.i586.rpm
> .../media/core/updates_testing/libk3bdevice7-2.10.0-9.1.mga6.i586.rpm
> .../media/tainted/release/transcode-1.1.7-17.mga6.tainted.$ARCH.rpm
> Results:
> ========
> MGA6 32bit real HW: it works for me
> MGA6 64bit real HW: it works for me
First, thanks for your own good tests.

BUT I am confused by all this. The original update was just K3b. Accepting that certain related libraries also needed updating, I do *not* understand:
"The update of the package k3b and transcode is not enough"
"urpme k3b transcode"
"urpmi k3b transcode"
because transcode does *not* seem to be involved in the update; it starts & finishes at the same version: transcode-1.1.7-17.mga6.tainted

OTOH it looks as if you have added (understandably) these libraries to the update:
- libk3bdevice7 libk3blib7
- lib64k3bdevice7 lib64k3blib7

To clarify this, could you please re-state the packages/SRPM list in Comment 2?
And what has 'k3b-devel' got to do with this? You say it is needed - but only apparently for M6.

I shall re-test this (x64) with just K3b + the 2 additional libraries, which I think is what you intend.
As for Len's problems with 'transcode', that looks like a new bug. This one is just about K3b not recognising the presence of transcode.
Comment 17 Martin Volf 2017-09-02 12:19:52 CEST
(In reply to Lewis Smith from comment #16)
> BUT I am confused by all this. The original update was just K3b. Accepting
> that certain related libraries also needed updating, I do *not* understand:
> "The update of the package k3b and transcode is not enough"
> "urpme k3b transcode"
> "urpmi k3b transcode"
> because transcode does *not* seem to be involved in the update; it starts &
> finishes at the same version: transcode-1.1.7-17.mga6.tainted

Correct, just to be sure the transcode package needs to be installed.

(In reply to Lewis Smith from comment #16)
> OTOH it looks as if you have added (understandably) these libraries to the
> update:
> - libk3bdevice7 libk3blib7
> - lib64k3bdevice7 lib64k3blib7
> 
> To clarify this, could you please re-state the packages/SRPM list in Comment
> 2?
Yes, when I enable updates_testing and want to update k3b '# urpmi k3b', then those two libs are not updated automatically and I have to update them too.

(In reply to Lewis Smith from comment #16)
> And what has 'k3b-devel' got to do with this? You say it is needed - but
> only apparently for M6.
Sorry, I was not correct. k3b-devel is not neccessary. Was listed from the command '# urpmq -yf k3b'

(In reply to Lewis Smith from comment #16)
> I shall re-test this (x64) with just K3b + the 2 additional libraries, which
> I think is what you intend.
Correct.

> As for Len's problems with 'transcode', that looks like a new bug. This one
> is just about K3b not recognising the presence of transcode.
I think too. It looks to new bug.
Comment 18 David Walser 2017-09-02 12:37:59 CEST
There are no additional SRPMs, but the list of RPMs generated from the k3b SRPM David gave was incomplete.  It also includes the two libraries and devel package.

Whiteboard: feedback => (none)

Comment 19 James Kerr 2017-09-02 12:54:00 CEST
The definitive list of RPM's is available on madb:

http://madb.mageia.org/tools/listRpmsForQaBug/bugnum/21465/application/0

To access that page click on the "RPM's" link on the relevant line in the QA Updates page.

CC: (none) => jim

Comment 20 David GEIGER 2017-09-02 14:06:03 CEST
Updated list:


Advisory:
========================

K3b can't rip DVD in case of missing transcode package, even though it is installed. Then the button "Start Ripping" isn't active.
So upstream decides to keep transcode until a replacement is implemented.
This update fixes this transcode issue.


========================

Packages in 6/core/updates_testing:
========================
k3b-2.10.0-9.1.mga6.i586.rpm
k3b-devel-2.10.0-9.1.mga6.i586.rpm
libk3bdevice7-2.10.0-9.1.mga6.i586.rpm
libk3blib7-2.10.0-9.1.mga6.i586.rpm

k3b-2.10.0-9.1.mga6.x86_64.rpm
k3b-devel-2.10.0-9.1.mga6.x86_64.rpm
lib64k3bdevice7-2.10.0-9.1.mga6.x86_64.rpm
lib64k3blib7-2.10.0-9.1.mga6.x86_64.rpm

Source RPM: 
========================
k3b-2.10.0-9.1.mga6.src.rpm
Comment 21 Len Lawrence 2017-09-02 21:57:52 CEST
Re comment 16.  I agree with your analysis Lewis so this bug can be moved on.  Shall raise a bug on transcode.
Raise your hand if you disagree about the OK.
Len Lawrence 2017-09-02 21:58:14 CEST

Whiteboard: (none) => MGA6-64-OK

Martin Volf 2017-09-02 22:47:57 CEST

Whiteboard: MGA6-64-OK => MGA6-32-OK MGA6-64-OK

Comment 22 Lewis Smith 2017-09-03 10:25:08 CEST
(In reply to James Kerr from comment #19)
> The definitive list of RPM's is available on madb:
> To access that page click on the "RPM's" link on the relevant line in the QA
> Updates page.
Very useful info! Never looked there before, and it shows just as you said. And the SRPMs for advisories.

In fact this whole very simple update test was queered simply by not realising that the 2 k3b libraries were involved as well. The initial tests would then have sufficed. A real mountain out of a molehill. But QA should not have to cross-check the RPMs cited in the bug.
-------------------------------------
Re-testing M6/64 (for my own sanity)
K3b with video DVD inserted. (Aside: under Mate, K3b has no menu entry].

AFTER the update:
k3b-2.10.0-9.1.mga6
lib64k3blib7-2.10.0-9.1.mga6
lib64k3bdevice7-2.10.0-9.1.mga6

1. WITHOUT transcode installed:
K3b-More actions-Rip video DVD
"K3b uses transcode to rip Video DVDs. Please make sure it is installed"
Correct.

2. WITH transcode installed: transcode-1.1.7-17.mga6.tainted
K3b-More actions-Rip video DVD
Does *not* show the missing transcode error, and the 'Start ripping' button is active. So essentially the update is OK.
---------------------------------------
Be a devil - try it. The trouble is, I know nothing about the various tracks of a video DVD, so just pressed 'start' on what it proposed. There was an immediate error:
 "Resizing picture of title 1
 Execution of transcode failed
 Please consult the debugging output [button for this below]
 Failed to rip title 1"
but it went on to 'title 2' OK with 2 progress bars: one for 'first pass [of 2]' and the other 'overall' which understandably advanced at half the speed.
+ a little progress window for transcode itself which advanced the same as the 'overall' progress. Not sitting this out, so the situation is not as black as it seemed.
-------
Validating, will do the advisory immediately.

Keywords: (none) => validated_update
CC: (none) => sysadmin-bugs

Lewis Smith 2017-09-03 10:31:37 CEST

Whiteboard: MGA6-32-OK MGA6-64-OK => MGA6-32-OK MGA6-64-OK advisory

Comment 23 Mageia Robot 2017-09-03 16:32:18 CEST
An update for this issue has been pushed to the Mageia Updates repository.

http://advisories.mageia.org/MGAA-2017-0074.html

Status: NEW => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.