• Home
  • Blog
  • Add-ons
  • Forum
  • Wiki
  • Developers
Nightingale - The tune of life, the tune of yours
  • Portal
  • Search
  • Member List
  • Calendar
  • Help
  • Portal
  • Search
  • Member List
  • Calendar
  • Help
Guest Hi, Guest
  • Login
  • Register
Login
Username:
Password: Lost Password?
 
Nightingale Forums Development Technical Development [BUG] lyrics non ASCII characters saved as ??????

Thread Rating:
  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
Thread Modes
[BUG] lyrics non ASCII characters saved as ??????
LookingMan Offline
Junior Member
**
Posts: 27
Threads: 4
Joined: Dec 2011
Reputation: 1
#1
07-23-2012, 06:25 AM (This post was last modified: 07-24-2012, 04:01 AM by LookingMan.)
Seems taglib (or not taglib) problem appeared again.
I'm talking about the bug http://getsatisfaction.com/songbird/topi..._non_ascii which has been closed.

Steps to reproduce:
Just install MLyrics, try to edit the lyrics with something like: Это проверка
Save (for save to complete you may need to pause playback). Now remove the track with saved lyrics from the library. Import the track again and you will see the lyrics was saved as ??? ????????

Found a temporary solution: replace sbMetadataHandlerTaglib.so in components directory with Songbird's (Tested with Songbird 2.1.0a, Build 2297 (20120526014106))
MLyrics developer
Find
Reply
LookingMan Offline
Junior Member
**
Posts: 27
Threads: 4
Joined: Dec 2011
Reputation: 1
#2
07-23-2012, 11:27 PM (This post was last modified: 07-24-2012, 12:02 AM by LookingMan.)
Just re-downloaded http://sourceforge.net/projects/ngale/fi...2/download and unfortunately the bug exist there too.

Found a temporary solution: replace sbMetadataHandlerTaglib.so in components directory with Songbird's (Tested with Songbird 2.1.0a, Build 2297 (20120526014106))

P.S: So the problem in the taglib.

If you look deeper at
http://bugzilla.songbirdnest.com/show_bug.cgi?id=22958
there was a patch
http://bugzilla.songbirdnest.com/attachm...ction=diff
which is applied to SB's taglib
http://src.songbirdnest.com/xref/vendor/...sframe.cpp
but not applied to NG's taglib
https://github.com/nightingale-media-pla...sframe.cpp
(I'm assuming NG uses this taglib)

So why NG doesn't use SB's taglib?

P.S: I cannot build NG now, so asking for someone to try the patch.
P.P.S: Saved lyrics in mp3 can also easily be viewed with eyeD3 program.

Opened an issue https://github.com/nightingale-media-pla...issues/115
MLyrics developer
Find
Reply
rsjtdrjgfuzkfg Offline
Developer
*******
Posts: 664
Threads: 15
Joined: Oct 2011
Reputation: 15
#3
07-24-2012, 03:24 AM
I'm on vacation right now and thus without a building environment. Note that at least on windows there are tons of issues with taglib we need to dive into, probably reimplement the wrapper from scratch so we do not rely on Songbird's patches there and can use system taglib on linux and build more easy on windows. There are many related issues on github, feel free to add yours Wink
Songbird/Nightingale Community Developer and German Translator
Find
Reply
LookingMan Offline
Junior Member
**
Posts: 27
Threads: 4
Joined: Dec 2011
Reputation: 1
#4
07-24-2012, 03:48 AM
OK, the issue was opened. But could we temporary use SB's taglib (or temporary apply patch after you come back)? Because this bug is really ugly. I found temporary solution, but other people may not see this thread and discover the bug after system or NG complete reinstall (and tons of lyrics will be corrupted Sad ).

P.S: This bug is freaking me out.
MLyrics developer
Find
Reply
ilikenwf Offline
Developer/Project Manager
*******
Posts: 412
Threads: 22
Joined: Nov 2010
Reputation: 8
#5
07-24-2012, 11:01 AM
Yeah - you can build against the SB taglib.

I'll rebuild the deps when we launch 1.11.1 - we need a few more bugfixes before we mark that milestone, though.
How to Build Nightingale | Nightingale on GitHub
Find
Reply
LookingMan Offline
Junior Member
**
Posts: 27
Threads: 4
Joined: Dec 2011
Reputation: 1
#6
07-24-2012, 04:32 PM
OK, thanks.
MLyrics developer
Find
Reply
rsjtdrjgfuzkfg Offline
Developer
*******
Posts: 664
Threads: 15
Joined: Oct 2011
Reputation: 15
#7
07-25-2012, 08:59 AM
(07-24-2012, 03:48 AM)LookingMan Wrote: But could we temporary use SB's taglib (or temporary apply patch after you come back)?

...on Windows we use SB's taglib currently. Doesn't it work there?
Songbird/Nightingale Community Developer and German Translator
Find
Reply
LookingMan Offline
Junior Member
**
Posts: 27
Threads: 4
Joined: Dec 2011
Reputation: 1
#8
07-25-2012, 10:52 PM
Don't know, NG crashes there every time I try to erase lyrics or set new, and then fails to correctly read track metadata (artist and album empty, track name set to mp3's name). Should I open an issue on that too?
MLyrics developer
Find
Reply
rsjtdrjgfuzkfg Offline
Developer
*******
Posts: 664
Threads: 15
Joined: Oct 2011
Reputation: 15
#9
07-25-2012, 11:55 PM
(07-25-2012, 10:52 PM)LookingMan Wrote: Don't know, NG crashes there every time I try to erase lyrics or set new, and then fails to correctly read track metadata (artist and album empty, track name set to mp3's name). Should I open an issue on that too?

No, that's known (issue #65 or something if I remember right). That's the reason we need to get away from the SB-patched taglib. Somehow our build and their deps don't go along that well on Windows...
Songbird/Nightingale Community Developer and German Translator
Find
Reply
ilikenwf Offline
Developer/Project Manager
*******
Posts: 412
Threads: 22
Joined: Nov 2010
Reputation: 8
#10
07-26-2012, 03:36 AM
Aside from the fact that using a custom patched taglib is retarded.
How to Build Nightingale | Nightingale on GitHub
Find
Reply
« Next Oldest | Next Newest »


  • View a Printable Version
  • Subscribe to this thread
Forum Jump:


Users browsing this thread: 1 Guest(s)
  • Return to Top
  • Lite (Archive) Mode
  • RSS Syndication
Current time: 03-25-2023, 06:19 PM Powered By MyBB, © 2002-2023 MyBB Group.
Design By AliReza_Tofighi In WhiteCrow Software Group.
white outlined nightingale project logo

Nightingale is free!
It is an Open Source project released under the terms of the GNU General Public License v2 (GPL v2).
For more details, please read the license information.

Follow us!
f  g  t

Support
  • Community Forum
  • Official Blog
  • Add-ons
  • Wiki
  • Help Forum
Contribute
  • Developer's Center
  • Translate Nightingale
  • Source Code
  • Report a Bug
Ressources
  • Download Nightingale
Linear Mode
Threaded Mode