I discovered when writing the new init script (See: https://bugs.gentoo.org/show_bug.cgi?id=483786) for sabnzbd that the sabnzbd.ini has DOS line endings (CRLF, ^M) in it. DOS line endings are a pain to deal with, especially in regexps. We should run dos2unix on the sabnzbd.ini file to get Unix line endings.
Fixed without a revision bump. Thanks for the report!
(In reply to Jeroen Roovers from comment #1) > Fixed without a revision bump. Thanks for the report! Hmm, can you explain how this is fixed? Having thought about this a bit more I'm not sure if it's as simple as running dos2unix since sabnzbd will be writing to that file it will most likely write out CRLFs at the end. This may be a bug to raise against the sabnzbd developers so that they detect the OS environment and write the correct line endings. Sigh, this means yet another modification to my init script!
(In reply to eponymous from comment #2) > (In reply to Jeroen Roovers from comment #1) > Hmm, can you explain how this is fixed? cvs/gentoo-x86/net-nntp/sabnzbd $ dos2unix files/sabnzbd.ini dos2unix: converting file files/sabnzbd.ini to Unix format ... > Having thought about this a bit more I'm not sure if it's as simple as > running dos2unix since sabnzbd will be writing to that file it will most > likely write out CRLFs at the end. > > This may be a bug to raise against the sabnzbd developers so that they > detect the OS environment and write the correct line endings. I guess if the daemon does indeed write to /etc/sabnzbd.ini, that is a design bug that should be fixed upstream. > Sigh, this means yet another modification to my init script! How?
(In reply to Jeroen Roovers from comment #3) > (In reply to eponymous from comment #2) > > (In reply to Jeroen Roovers from comment #1) > > Hmm, can you explain how this is fixed? > > cvs/gentoo-x86/net-nntp/sabnzbd $ dos2unix files/sabnzbd.ini > dos2unix: converting file files/sabnzbd.ini to Unix format ... > Ah fair enough. I did do a re-fetch of the source and a "vim -b" on the file but couldn't see the "DOSless" result. > > Having thought about this a bit more I'm not sure if it's as simple as > > running dos2unix since sabnzbd will be writing to that file it will most > > likely write out CRLFs at the end. > > > > This may be a bug to raise against the sabnzbd developers so that they > > detect the OS environment and write the correct line endings. > > I guess if the daemon does indeed write to /etc/sabnzbd.ini, that is a > design bug that should be fixed upstream. > Yeah, I've confirmed it does so it can be raised with them. > > Sigh, this means yet another modification to my init script! > > How? It's no biggie :) I've submitted a change to the init script to make it explicitly line-ending agnostic so we now don't care about the format of sabnzbd.ini.
