-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117580/
-----------------------------------------------------------

(Updated April 16, 2014, 11:09 a.m.)


Review request for Amarok and Myriam Schweingruber.


Changes
-------

Added "amarok" as group.


Bugs: 332811
    https://bugs.kde.org/show_bug.cgi?id=332811


Repository: amarok


Description
-------

Short bug description:
Basically for m4a/mp4 files the tag is inserted but it's not recognized if the 
tool runs again. So the file is modified and a possibly-new tag is rewritten.

Please see bug tracking for more information.


Diffs
-----

  utilities/afttagger/AFTTagger.cpp 5a60c6b 

Diff: https://git.reviewboard.kde.org/r/117580/diff/


Testing
-------

Tested myself on the same installation on which I could reproduce the bug.

Thank you,
Stefano

INFO: Processing file /home/xxx/Music/Test/Code 64/2003 - Storm/12 - Reaktor 
(Original mix).m4a
INFO: Temporary file is at /home/xxx/Music/Test/Code 64/2003 - Storm/12 - 
Reaktor (Original mix).m4a.amarok-afttaggertemp.pid-20234.random-xrczmmvk.m4a
INFO: File is a MP4 file, opening...
INFO: Found an existing AFT identifier: ----:com.apple.iTunes:Amarok 2 AFTv1 - 
amarok.kde.org
INFO: AFT identifier is version 1
INFO: ID is current
INFO: Cleaning up...
INFO: Processing file /home/xxx/Music/Test/Code 64/2003 - Storm/13 - Without 
You (Zprochek mix).m4a
INFO: Temporary file is at /home/xxx/Music/Test/Code 64/2003 - Storm/13 - 
Without You (Zprochek 
mix).m4a.amarok-afttaggertemp.pid-20234.random-fbNJp3UM.m4a
INFO: File is a MP4 file, opening...
INFO: Found an existing AFT identifier: ----:com.apple.iTunes:Amarok 2 AFTv1 - 
amarok.kde.org
INFO: AFT identifier is version 1
INFO: ID is current
INFO: Cleaning up...
INFO: Processing file /home/xxx/Music/Test/Code 64/2003 - Storm/14 - Without 
You (Spektron mix).m4a
INFO: Temporary file is at /home/xxx/Music/Test/Code 64/2003 - Storm/14 - 
Without You (Spektron 
mix).m4a.amarok-afttaggertemp.pid-20234.random-OLe7wC54.m4a
INFO: File is a MP4 file, opening...
INFO: Found an existing AFT identifier: ----:com.apple.iTunes:Amarok 2 AFTv1 - 
amarok.kde.org
INFO: AFT identifier is version 1
INFO: ID is current
INFO: Cleaning up...
INFO: All done, exiting...


Thanks,

Stefano Pettini

_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel

Reply via email to